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.

Sign up for GitHub

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

Jump to bottom

[CNET] Script has to be updated as per test plan #82

Closed
KishokG opened this issue Dec 13, 2023 · 5 comments
Closed

[CNET] Script has to be updated as per test plan #82

KishokG opened this issue Dec 13, 2023 · 5 comments
Assignees
@raul-marquez-csa
Labels
Matter_1.3 PR-Ready This item has a PR addressing the issue

Comments

@KishokG
Copy link

KishokG commented Dec 13, 2023 •
edited
Loading

Description:

Commissioning related Precondition step has to be removed from the script as it is not necessary for below test cases:

  • TC-CNET-4.1
  • TC-CNET-4.2
  • TC-CNET-4.4
  • TC-CNET-4.5
  • TC-CNET-4.6
  • TC-CNET-4.9
  • TC-CNET-4.10
  • TC-CNET-4.11
  • TC-CNET-4.15
  • TC-CNET-4.16
  • TC-CNET-4.22

TC-CNET-4.5:

  • In step 1, breadcrumb has to be changed to Breadcrumb in script.

UI Snapshot:
image
@manjunath-grl manjunath-grl self-assigned this Dec 13, 2023
@manjunath-grl
Copy link
Collaborator

@KishokG fixed in PR 30951 please verify it and close the ticket.

@KishokG
Copy link
Author

KishokG commented Jan 24, 2024

@manjunath-grl

Commissioning related Precondition step has to be removed from the script as it is not necessary for below test cases:

  • TC-CNET-4.1
  • TC-CNET-4.2
  • TC-CNET-4.4
  • TC-CNET-4.5
  • TC-CNET-4.6
  • TC-CNET-4.9
  • TC-CNET-4.10
  • TC-CNET-4.11
  • TC-CNET-4.15
  • TC-CNET-4.16
  • TC-CNET-4.22

@raul-marquez-csa
Copy link

  • TC-CNET-4.1
  • TC-CNET-4.2

TC-CNET-4.1 and TC-CNET-4.2 didn't contain a commissioning precondition step but were removed for the rest in the list.

@raul-marquez-csa raul-marquez-csa mentioned this issue Mar 22, 2024
Merged
@raul-marquez-csa
Copy link

PR
project-chip/connectedhomeip#32659

@raul-marquez-csa raul-marquez-csa added the PR-Ready This item has a PR addressing the issue label Mar 22, 2024
@woody-apple woody-apple mentioned this issue Apr 11, 2024
Merged
@KishokG
Copy link
Author

KishokG commented May 13, 2024

Issue fixed

@KishokG KishokG closed this as completed May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raul-marquez-csa raul-marquez-csa

Labels
Matter_1.3 PR-Ready This item has a PR addressing the issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@raju-apple @manjunath-grl @raul-marquez-csa @KishokG