forked from auto-pts/auto-pts
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upmerge 2024 11 27 #21
Open
AyturkDuzen
wants to merge
32
commits into
nrfconnect:master
Choose a base branch
from
AyturkDuzen:upmerge_2024_11_27
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
This enabled PAST related tests in BAP.
Added venv and .venv which are common names for python virtual environments to the gitignore. Signed-off-by: Emil Gydesen <[email protected]>
Added .vscode to gitignore so that local editor settings are not pushed to the repo. Signed-off-by: Emil Gydesen <[email protected]>
The tmp directory is creating during manual testing and should never be pushed. Signed-off-by: Emil Gydesen <[email protected]>
Fix following: autopts/wid/tmap.py:718: SyntaxWarning: invalid escape sequence '\d'
This was due to bug in Qualification Workspace.
The test also requires discovery and was failing due to this. Re-arranged the list of tests to be in alphabetical order to make it easier to follow. Signed-off-by: Emil Gydesen <[email protected]>
Redefine the btp service values from decimal to hexidecimal, so they are aligned with opcodes, indexes, etc. Signed-off-by: Emil Gydesen <[email protected]>
Log BTP headers as hex instead of decimal so it is easier to compare to the defined values in defs.py Since the headers are using a namedtuple instead of a class, it is not straigh forward to just update the str or repr functions of the class. Since printing a namedtuple uses the repr of the class, a new function, repr_hdr, was added to format the header as a string as hex. Signed-off-by: Emil Gydesen <[email protected]>
OVERLAY_CONFIG was deprected in Zephyr 3.4: https://docs.zephyrproject.org/latest/releases/release-notes-3.4.html#build-system-and-infrastructure
The following tests require 2 CCIDs, and are failing if the device does not have have 2 CCIDs registered: CAP/INI/BST/BV-07-C CAP/INI/BST/BV-10-C CAP/INI/UST/BV-13-C CAP/INI/UST/BV-14-C CAP/INI/UST/BV-27-C CAP/INI/UST/BV-28-C CAP/INI/UST/BV-31-C CAP/INI/UST/BV-36-C CAP/INI/UST/BV-37-C CAP/INI/UTB/BV-04-C Signed-off-by: Emil Gydesen <[email protected]>
Make this consistent with rest of opcodes.
30 decimal is 0x1e hex.
The Request ID 147314 applies to nearly all tests that uses multiple lower testers, as it affects how IRKs are handled by PTS. Signed-off-by: Emil Gydesen <[email protected]>
If a PTS dongle has been corrupted, its stack has to be reinitialized. The PTS apparently can do this by calling GetPTSBluetoothAddress() before RunTestCase(). The dongle verification takes less than 100ms on a real-hardware machine, but for VM it might take longer. Let's add an option --dongle_init_delay that allows to customize this time.
It happens rarely, but sometimes a socket port does not get released in time and is not usable for a while. File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 653, in run_test _run_test(config) File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 644, in _run_test srv_process, bot_process = _restart_processes(config) File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 582, in _restart_processes return _start_processes(config, checkout_repos=False) File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 557, in _start_processes start_remote_autoptsserver(config, checkout_repos) File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 474, in start_remote_autoptsserver log(client.open_process(config['server_start_cmd'], File "/usr/lib/python3.10/xmlrpc/client.py", line 1122, in __call__ return self.__send(self.__name, args) self.sock = self._create_connection( (...) File "/usr/lib/python3.10/socket.py", line 833, in create_connection sock.connect(sa) OSError: [Errno 113] No route to host
When running autoptsclient-zephyr.py --help, traceback was printed in terminal. This was caused by logging.exception(e) which should not be called when exiting with status 0.
There is no point in repeating test case if MISSING WID ERROR is returned.
These tests (and possibly more) makes the PTS dongle crash. Tested with both SDC and ZLL with the Zephyr host, using sha 67980643d799f76aff93118346a8dabfb472a642. Signed-off-by: Emil Gydesen <[email protected]>
Modified the hdl_wid_312 handler to call cap_unicast_audio_stop with release = False, so that it performs a disable (and stop) without release. Why PTS wants us to disable first is unclear, as the test spec states to release, which would be handled by hdl_wid_309. This affects tests where PTS asks us to stop/disable the streams (hdl_wid_312) before releasing it (hdl_wid_309), such as CAP/INI/UST/BV-40-C. Signed-off-by: Emil Gydesen <[email protected]>
Add sysbuild option for 54h20 Pass --no-sysbuild for 5340 same as old behaviour Signed-off-by: Aytürk Düzen <[email protected]>
rename hci_ipc configuration for generic multicore usage. related change in tester: zephyrproject-rtos/zephyr#81755 Signed-off-by: Aytürk Düzen <[email protected]>
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.
upmerge from auto-pts/master, support for nRF54H20 Lilium