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

Add support for configuring terms and conditions in device commissioning #173

Open
wants to merge 20 commits into
base: v2.12
Choose a base branch
from

Conversation

swan-amazon
Copy link

This commit introduces a new enhanced_setup_flow configuration property to the test harness backend, allowing device manufacturers to define the terms and conditions (T&C) that must be acknowledged and set during device commissioning. These values are critical for supporting the Terms and Conditions Certification Test Suite.

Key changes include:

  • Added the EnhancedSetupFlowConfig class to DutConfig, which allows specifying the T&C version and user response required during commissioning.
  • Updated generate_command_arguments in utils.py to append the necessary arguments for T&C validation and the device's commissioning method.
  • Adjusted model validation to ensure the enhanced_setup_flow field is processed correctly within DutConfig.

These changes ensure the test harness can enforce manufacturer-specific T&C values during the commissioning process, aligning with the broader requirements of T&C validation.

This commit introduces a new `enhanced_setup_flow` configuration
property to the test harness backend, allowing device manufacturers to
define the terms and conditions (T&C) that must be acknowledged and set
during device commissioning. These values are critical for supporting
the Terms and Conditions Certification Test Suite.

Key changes include:
- Added the `EnhancedSetupFlowConfig` class to `DutConfig`, which allows
  specifying the T&C version and user response required during
  commissioning.
- Updated `generate_command_arguments` in `utils.py` to append the
  necessary arguments for T&C validation and the device's commissioning
  method.
- Adjusted model validation to ensure the `enhanced_setup_flow` field is
  processed correctly within `DutConfig`.

These changes ensure the test harness can enforce manufacturer-specific
T&C values during the commissioning process, aligning with the broader
requirements of T&C validation.
@rquidute rquidute changed the base branch from main to v2.12+winter2024 December 17, 2024 17:14
@rquidute rquidute deleted the branch project-chip:v2.12 December 17, 2024 17:23
@rquidute rquidute closed this Dec 17, 2024
@rquidute rquidute reopened this Dec 17, 2024
@rquidute rquidute changed the base branch from v2.12+winter2024 to v2.12 December 17, 2024 17:26
Copy link
Contributor

@rquidute rquidute left a comment

Choose a reason for hiding this comment

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

Please check some comments in the PR

@rquidute
Copy link
Contributor

rquidute commented Dec 17, 2024

Hi @swan-amazon I changed the target from main to v2.12.
Please check my comments in the PR and also check the Unit Tests tasks which is failing.
The Check spelling task you don't need to worry about it.
Thanks in advance.

@swan-amazon
Copy link
Author

Hi @swan-amazon I changed the target from main to v2.12. Please check my comments in the PR and also check the Unit Tests tasks which is failing. The Check spelling task you don't need to worry about it. Thanks in advance.

Hey @rquidute , I'm not sure why those spell checks weren't failing previously. I didn't add the words that it's flagging on now... I was asking Raju but he didn't know. I asked if something was recently enabled and he doesn't think so. What should we do?

Copy link
Contributor

@rquidute rquidute left a comment

Choose a reason for hiding this comment

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

By now, just minor items and also the Unit Tests must be fixed.

@rquidute
Copy link
Contributor

Hi @swan-amazon I changed the target from main to v2.12. Please check my comments in the PR and also check the Unit Tests tasks which is failing. The Check spelling task you don't need to worry about it. Thanks in advance.

Hey @rquidute , I'm not sure why those spell checks weren't failing previously. I didn't add the words that it's flagging on now... I was asking Raju but he didn't know. I asked if something was recently enabled and he doesn't think so. What should we do?

Hi @swan-amazon don't worry about spelling issue in this PR. There is still just one spelling issue, but this is a temp code. This is something we need to fix.

…ning validation

Add new parameter to control commissioning flow during tests, allowing manual
validation of the commissioning process instead of auto-commissioning. This
enables test cases to verify terms and conditions acknowledgements during the
commissioning flow.

- Add in-test-commissioning-method parameter to test flags
- Support manual commissioning flow validation
- Enable testing of T&C acknowledgement scenarios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants