Skip to content

gh453 change the test flow for L3 dsAudio_test24#482

Open
revram16 wants to merge 2 commits intodevelopfrom
feature/gh_453_Change_test_flow_dsAudio_Test24
Open

gh453 change the test flow for L3 dsAudio_test24#482
revram16 wants to merge 2 commits intodevelopfrom
feature/gh_453_Change_test_flow_dsAudio_Test24

Conversation

@revram16
Copy link
Contributor

Modified the flow of iterating primary and secondary languages.
As suggested, done separate iteration for primary and secondary languages.

@github-actions
Copy link

github-actions bot commented Jul 22, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@kanjoe24
Copy link
Contributor

kanjoe24 commented Nov 4, 2025

recheck

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR restructures the Primary and Secondary language test to separate them into distinct test phases. The code now:

  1. Tests secondary languages first, then disables ports before moving to primary language testing
  2. Adds a sleep delay between secondary and primary language test phases
  3. Removes a trailing blank line at the end of the test function

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +122 to +124

time.sleep(3)

Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line 122 uses tab characters for indentation instead of spaces. Python PEP 8 requires spaces for indentation. Replace the tab with appropriate spaces to maintain consistent indentation.

Suggested change
time.sleep(3)
# (blank line preserved, now with spaces)
time.sleep(3)

Copilot uses AI. Check for mistakes.
Comment on lines +122 to +124

time.sleep(3)

Copy link

Copilot AI Nov 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line 124 uses tab characters for indentation instead of spaces. Python PEP 8 requires spaces for indentation. Replace the tab with appropriate spaces to maintain consistent indentation.

Suggested change
time.sleep(3)
time.sleep(3)

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Review Requested

Development

Successfully merging this pull request may close these issues.

change the test flow for L3 dsAudio_test24_PrimarySecondaryLanguage.py

3 participants