forked from auto-pts/auto-pts
-
Notifications
You must be signed in to change notification settings - Fork 7
Upmerge 06 25 #28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Upmerge 06 25 #28
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
170155 is resolved now.
Keep code up to code.
Add dummy value LE supported features to GAP initialization.
Some le-audio services have write encrypted permissions and were breaking security tests, as writing to them resulted in application errors. Added option for disabling le-audio support.
No. 248 WID was no present for GAP layer. Added wid with proper handling WID expects to return handle of a characteristic with specified permissions.
This updates the current BlueZ AutoPTS support, by fixing IUT control handling and updating the GAP and SM scripts for latest PTS and BlueZ support, to be able to run the test cases.
Current description is bogus (likely copy-paste error from Periodic Report event) and makes no sense for PAST received event. Since event content is not currently used by autoPTS we can fix this. Only upstream user (Mynewt) is not following current doc for this event anyway. While there are couple other parameters that could be added to this event for now lets keep it in line with detail level of Periodic Sync Established event.
This requires only local periodic advertising to be enabled. No need to synchronize to peer's padv.
This improves tests stability as 100ms might be not enough to sync to PTS PADV (PTS seems to be using 150ms interval).
Add errata for testcase. This test case fails if we support only one L2CAP channel. Specification states "one or more".
Previous command was failing due to incorrect formatting ">" symbol was passed to btmon causing and error.
Modification README.md Add poetry_tools.py for check and fix code with PEP8 rules Add new workflows yml file for git Add pyproject.toml
Modify wid files to prevent cyclical imports
The method _get_response_safely was missing the 'self' parameter, which would cause a runtime error when called on an instance.
The recent PEP8-related changes in the client_bot module introduced syntax and structural issues that caused runtime failures. This commit reverts them to the previous working state. Further formatting will be applied after thorough testing.
This adds command for explicit database initialization.
Because the wid 52 will be called after the wid 53 called. The case will fail if the function `hdl_wid_53` is executed. Due to the undirectly connectable mode is required by the case, while the advertising mode is non-connectable mode set by handling of wid 53, bypass the handling of wid 53 to fix the issue. Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Enable bondable flag for the following cases, GAP/SEC/SEM/BV-50-C, GAP/SEC/SEM/BV-51-C, GAP/SEC/SEM/BV-52-C, GAP/SEC/SEM/BV-53-C Signed-off-by: Lyle Zhu <[email protected]>
The global variable `SM_ACL_DISCONN_ROUND` is used to save the current number times of ACL disconnection. The default value of the `SM_ACL_DISCONN_ROUND` should be 0 when the test case is starting. But when testing more than one case, the initialization value of the variable SM_ACL_DISCONN_ROUND is not 0, because the previous status is not cleared. Add a global variable `SM_TEST_CASE_NAME` to save the test case name corresponding to the variable `SM_ACL_DISCONN_ROUND`. When the test case name is not same as the current test case name, clear the variable `SM_ACL_DISCONN_ROUND` because it is a new test sequence of new test case. Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Signed-off-by: Lyle Zhu <[email protected]>
Add a check for cases where the PR comments API returns a non-list response,
e.g. an error message as a string. Fixes exception:
File "/home/codecoup/auto-pts/tools/cron/github_cron.py", line 202, in run
tagged_comments = self.get_pr_comments_with_magic_tag()
File "/home/codecoup/auto-pts/tools/cron/github_cron.py", line 166, in get_pr_comments_with_magic_tag
comment_id = comment['id']
TypeError: string indices must be integers
Plain 'terminate()' was insufficient, mostly with Zephyr, when btmon was started with shell pipelines. Now we start the subprocess within a process group. Using 'os.killpg' and 'stop()' ensures that all child process, are terminated - this resolves lingering btmon processes after test teardown.
Adjusted the btmon command to exclude unnecessary lines from the plain log, reducing noise and improving clarity. This ensures that only relevant output is captured for analysis. This was needed due to Zephyr dumping logs from tester app into btsnoop log.
Add missing return that was accidentally deleted.
PTS is not behaving correctly if EATT is used in that test.
This includes update to TCRL 2025-2 and sync ICS with corresponding Zephyr qualification workspace.
There were some leftovers with updating removed IXIT values.
Those are created by IDE when opening project.
Don't use DOS newlines.
This enables more features supported by Zephyr.
Some erratas were integrated but there are some regressions in latest PTS release.
Qualification Workspace bug is not exporting those correctly.
This WID requires context from previous MMIs and is used to pass size of characteristic value to LT. It is used in following tests: GATT/SR/GAW/BI-32-C GATT/SR/GAW/BI-33-C GATT/SR/GAW/BI-39-C GATT/SR/GAW/BI-40-C
Some Edit MMI style WIDs are used to enter values and boolean return would be converted to integer anyway.
m-alperen-sener
approved these changes
Jun 16, 2025
Collaborator
m-alperen-sener
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.