featureprofiles icon indicating copy to clipboard operation
featureprofiles copied to clipboard

Transceiver13: Changes to zr_low_power_mode test

Open hattikals opened this issue 1 year ago • 1 comments

  • Changed readme to also accept -40dbm as outputpower in low power mode. As per https://github.com/openconfig/featureprofiles/tree/main/feature/platform/transceiver/tests/zr_input_output_power_test test -40 dbm is correct value when interface is down.
  • Changed the components path to use opticalComponent name instead of using transceiver-name for opticalChannelComponent paths.
  • Corrected the logic to get the opticalComponentName, in accordance with the YANG documentation. In the previous logic the code was to get the optical component name by traversing through the parents of the PORT component, this was leading to no results. Hence changed it to find the OPTICAL_CHANNEL comp first and match its parent against HARDWARE_PORT of the interface. Current flow : PORT <-- TRANSCIEVER <-- OPTICAL_CHANNEL

hattikals avatar Jun 07 '24 17:06 hattikals

Pull Request Test Coverage Report for Build 9420729182

Details

  • 0 of 17 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.06%) to 55.438%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/components/components.go 0 17 0.0%
<!-- Total: 0 17
Totals Coverage Status
Change from base Build 9420403184: -0.06%
Covered Lines: 1983
Relevant Lines: 3577

💛 - Coveralls

coveralls avatar Jun 07 '24 17:06 coveralls

Pull Request Test Coverage Report for Build 11340401732

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 55.268%

Totals Coverage Status
Change from base Build 11337664428: 0.0%
Covered Lines: 1983
Relevant Lines: 3588

💛 - Coveralls

coveralls avatar Oct 15 '24 06:10 coveralls