Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the README.md to clarify the BeToCQ test framework, its test categories, and validation targets (Quick Start and AQT). It also updates specific test case names, standardizes default Wi-Fi configurations, and provides updated commands for running the onboarding and AQT test suites. The review feedback suggests correcting a directory name reference to compound_tests, fixing a minor grammatical error, and resolving a formatting issue in the execution instructions.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| * **Directed test**: Measures performance of each wireless medium against expectations. To discover radio concurrency issues, sometimes multiple mediums are enabled simultaneously during the test. | ||
| * The function and directed tests are the foundational tests running with fixed wireless mediums. This helps isolate issues to an individual medium, making the debugging process more straightforward. | ||
| * **Critical user journey (CUJ) test**: Tests real-world use cases. Unlike function and directed tests, CUJ tests use multiple radios in a dynamic way. Debugging is typically more complex in CUJ tests, so they are executed after the foundational tests have passed. | ||
| * CUJ tests are implemented in the `compound_test` directory and are dynamically configured based on the CUJ requirements. |
There was a problem hiding this comment.
The directory name is compound_tests (plural), not compound_test (singular), as seen in the codebase imports.
| at the same time: | ||
|
|
||
| **Note**: For betocq_aqt_test_suite, this is not allowed, all 2G, 5G and 5G DFS APs should be available. | ||
| **Note**: For betocq_aqt_test_suite, this is not allowed, all 2G, 5G and 5G DFS APs should be available; 5G or 5G DFS APs are required only if the device support 5G. |
There was a problem hiding this comment.
Fix a grammatical error: "device support 5G" should be "device supports 5G".
| **Note**: For betocq_aqt_test_suite, this is not allowed, all 2G, 5G and 5G DFS APs should be available; 5G or 5G DFS APs are required only if the device support 5G. | |
| **Note**: For betocq_aqt_test_suite, this is not allowed, all 2G, 5G and 5G DFS APs should be available; 5G or 5G DFS APs are required only if the device supports 5G. |
| * **To run the AQT (GTS - only for debugging, refer to GTS instructions for | ||
| formal submission) :** |
There was a problem hiding this comment.
Remove the extra space before the colon in the bold text formatting.
| * **To run the AQT (GTS - only for debugging, refer to GTS instructions for | |
| formal submission) :** | |
| * **To run the AQT (GTS - only for debugging, refer to GTS instructions for | |
| formal submission):** |
2 participants
No description provided.