Initialize unsupported values to 'N/A' in CmisApi::get_transceiver_info

sonic_platform_base/sonic_xcvr/api/public/cmis.py

@@ -82,6 +82,49 @@ class CmisApi(XcvrApi):
     LowPwrRequestSW = 4
     LowPwrAllowRequestHW = 6
 
+    cmis_xcvr_info_dict = {
+        "type": "N/A",
+        "type_abbrv_name": "N/A",
+        "hardware_rev": "N/A",
+        "serial": "N/A",
+        "manufacturer": "N/A",
+        "model": "N/A",
+        "connector": "N/A",
+        "encoding": "N/A",
+        "ext_identifier": "N/A",
+        "ext_rateselect_compliance": "N/A",
+        "cable_length": 0.0,
+        "nominal_bit_rate": 0,
+        "vendor_date": "N/A",
+        "vendor_oui": "N/A",
+        "is_replaceable": "N/A",
+        "dom_capability": "N/A",
+        "active_apsel_hostlane1": "N/A",
+        "active_apsel_hostlane2": "N/A",
+        "active_apsel_hostlane3": "N/A",
+        "active_apsel_hostlane4": "N/A",
+        "active_apsel_hostlane5": "N/A",
+        "active_apsel_hostlane6": "N/A",
+        "active_apsel_hostlane7": "N/A",
+        "active_apsel_hostlane8": "N/A",
+        "supported_max_tx_power": "N/A",
+        "supported_min_tx_power": "N/A",
+        "supported_max_laser_freq": "N/A",
+        "supported_min_laser_freq": "N/A",
+        "application_advertisement": "N/A",
+        "host_electrical_interface": "N/A",
+        "media_interface_code": "N/A",
+        "host_lane_count": 0,
+        "media_lane_count": "N/A",
+        "host_lane_assignment_option": "N/A",
+        "media_lane_assignment_option": "N/A",
+        "cable_type": "N/A",
+        "media_interface_technology": "N/A",
+        "vendor_rev": 0.0,
+        "cmis_rev": 0.0,
+        "specification_compliance": "N/A"
+    }
+
     def __init__(self, xcvr_eeprom):
         super(CmisApi, self).__init__(xcvr_eeprom)
         self.vdm = CmisVdmApi(xcvr_eeprom) if not self.is_flat_memory() else None
@@ -267,50 +310,45 @@ def get_transceiver_info(self):
         ext_id = admin_info[consts.EXT_ID_FIELD]
         power_class = ext_id[consts.POWER_CLASS_FIELD]
         max_power = ext_id[consts.MAX_POWER_FIELD]
-
-        xcvr_info = {
+        self.cmis_xcvr_info_dict.update({
             "type": admin_info[consts.ID_FIELD],
             "type_abbrv_name": admin_info[consts.ID_ABBRV_FIELD],
             "hardware_rev": self.get_module_hardware_revision(),
             "serial": admin_info[consts.VENDOR_SERIAL_NO_FIELD],
             "manufacturer": admin_info[consts.VENDOR_NAME_FIELD],
             "model": admin_info[consts.VENDOR_PART_NO_FIELD],
             "connector": admin_info[consts.CONNECTOR_FIELD],
-            "encoding": "N/A", # Not supported
             "ext_identifier": "%s (%sW Max)" % (power_class, max_power),
-            "ext_rateselect_compliance": "N/A", # Not supported
             "cable_length": float(admin_info[consts.LENGTH_ASSEMBLY_FIELD]),
-            "nominal_bit_rate": 0, # Not supported
             "vendor_date": admin_info[consts.VENDOR_DATE_FIELD],
-            "vendor_oui": admin_info[consts.VENDOR_OUI_FIELD]
-        }
-        appl_advt = self.get_application_advertisement()
-        xcvr_info['application_advertisement'] = str(appl_advt) if len(appl_advt) > 0 else 'N/A'
-        xcvr_info['host_electrical_interface'] = self.get_host_electrical_interface()
-        xcvr_info['media_interface_code'] = self.get_module_media_interface()
-        xcvr_info['host_lane_count'] = self.get_host_lane_count()
-        xcvr_info['media_lane_count'] = self.get_media_lane_count()
-        xcvr_info['host_lane_assignment_option'] = self.get_host_lane_assignment_option()
-        xcvr_info['media_lane_assignment_option'] = self.get_media_lane_assignment_option()
-        xcvr_info['cable_type'] = self.get_cable_length_type()
+            "vendor_oui": admin_info[consts.VENDOR_OUI_FIELD],
+            "application_advertisement": str(self.get_application_advertisement()) if len(self.get_application_advertisement()) > 0 else 'N/A',
+            "host_electrical_interface": self.get_host_electrical_interface(),
+            "media_interface_code": self.get_module_media_interface(),
+            "host_lane_count": self.get_host_lane_count(),
+            "media_lane_count": self.get_media_lane_count(),
+            "host_lane_assignment_option": self.get_host_lane_assignment_option(),
+            "media_lane_assignment_option": self.get_media_lane_assignment_option(),
+            "cable_type": self.get_cable_length_type(),
+            "media_interface_technology": self.get_media_interface_technology(),
+            "vendor_rev": self.get_vendor_rev(),
+            "cmis_rev": self.get_cmis_rev(),
+            "specification_compliance": self.get_module_media_type()
+        })
         apsel_dict = self.get_active_apsel_hostlane()
