-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
## Automation REFALM_2_2 ## #37108
base: master
Are you sure you want to change the base?
## Automation REFALM_2_2 ## #37108
Conversation
Changed Files
|
As mentioned I raised an issue related to executing arbitrary commands. Link to issue: |
3ce3206
to
836c55d
Compare
PR #37108: Size comparison from eea382e to 1b42ced Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left minor comments that would be nice to have. Overall looks good
This test verify the basic functionality of the DUT as a server. Added custom manual actions for Dooropen. Implemented Fake Methods to run disallowd commands. Assert Status Attribute. Assert Status attribute from the Notify event. Added TODO for FakeMethods. Issue raised to add method to run arbitraty commands.
… for wait threshold. Added missing copyright. Added missing docstring.
1b42ced
to
f0138e4
Compare
PR #37108: Size comparison from da6a512 to f0138e4 Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #37108: Size comparison from 5621ff6 to 6a4df79 Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nxp, psoc6, qpg, stm32, telink, tizen)
|
…o test steps. Removed ci check to command send. Added 32bit value check for State attribute. Reduced tests steps as the test plan.
Description
This is the proposal to Automate the REFALM_2_2
This test case verifies the primary functionality of the Refrigerator Alarm Cluster server.
Manual actions are performed in order to verify the device Attributes.
Verify the commands from the device are Disallowed as described in the spec.
Subscribe to Notify event and wait for the event data.
Testing
Verified by ci with all clusters app:
./out/linux-x64-all-clusters/chip-all-clusters-app
Run the tests providing the app-pid of the clusters to send manual actions.
python3 src/python_testing/TC_REFALM_2_2.py --commissioning-method on-network --qr-code MT:-24J0AFN00KA0648G00 --PICS src/app/tests/suites/certification/ci-pics-values --int-arg PIXIT.REFALM.AlarmThreshold:5 --app-pid $(ps waux | grep all-clusters | grep -v grep | awk '{print $2}' )