Refactoring changes to create a test runner option from passed flags, correct indentation and add package level comment for test (#1165)

* corrected indentation and added package comment for /test
* fix indentation for required_labels test file
* added test runner options from flags as a public method

Author: aakash070

policy/testdata/context_pb/tests.yaml

@@ -16,13 +16,13 @@ description: "Protobuf input tests"
 section:
   - name: "valid"
     tests:
-    - name: "good spec"
-      context_expr: "test.TestAllTypes{single_int32: 10}"
-      output:
-        expr: "optional.none()"
+      - name: "good spec"
+        context_expr: "test.TestAllTypes{single_int32: 10}"
+        output:
+          expr: "optional.none()"
   - name: "invalid"
     tests:
-    - name: "bad spec"
-      context_expr: "test.TestAllTypes{single_int32: 11}"
-      output:
-        value: "invalid spec, got single_int32=11, wanted <= 10"
+      - name: "bad spec"
+        context_expr: "test.TestAllTypes{single_int32: 11}"
+        output:
+          value: "invalid spec, got single_int32=11, wanted <= 10"

policy/testdata/nested_rule7/tests.yaml

@@ -16,27 +16,27 @@ description: "Nested rule tests which explore optional vs non-optional returns"
 section:
   - name: "valid"
     tests:
-     - name: "x=1"
-       input:
+      - name: "x=1"
+        input:
           x:
             value: 1
-       output:
-         expr: "optional.none()"
-     - name: "x=2"
-       input:
+        output:
+          expr: "optional.none()"
+      - name: "x=2"
+        input:
           x:
             value: 2
-       output:
-         value: false
-     - name: "x=3"
-       input:
+        output:
+          value: false
+      - name: "x=3"
+        input:
           x:
             value: 3
-       output:
-         value: true
-     - name: "x=4"
-       input:
+        output:
+          value: true
+      - name: "x=4"
+        input:
           x:
             value: 4
-       output:
-         value: true
+        output:
+          value: true

policy/testdata/pb/tests.yaml

@@ -16,19 +16,19 @@ description: "Protobuf input tests"
 section:
   - name: "valid"
     tests:
-     - name: "good spec"
-       input:
-         spec:
-           expr: >
-             test.TestAllTypes{single_int32: 10}
-       output:
-         expr: "optional.none()"
+      - name: "good spec"
+        input:
+          spec:
+            expr: >
+              test.TestAllTypes{single_int32: 10}
+        output:
+          expr: "optional.none()"
   - name: "invalid"
     tests:
-     - name: "bad spec"
-       input:
-         spec:
-           expr: >
-             test.TestAllTypes{single_int32: 11}
-       output:
-         value: "invalid spec, got single_int32=11, wanted <= 10"
+      - name: "bad spec"
+        input:
+          spec:
+            expr: >
+              test.TestAllTypes{single_int32: 11}
+        output:
+          value: "invalid spec, got single_int32=11, wanted <= 10"

policy/testdata/required_labels/tests.yaml

@@ -16,65 +16,65 @@ description: "Required labels conformance tests"
 section:
   - name: "valid"
     tests:
-     - name: "matching"
-       input:
-         spec:
-           value:
-             labels:
-               env: prod
-               experiment: "group b"
-         resource:
-           value:
-             labels:
-               env: prod
-               experiment: "group b"
-               release: "v0.1.0"
-       output:
-         expr: "optional.none()"
+      - name: "matching"
+        input:
+          spec:
+            value:
+              labels:
+                env: prod
+                experiment: "group b"
+          resource:
+            value:
+              labels:
+                env: prod
+                experiment: "group b"
+                release: "v0.1.0"
+        output:
+          expr: "optional.none()"
   - name: "missing"
     tests:
-     - name: "env"
-       input:
-         spec:
-           value:
-             labels:
-               env: prod
-               experiment: "group b"
-         resource:
-           value:
-             labels:
-               experiment: "group b"
-               release: "v0.1.0"
-       output:
-         value: "missing one or more required labels: [\"env\"]"
-     - name: "experiment"
-       input:
-         spec:
-           value:
-             labels:
-               env: prod
-               experiment: "group b"
-         resource:
-           value:
-             labels:
-               env: staging
-               release: "v0.1.0"
-       output:
-         value: "missing one or more required labels: [\"experiment\"]"
+      - name: "env"
+        input:
+          spec:
+            value:
+              labels:
+                env: prod
+                experiment: "group b"
+          resource:
+            value:
+              labels:
+                experiment: "group b"
+                release: "v0.1.0"
+        output:
+          value: "missing one or more required labels: [\"env\"]"
+      - name: "experiment"
+        input:
+          spec:
+            value:
+              labels:
+                env: prod
+                experiment: "group b"
+          resource:
+            value:
+              labels:
+                env: staging
+                release: "v0.1.0"
+        output:
+          value: "missing one or more required labels: [\"experiment\"]"
   - name: "invalid"
     tests:
-     - name: "env"
-       input:
-         spec:
-           value:
-             labels:
-               env: prod
-               experiment: "group b"
-         resource:
-           value:
-             labels:
-               env: staging
-               experiment: "group b"
-               release: "v0.1.0"
-       output:
-         value: "invalid values provided on one or more labels: [\"env\"]"
+      - name: "env"
+        input:
+          spec:
+            value:
+              labels:
+                env: prod
+                experiment: "group b"
+          resource:
+            value:
+              labels:
+                env: staging
+                experiment: "group b"
+                release: "v0.1.0"
+        output:
+          value: "invalid values provided on one or more labels: [\"env\"]"

policy/testdata/restricted_destinations/base_config.yaml

@@ -30,8 +30,8 @@ variables:
   - name: "request"
     type_name: "map"
     params:
-    - type_name: "string"
-    - type_name: "dyn"
+      - type_name: "string"
+      - type_name: "dyn"
   - name: "resource"
     type_name: "map"
     params: