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

[CERT-TEST-FAILURE] - TC-EPM-2.1 - Test failed #31925

Closed
Survensa opened this issue Feb 5, 2024 · 0 comments · Fixed by #32095
Closed

[CERT-TEST-FAILURE] - TC-EPM-2.1 - Test failed #31925

Survensa opened this issue Feb 5, 2024 · 0 comments · Fixed by #32095
Labels
bug Something isn't working cert blocker needs triage

Comments

@Survensa
Copy link

Survensa commented Feb 5, 2024

Feature Area

Other

Test Case

TC-EPM-2.1

Reproduction steps

  1. Bring up the DUT with All-clusters-app.
  2. Pair the DUT using chip-tool through "Onnetwork" transport.
  3. Run the commands ./chip-tool electricalpowermeasurement read harmonic-currents 1 1 and ./chip-tool electricalpowermeasurement read harmonic-phases 1 1

Expected Outcome: Response that contains a list of HarmonicMeasurementStruct entries with mandatory fields.

Actual Outcome:

[1707122207.106306][24068:24070] CHIP:TOO: HarmonicCurrents: 0 entries

[1707122225.834546][24072:24074] CHIP:TOO: HarmonicPhases: 0 entries

Bug prevalence

Whenever I do this

GitHub hash of the SDK that was being used

83f8665

Platform

other

Anything else?

As per spec, the HarmonicCurrents and HarmonicPhases attribute response should contains a list of HarmonicMeasurementStruct entries Order Measurement field as it is mandatory.

Specification Reference:

image
image
@Survensa Survensa added bug Something isn't working cert blocker needs triage labels Feb 5, 2024
jamesharrow added a commit that referenced this issue Feb 13, 2024
@jamesharrow
Fixes #31925 - QA team expect HarmonicCurrents and HarmonicPhases to …
c2cdde3 
…have 1 entry in the list for testing
@jamesharrow jamesharrow mentioned this issue Feb 13, 2024
Closed
2 tasks
@jamesharrow jamesharrow linked a pull request Feb 13, 2024 that will close this issue
QA team requested that EPM cluster has at least 1 entry in the list for HarmonicCurrent and HarmonicPhases #32103
Closed
2 tasks
@jamesharrow jamesharrow mentioned this issue Feb 13, 2024
Merged
5 tasks
@mergify mergify bot closed this as completed in #32095 Feb 14, 2024
@mergify mergify bot closed this as completed in 820034e Feb 14, 2024
raul-marquez-csa pushed a commit to raul-marquez-csa/connectedhomeip that referenced this issue Feb 16, 2024
@jamesharrow @raul-marquez-csa
QA team requested that all attributes in EPM cluster be turned on in …
38c3610 
…energy-management-app (project-chip#32095)

* Fixes: project-chip#32089
In all-clusters-app, Set the EPM PowerMode attribute to AC. In energy-management-app turned on all features and attributes for testing. Fixed issue in TC_EPM_2_1.py when valid attribute is returned. Improved type checking for HarmonicCurrents and HarmonicPhases

* Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes.

* Turned on additional attributes in EPM cluster on energy-management-app. Now runs tests without skipping unsupported attributes.

* Fixes project-chip#31925 - QA team expect HarmonicCurrents and HarmonicPhases to have 1 entry in the list for testing

* Turned off 3 provisional features which fail the TC_DeviceConformance.py test

* Removed EVSE attributes which are provisional in 1.3 (SOC, PNC, V2X)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working cert blocker needs triage
Projects
[Certification] Blockers
Archived in project
Milestone
No milestone
1 participant
@Survensa