Fix YAML script issues #29339
Merged
Fix YAML script issues #29339
+594
−2,770
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
PR #29339: Size comparison from b516ff4 to 5b1380d Full report (66 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
|
PR #29339: Size comparison from b516ff4 to 69dc8f1 Full report (66 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
bzbarsky-apple
requested changes
Sep 20, 2023
There was a problem hiding this comment.
The PR claims to address https://github.com/CHIP-Specifications/chip-test-scripts/issues/655 but I see no changes to those test files here.... Please either fix the PR description or fix the PR.
manjunath-grl
force-pushed
the
Fix_Script_Issues_Sep_15
branch
from
82194fb
to
b59d55e
Compare
Updated the ACL YAML's according to issue 655. |
|
PR #29339: Size comparison from bfa9b4c to b59d55e Increases (1 build for telink)
Full report (66 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
manjunath-grl
force-pushed
the
Fix_Script_Issues_Sep_15
branch
from
b59d55e
to
6ba5e68
Compare
pullapprove
bot
requested review from
andy31415,
andyg-apple,
anush-apple,
arkq,
carol-apple,
cecille,
chrisdecenzo,
chshu,
chulspro,
cliffamzn,
Damian-Nordic,
dhrishi,
electrocucaracha,
gjc13,
harsha-rajendran and
hawk248
manjunath-grl
force-pushed
the
Fix_Script_Issues_Sep_15
branch
from
99ac969
to
e50bcfb
Compare
|
PR #29339: Size comparison from 433e676 to e50bcfb Full report (52 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg)
|
|
PR #29339: Size comparison from 433e676 to 5918e0e Full report (65 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
bzbarsky-apple
approved these changes
Sep 29, 2023
Modified tests: TC_ACL_2_10 TC_ACL_2_7 TC_ACL_2_8 TC_DGWIFI_3_2_Simulated TC_FLW_2_1 TC_LUNIT_3_1 TC_RH_2_1 TC_SC_4_1 Automated test: TC_RVCCLEANM_3_3 Semi-automated tests: TC_REFALM_2_1 Fixes issue: 29331
since there are Python script for the testcases. TC-DT-1.1 TC-IDM-10.1 TC-IDM-11.1 TC-SM-1.1 TC-SM-1.2
Modified tests: TC-GRPKEY-2.1 Removed test: TC-LVL-2.3_Simulated
Modified tests: TC-ACL-2.4 TC-ACL-2.5 TC-ACL-2.6 TC-ACL-2.7 TC-SC-5.2 Revert back the DGGEN-2.1 testcase
TC-DGGEN-2.1 TC-TSTAT-1.1 TC-TSUIC-1.1 TC-TSTAT-2.1 TC-WNCV-3.4 TC-WNCV-3.5 Manual tests: TC-REFALM-2.2 TC-REFALM-2.3
TC-TCTL-1.1
TC-DGGEN-2.1 TC-REFALM-2.1 TC-RVCCLEANM-3.3
manjunath-grl
force-pushed
the
Fix_Script_Issues_Sep_15
branch
from
5918e0e
to
8608cfd
Compare
|
PR #29339: Size comparison from 6bd473d to 8608cfd Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
raju-apple
approved these changes
Sep 29, 2023
andy31415
approved these changes
Sep 30, 2023
HunsupJung
pushed a commit
to HunsupJung/connectedhomeip
that referenced
this pull request
Oct 23, 2023
* Fix script issue: 649, 650, 651, 652, 653, 654, Modified tests: TC_ACL_2_10 TC_ACL_2_7 TC_ACL_2_8 TC_DGWIFI_3_2_Simulated TC_FLW_2_1 TC_LUNIT_3_1 TC_RH_2_1 TC_SC_4_1 Automated test: TC_RVCCLEANM_3_3 Semi-automated tests: TC_REFALM_2_1 Fixes issue: 29331 * Auto generated files * The following YAMLs are removed since there are Python script for the testcases. TC-DT-1.1 TC-IDM-10.1 TC-IDM-11.1 TC-SM-1.1 TC-SM-1.2 * Fixes issue: 656 & 658 Modified tests: TC-GRPKEY-2.1 Removed test: TC-LVL-2.3_Simulated * Modified testcase TC-SC-5.1 * Fixes issue: 655, 660 Modified tests: TC-ACL-2.4 TC-ACL-2.5 TC-ACL-2.6 TC-ACL-2.7 TC-SC-5.2 Revert back the DGGEN-2.1 testcase * Modified test FLW-2.1 * Fixes YAML script issue: 664, 668, 671, 672, 673 TC-DGGEN-2.1 TC-TSTAT-1.1 TC-TSUIC-1.1 TC-TSTAT-2.1 TC-WNCV-3.4 TC-WNCV-3.5 Manual tests: TC-REFALM-2.2 TC-REFALM-2.3 * Restyled by whitespace * Fixes issue 984 TC-TCTL-1.1 * Reverted back changes for tests: TC-DGGEN-2.1 TC-REFALM-2.1 TC-RVCCLEANM-3.3 * Modify ciTest.json file * Fix type --------- Co-authored-by: Restyled.io <commits@restyled.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes issue: 29243, 649, 650, 651, 652, 653, 654, 655, 656, 658, 660, 664, 668, 671, 672, 673
Modified tests:
Modified manual testcases:
Removed manual testcases:
Removed simulated testcase:
TC_LVL_2_3_SimulatedAdded auto generated files
Tested
exe_logs.txt