refactor: Remove redundant open mode argument

.ruff.toml

@@ -16,7 +16,6 @@ select = [
 ignore = [
     "E501",   # line too long
 
-    "UP015",  # redundant-open-modes
     "UP031",  # printf-string-formatting
     "UP035",  # deprecated-import
     "B006",   # mutable-argument-default

src/powerapi/cli/config_parser.py

@@ -510,7 +510,7 @@ def parse_config_dict(self, file_name: str) -> dict:
         Return a configuration dict that has all the arguments' names in the long form.
         If an argument does not exist, a UnknownArgException is raised
         """
-        with open(file_name, 'r', encoding='utf-8') as config_file:
+        with open(file_name, encoding='utf-8') as config_file:
             conf = json.load(config_file)
             return self.normalize_configuration(conf=conf)
 

src/powerapi/database/csvdb.py

@@ -90,7 +90,7 @@
         # Open all files with csv and read first line
         for filename in self.filenames:
             try:
-                self.tmp_read[filename]['file'] = open(filename, 'r', encoding='utf-8')
+                self.tmp_read[filename]['file'] = open(filename, encoding='utf-8')
                 self.tmp_read[filename]['reader'] = csv.DictReader(self.tmp_read[filename]['file'])
             except FileNotFoundError as error:
                 raise CsvBadFilePathError(error) from error

src/powerapi/database/file_db.py

@@ -71,7 +71,7 @@
         :raise: StopIteration in stream mode when no report was found.
         """
 
-        with open(self.filename, 'r', encoding='utf-8') as file_object:
+        with open(self.filename, encoding='utf-8') as file_object:
             json_str = file_object.read()
 
             if json_str is None:

tests/utils/cli/base_config_parser.py

@@ -40,7 +40,7 @@ def load_configuration_from_json_file(file_name: str) -> dict:
     """
     configuration = {}
     path = parent_module.__path__[0]
-    with open(path + '/' + file_name, 'r', encoding='utf-8') as json_file:
+    with open(path + '/' + file_name, encoding='utf-8') as json_file:
         configuration = json.load(json_file)
     return configuration
 

tests/utils/report/hwpc.py

@@ -40,7 +40,7 @@ def extract_rapl_reports_with_2_sockets(number_of_reports: int) -> list[dict]:
     :return: a list of reports. Each reports is a json dictionary
     """
     path = parent_module.__path__[0]
-    with open(f'{path}/hwpc_rapl_2_socket.json', 'r', encoding='utf-8') as json_file:
+    with open(f'{path}/hwpc_rapl_2_socket.json', encoding='utf-8') as json_file:
         reports = json.load(json_file)
         return reports['reports'][:number_of_reports]
 
@@ -60,7 +60,7 @@ def extract_all_events_reports_with_2_sockets(number_of_reports: int) -> list[di
     :return: a list of reports. Each reports is a json dictionary
     """
     path = parent_module.__path__[0]
-    with open(f'{path}/hwpc_all_events_2_socket.json', 'r', encoding='utf-8') as json_file:
+    with open(f'{path}/hwpc_all_events_2_socket.json', encoding='utf-8') as json_file:
         reports = json.load(json_file)
         return reports['reports'][:number_of_reports]
 
@@ -71,6 +71,6 @@ def gen_HWPCReports(number_of_reports: int) -> list[HWPCReport]:
     :return: a list of HWPCReport
     """
     path = parent_module.__path__[0]
-    with open(f'{path}/hwpc.json', 'r', encoding='utf-8') as json_file:
+    with open(f'{path}/hwpc.json', encoding='utf-8') as json_file:
         reports = list(map(HWPCReport.from_json, json.load(json_file)['reports']))
         return reports[:number_of_reports]