Merge pull request #247 from mgoerens/apply_black

Apply black and fix some PEP8 violations
openshift-helm-charts · Aug 8, 2023 · ba8a8f2 · ba8a8f2
2 parents 671f21d + 8b71e3b
commit ba8a8f2
Show file tree

Hide file tree

Showing 62 changed files with 4,841 additions and 2,237 deletions.
diff --git a/scripts/src/chartprreview/chartprreview.py b/scripts/src/chartprreview/chartprreview.py
diff --git a/scripts/src/chartprreview/chartprreview_test.py b/scripts/src/chartprreview/chartprreview_test.py
@@ -4,10 +4,12 @@
 from chartprreview.chartprreview import check_owners_file_against_directory_structure
 from chartprreview.chartprreview import write_error_log
 
+
 def test_verify_user():
     with pytest.raises(SystemExit):
         verify_user("mbaiju", "partners", "test-org1", "test-chart")
 
+
 owners_with_wrong_vendor_label = """\
 ---
 chart:
@@ -51,21 +53,33 @@ def test_verify_user():
 """
 
 
-
 def test_check_owners_file_against_directory_structure(tmpdir):
     original_cwd = os.getcwd()
-    p = tmpdir.mkdir("charts").mkdir("partners").mkdir("test-org").mkdir("test-chart").join("OWNERS")
+    p = (
+        tmpdir.mkdir("charts")
+        .mkdir("partners")
+        .mkdir("test-org")
+        .mkdir("test-chart")
+        .join("OWNERS")
+    )
     p.write(owners_with_wrong_vendor_label)
     os.chdir(tmpdir)
     new_cwd = os.getcwd()
     print("new_cwd", new_cwd)
     with pytest.raises(SystemExit):
-        check_owners_file_against_directory_structure("baijum", "partners", "test-org", "test-chart")
+        check_owners_file_against_directory_structure(
+            "baijum", "partners", "test-org", "test-chart"
+        )
     p.write(owners_with_wrong_chart_name)
     with pytest.raises(SystemExit):
-        check_owners_file_against_directory_structure("baijum", "partners", "test-org", "test-chart")
+        check_owners_file_against_directory_structure(
+            "baijum", "partners", "test-org", "test-chart"
+        )
     p.write(owners_with_correct_values)
-    check_owners_file_against_directory_structure("baijum", "partners", "test-org", "test-chart")
+    check_owners_file_against_directory_structure(
+        "baijum", "partners", "test-org", "test-chart"
+    )
+
 
 def test_write_error_log(tmpdir):
     write_error_log(tmpdir, "First message")