Address Code Review

Author: yashnap

src/core/src/bootstrap/ConfigurationFactory.py

@@ -71,19 +71,19 @@ def __init__(self, log_file_path, events_folder, telemetry_supported):
 
         self.configurations = {
             'apt_prod_config':    self.new_prod_configuration(Constants.APT, AptitudePackageManager),
-            'dnf5_prod_config':    self.new_prod_configuration(Constants.DNF, Dnf5PackageManager),
+            'dnf5_prod_config':    self.new_prod_configuration(Constants.DNF5, Dnf5PackageManager),
             'tdnf_prod_config': self.new_prod_configuration(Constants.TDNF, AzL3TdnfPackageManager),
             'yum_prod_config':    self.new_prod_configuration(Constants.YUM, YumPackageManager),
             'zypper_prod_config': self.new_prod_configuration(Constants.ZYPPER, ZypperPackageManager),
 
             'apt_dev_config':     self.new_dev_configuration(Constants.APT, AptitudePackageManager),
-            'dnf5_dev_config':     self.new_dev_configuration(Constants.DNF, Dnf5PackageManager),
+            'dnf5_dev_config':     self.new_dev_configuration(Constants.DNF5, Dnf5PackageManager),
             'tdnf_dev_config': self.new_dev_configuration(Constants.TDNF, AzL3TdnfPackageManager),
             'yum_dev_config':     self.new_dev_configuration(Constants.YUM, YumPackageManager),
             'zypper_dev_config':  self.new_dev_configuration(Constants.ZYPPER, ZypperPackageManager),
 
             'apt_test_config':    self.new_test_configuration(Constants.APT, AptitudePackageManager),
-            'dnf5_test_config':    self.new_test_configuration(Constants.DNF, Dnf5PackageManager),
+            'dnf5_test_config':    self.new_test_configuration(Constants.DNF5, Dnf5PackageManager),
             'tdnf_test_config': self.new_test_configuration(Constants.TDNF, AzL3TdnfPackageManager),
             'yum_test_config':    self.new_test_configuration(Constants.YUM, YumPackageManager),
             'zypper_test_config': self.new_test_configuration(Constants.ZYPPER, ZypperPackageManager)
@@ -120,7 +120,7 @@ def get_configuration(self, env, package_manager_name):
             print ("Error: Environment configuration not supported - " + str(env))
             return None
 
-        if str(package_manager_name) not in [Constants.APT, Constants.DNF, Constants.TDNF, Constants.YUM, Constants.ZYPPER]:
+        if str(package_manager_name) not in [Constants.APT, Constants.DNF5, Constants.TDNF, Constants.YUM, Constants.ZYPPER]:
             print ("Error: Package manager configuration not supported - " + str(package_manager_name))
             return None
 

src/core/src/bootstrap/Constants.py

@@ -201,7 +201,7 @@ class StatusTruncationConfig(EnumBackport):
 
     # Package Managers
     APT = 'apt'
-    DNF = 'dnf5'
+    DNF5 = 'dnf5'
     TDNF = 'tdnf'
     YUM = 'yum'
     ZYPPER = 'zypper'