Skip to content
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

Update YAML file Test_TC_OPCREDS_3_7 #36871

Merged

Conversation

juandediosg
Copy link
Contributor

This PR updates the TC_OPCREDS_3_7 YAML file by removing steps to align the test procedure with the Fixes: #423

Updates:

  • The YAML test script was updated to reflect these changes.
  • The TC_OPCREDS_3_7 test procedure in the test plan has been updated in PR #4820.

Testing for Multiple Fabrics:

This test validates the TC_OPCREDS_3_7 procedure when two fabrics, gamma and the default alpha, are present on the Device Under Test (DUT). The process includes commissioning the DUT with gamma using the --commissioner-name gamma flag, opening a commissioning window, re-commissioning with alpha, and confirming the test passes successfully with both fabrics in use.

Example of Test Steps (Passed):

  1. In the first console, Run the all-clusters app:
    rm /tmp/chip_*                                        
    ./out/darwin-arm64-all-clusters/chip-all-clusters-app
  2. In the second console, Perform the first commissioning:
    ./out/darwin-arm64-chip-tool/chip-tool pairing code 0x12344321 749701123365521327694 --commissioner-name gamma
    Note: As example I use the manual pairing code 749701123365521327694 instead of the QR code payload ID MT:Y.K9042C00KA0648G00.
    
  3. Opened the commissioning window and searched for "Manual pairing code" in the output:
    ./out/darwin-arm64-chip-tool/chip-tool pairing open-commissioning-window 0x12344321 1 300 1000 2365 --commissioner-name gamma
  4. Re-commissioned with a second commissioner (alpha):
    ./out/darwin-arm64-chip-tool/chip-tool pairing code 0x12344321 23064351703 --commissioner-name alpha
  5. In the third console, Run the test:
    ./scripts/tests/chipyaml/chiptool.py tests Test_TC_OPCREDS_3_7 --server_path ./out/darwin-arm64-chip-tool/chip-tool.
  6. As result:
    ***** Test Complete: Test_TC_OPCREDS_3_7
    ✓ Test finished in 1344ms with 35 success, 0 errors and 0 warnings
    ✓ Parsing finished in 8ms with 1 success and 0 errors.
    Run finished in 2548ms with 16 steps runned and 0 steps skipped.
    

Changes made:

  • Removed the factory reset from the YAML file.
  • Adjusted the test procedure to align with the updated format from the linked test plan PR.

Copy link

Review changes with  SemanticDiff

@CLAassistant
Copy link

CLAassistant commented Dec 17, 2024

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added app tests matter-1.4-te2-script-change Script changes before end of Matter 1.4 TE2 labels Dec 17, 2024
@juandediosg juandediosg marked this pull request as ready for review December 18, 2024 21:12
@mergify mergify bot merged commit b5313a7 into project-chip:master Jan 15, 2025
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app fast track matter-1.4-te2-script-change Script changes before end of Matter 1.4 TE2 tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants