Merge pull request buildtesters#1545 from buildtesters/pyproject.toml

Add pyproject.toml file

.pre-commit-config.yaml

@@ -11,7 +11,7 @@ repos:
         args: ["--profile", "black", "--filter-files", "buildtest", "tests", "docs"]
 
   - repo: https://github.com/psf/black
-    rev: 23.3.0
+    rev: 'refs/tags/22.3.0'
     hooks:
       - id: black
         language_version: python3

buildtest/builders/base.py

@@ -646,8 +646,7 @@ def _write_test(self):
         self._set_execute_perm(self.testpath)
         # copy testpath to run_dir
         shutil.copy2(
-            self.testpath,
-            os.path.join(self.test_root, os.path.basename(self.testpath)),
+            self.testpath, os.path.join(self.test_root, os.path.basename(self.testpath))
         )
 
     def _emit_command(self):

buildtest/buildsystem/checks.py

@@ -49,10 +49,7 @@ def returncode_check(builder):
     logger.debug("Conducting Return Code check")
     logger.debug(
         "Status Return Code: %s   Result Return Code: %s"
-        % (
-            buildspec_returncode,
-            builder.metadata["result"]["returncode"],
-        )
+        % (buildspec_returncode, builder.metadata["result"]["returncode"])
     )
     # checks if test returncode matches returncode specified in Buildspec and assign boolean to returncode_match
     returncode_match = builder.metadata["result"]["returncode"] in buildspec_returncode
@@ -389,12 +386,10 @@ def assert_ge_check(builder):
 
         # convert metric value and reference value to int
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
         ref_value = convert_metrics(
-            metric_value=ref_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=ref_value, dtype=builder.metrics[name]["type"]
         )
 
         # if there is a type mismatch then let's stop now before we do comparison
@@ -460,12 +455,10 @@ def assert_le_check(builder):
 
         # convert metric value and reference value to int
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
         ref_value = convert_metrics(
-            metric_value=ref_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=ref_value, dtype=builder.metrics[name]["type"]
         )
 
         # if there is a type mismatch then let's stop now before we do comparison
@@ -531,12 +524,10 @@ def assert_gt_check(builder):
 
         # convert metric value and reference value to int
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
         ref_value = convert_metrics(
-            metric_value=ref_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=ref_value, dtype=builder.metrics[name]["type"]
         )
 
         # if there is a type mismatch then let's stop now before we do comparison
@@ -602,12 +593,10 @@ def assert_lt_check(builder):
 
         # convert metric value and reference value to int
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
         ref_value = convert_metrics(
-            metric_value=ref_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=ref_value, dtype=builder.metrics[name]["type"]
         )
 
         # if there is a type mismatch then let's stop now before we do comparison
@@ -665,12 +654,10 @@ def assert_eq_check(builder):
             continue
 
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
         ref_value = convert_metrics(
-            metric_value=ref_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=ref_value, dtype=builder.metrics[name]["type"]
         )
 
         # if either converted value and reference value is None stop here before proceeding to equality check
@@ -728,12 +715,10 @@ def assert_ne_check(builder):
             continue
 
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
         ref_value = convert_metrics(
-            metric_value=ref_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=ref_value, dtype=builder.metrics[name]["type"]
         )
 
         # if either converted value and reference value is None stop here before proceeding to the not equal check
@@ -791,8 +776,7 @@ def contains_check(builder):
             continue
 
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
 
         # if either converted value and reference value is None stop here before proceeding to the not equal check
@@ -854,8 +838,7 @@ def notcontains_check(builder):
             continue
 
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
 
         # if either converted value and reference value is None stop here before proceeding to the not equal check
@@ -926,20 +909,16 @@ def assert_range_check(builder):
             continue
 
         conv_value = convert_metrics(
-            metric_value=metric_value,
-            dtype=builder.metrics[name]["type"],
+            metric_value=metric_value, dtype=builder.metrics[name]["type"]
         )
         lower_bound = convert_metrics(
-            metric_value=lower_bound,
-            dtype=builder.metrics[name]["type"],
+            metric_value=lower_bound, dtype=builder.metrics[name]["type"]
         )
         lower_bound = convert_metrics(
-            metric_value=lower_bound,
-            dtype=builder.metrics[name]["type"],
+            metric_value=lower_bound, dtype=builder.metrics[name]["type"]
         )
         upper_bound = convert_metrics(
-            metric_value=upper_bound,
-            dtype=builder.metrics[name]["type"],
+            metric_value=upper_bound, dtype=builder.metrics[name]["type"]
         )
 
         # if any item is None we stop before we run comparison

buildtest/buildsystem/parser.py

@@ -115,9 +115,7 @@ def _check_schema_type(self, test):
             msg = f"Schema type must be one of the following: {schema_table['types']}. "
             raise InvalidBuildspecSchemaType(self.buildspec, msg)
 
-        self.logger.info(
-            f"Test: '{test}' is using schema type: '{self.schema_type}'",
-        )
+        self.logger.info(f"Test: '{test}' is using schema type: '{self.schema_type}'")
 
     def _check_executor(self, test):
         """This method checks if ``executor`` property is not None and executor