add networking api tests

Signed-off-by: Patryk Diak <pdiak@redhat.com>
openshift · Oct 31, 2023 · aaa98f3 · aaa98f3
1 parent d7d5e38
commit aaa98f3
Show file tree

Hide file tree

Showing 6 changed files with 143 additions and 1 deletion.
diff --git a/config/v1/custom.network.testsuite.yaml b/config/v1/custom.network.testsuite.yaml
@@ -0,0 +1,32 @@
+apiVersion: apiextensions.k8s.io/v1 # Hack because controller-gen complains if we don't have this
+name: "[CustomNoUpgrade] Network"
+crd: 0000_10_config-operator_01_network-CustomNoUpgrade.crd.yaml
+tests:
+  onUpdate:
+    - name: Should be able to set status conditions
+      initial: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {} # No spec is required for a Network
+      updated: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {} # No spec is required for a Network
+        status:
+          conditions:
+            - type: NetworkTypeMigrationInProgress
+              status: "False"
+              reason: "Reason"
+              message: "Message"
+              lastTransitionTime: "2023-10-25T12:00:00Z"
+      expected: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {}
+        status:
+          conditions:
+            - type: NetworkTypeMigrationInProgress
+              status: "False"
+              reason: "Reason"
+              message: "Message"
+              lastTransitionTime: "2023-10-25T12:00:00Z"
diff --git a/config/v1/stable.network.testsuite.yaml b/config/v1/stable.network.testsuite.yaml
@@ -12,3 +12,25 @@ tests:
       apiVersion: config.openshift.io/v1
       kind: Network
       spec: {}
+  onUpdate:
+    - name: Should not be able to set status conditions
+      initial: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {} # No spec is required for a Network
+      updated: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {} # No spec is required for a Network
+        status:
+          conditions:
+            - type: NetworkTypeMigrationInProgress
+              status: "False"
+              reason: "Reason"
+              message: "Message"
+              lastTransitionTime: "2023-10-25T12:00:00Z"
+      expected: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {}
+        status: {}
diff --git a/config/v1/techpreview.network.testsuite.yaml b/config/v1/techpreview.network.testsuite.yaml
@@ -0,0 +1,32 @@
+apiVersion: apiextensions.k8s.io/v1 # Hack because controller-gen complains if we don't have this
+name: "[TechPreviewNoUpgrade] Network"
+crd: 0000_10_config-operator_01_network-TechPreviewNoUpgrade.crd.yaml
+tests:
+  onUpdate:
+    - name: Should be able to set status conditions
+      initial: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {} # No spec is required for a Network
+      updated: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {} # No spec is required for a Network
+        status:
+          conditions:
+            - type: NetworkTypeMigrationInProgress
+              status: "False"
+              reason: "Reason"
+              message: "Message"
+              lastTransitionTime: "2023-10-25T12:00:00Z"
+      expected: |
+        apiVersion: config.openshift.io/v1
+        kind: Network
+        spec: {}
+        status:
+          conditions:
+            - type: NetworkTypeMigrationInProgress
+              status: "False"
+              reason: "Reason"
+              message: "Message"
+              lastTransitionTime: "2023-10-25T12:00:00Z"
diff --git a/operator/v1/custom.network.testsuite.yaml b/operator/v1/custom.network.testsuite.yaml
@@ -0,0 +1,21 @@
+apiVersion: apiextensions.k8s.io/v1 # Hack because controller-gen complains if we don't have this
+name: "[CustomNoUpgrade] Network"
+crd: 0000_70_cluster-network-operator_01-CustomNoUpgrade.crd.yaml
+tests:
+  onCreate:
+    - name: Should be able to create migration mode
+      initial: |
+        apiVersion: operator.openshift.io/v1
+        kind: Network
+        spec: 
+          migration:
+            mode: Live
+      expected: |
+        apiVersion: operator.openshift.io/v1
+        kind: Network
+        spec:
+          disableNetworkDiagnostics: false
+          logLevel: Normal
+          operatorLogLevel: Normal
+          migration:
+            mode: Live
diff --git a/operator/v1/stable.network.testsuite.yaml b/operator/v1/stable.network.testsuite.yaml
@@ -227,4 +227,18 @@ tests:
               ipv6:
                 internalMasqueradeSubnet: "abcd:ef01:2345:6789:abcd:ef01:2345::/125"
     expectedError: "Invalid value: \"string\": a valid IPv6 address must contain 8 segments unless elided (::), in which case it must contain at most 6 non-empty segments"
-
+  - name: Should not be able to create migration mode
+    initial: |
+      apiVersion: operator.openshift.io/v1
+      kind: Network
+      spec: 
+        migration:
+          mode: Live
+    expected: |
+      apiVersion: operator.openshift.io/v1
+      kind: Network
+      spec:
+        disableNetworkDiagnostics: false
+        logLevel: Normal
+        operatorLogLevel: Normal
+        migration: {}
diff --git a/operator/v1/techpreview.network.testsuite.yaml b/operator/v1/techpreview.network.testsuite.yaml
@@ -0,0 +1,21 @@
+apiVersion: apiextensions.k8s.io/v1 # Hack because controller-gen complains if we don't have this
+name: "[TechPreviewNoUpgrade] Network"
+crd: 0000_70_cluster-network-operator_01-TechPreviewNoUpgrade.crd.yaml
+tests:
+  onCreate:
+  - name: Should be able to create migration mode
+    initial: |
+      apiVersion: operator.openshift.io/v1
+      kind: Network
+      spec: 
+        migration:
+          mode: Live
+    expected: |
+      apiVersion: operator.openshift.io/v1
+      kind: Network
+      spec:
+        disableNetworkDiagnostics: false
+        logLevel: Normal
+        operatorLogLevel: Normal
+        migration:
+          mode: Live