Joe LeVeque
Joe LeVeque
> Just trying to see, tentatively how long these enhancements/refactoring would take? > That would help us determine as to what alternative we can have think/plan at platform's infra layer...
Unit test infrastructure has been added to the repo via https://github.com/Azure/sonic-platform-common/pull/139. You should now be able to add unit tests.
@Piyushagg19: Thank you for volunteering! I have assigned the issue to you. If you have any questions along the way, please feel free to discuss here.
@Piyushagg19: You'll notice that the `open_eeprom()` method returns a file handle to the EEPROM, which is in turn passed back into most of the methods in the class. It would...
@Piyushagg19: You should be able to use the `decode-syseeprom` utility for testing your changes.
Log into a SONiC device and run `sudo decode-syseeprom --help`: ``` admin@sonic~$ sudo decode-syseeprom --help Usage: /usr/local/bin/decode-syseeprom [-s][-m] Options: -h, --help show this help message and exit -d print eeprom...
Unfortunately, a virtual switch doesn't have an EEPROM, and we currently are not mocking one, so you would need access to a physical switch to work on this.
Retest this please
@qiluo-msft: @randygaulmsft is working on adding the PR check build.
Retest this please