-        for lane in range(1, self.NUM_CHANNELS+1):
-            xcvr_info["%s%d" % ("active_apsel_hostlane", lane)] = \
-                    apsel_dict["%s%d" % (consts.ACTIVE_APSEL_HOSTLANE, lane)]
-        xcvr_info['media_interface_technology'] = self.get_media_interface_technology()
-        xcvr_info['vendor_rev'] = self.get_vendor_rev()
-        xcvr_info['cmis_rev'] = self.get_cmis_rev()
-        xcvr_info['specification_compliance'] = self.get_module_media_type()
+        for lane in range(1, self.NUM_CHANNELS + 1):
+            self.cmis_xcvr_info_dict["%s%d" % ("active_apsel_hostlane", lane)] = \
+            apsel_dict["%s%d" % (consts.ACTIVE_APSEL_HOSTLANE, lane)]
 
         # In normal case will get a valid value for each of the fields. If get a 'None' value
         # means there was a failure while reading the EEPROM, either because the EEPROM was
         # not ready yet or experincing some other issues. It shouldn't return a dict with a
         # wrong field value, instead should return a 'None' to indicate to XCVRD that retry is
         # needed.
-        if None in xcvr_info.values():
+        if None in self.cmis_xcvr_info_dict.values():
             return None
         else:
-            return xcvr_info
+            return self.cmis_xcvr_info_dict
 
     def get_transceiver_info_firmware_versions(self):
         return_dict = {"active_firmware" : "N/A", "inactive_firmware" : "N/A"}

tests/sonic_xcvr/test_cmis.py

@@ -1498,7 +1498,13 @@ def test_module_fw_upgrade(self, input_param, mock_response, expected):
                 'media_lane_assignment_option': 1,
                 'connector': 'LC',
                 'host_lane_assignment_option': 1,
-                'vendor_date': '21010100'
+                'vendor_date': '21010100',
+                'dom_capability': 'N/A',
+                'is_replaceable': 'N/A',
+                'supported_max_laser_freq': 'N/A',
+                'supported_max_tx_power': 'N/A',
+                'supported_min_laser_freq': 'N/A',
+                'supported_min_tx_power': 'N/A'
             }
         )
     ])
@@ -1526,11 +1532,11 @@ def test_get_transceiver_info(self, mock_response, expected):
         self.api.get_cmis_rev = MagicMock()
         self.api.get_cmis_rev.return_value = mock_response[10]
         self.api.get_module_fw_info = MagicMock()
+        self.api.get_module_fw_info.return_value = mock_response[14]
         self.api.get_module_media_type = MagicMock()
         self.api.get_module_media_type.return_value = mock_response[13]
         self.api.get_module_hardware_revision = MagicMock()
         self.api.get_module_hardware_revision.return_value = '0.0'
-        self.api.get_module_fw_info.return_value = mock_response[14]
         self.api.is_flat_memory = MagicMock()
         self.api.is_flat_memory.return_value = False
         result = self.api.get_transceiver_info()
@@ -3082,7 +3088,7 @@ def test_get_error_description(self):
             }
             self.api.xcvr_eeprom.read = MagicMock()
             self.api.xcvr_eeprom.read.return_value = 0x10
-    
+
             result = self.api.get_error_description()
             assert result is 'OK'