Fix unit tests + linter issues

Author: MaciejKaras

docker/mongodb-kubernetes-tests/kubetester/__init__.py

@@ -273,14 +273,12 @@ def delete_namespace(name: str):
     c = client.CoreV1Api()
     c.delete_namespace(name, body=c.V1DeleteOptions())
 
+
 def label_namespace(name: str, labels: dict):
-    body = {
-        "metadata": {
-            "labels": labels
-        }
-    }
+    body = {"metadata": {"labels": labels}}
     client.CoreV1Api().patch_namespace(name, body)
 
+
 def get_deployments(namespace: str):
     return client.AppsV1Api().list_namespaced_deployment(namespace)
 

docker/mongodb-kubernetes-tests/tests/opsmanager/om_ops_manager_backup_sharded_cluster.py

@@ -5,8 +5,9 @@
 from kubetester import (
     create_or_update_secret,
     get_default_storage_class,
+    label_namespace,
     try_load,
-    wait_until, create_or_update_namespace, label_namespace,
+    wait_until,
 )
 from kubetester.awss3client import AwsS3Client
 from kubetester.kubetester import KubernetesTester, ensure_ent_version
@@ -54,10 +55,9 @@ def ops_manager(
     )
 
     # Change pod-security mode from warn to enforce. This will make test fail if operator and deployments don't support enforce mode
-    label_namespace(namespace, {
-        "pod-security.kubernetes.io/enforce": None,
-        "pod-security.kubernetes.io/warn": "restricted"
-    })
+    label_namespace(
+        namespace, {"pod-security.kubernetes.io/enforce": None, "pod-security.kubernetes.io/warn": "restricted"}
+    )
 
     try_load(resource)
     return resource
@@ -241,7 +241,7 @@ def test_om_failed_oplog_no_user_ref(self, ops_manager: MongoDBOpsManager):
         ops_manager.backup_status().assert_reaches_phase(
             Phase.Failed,
             msg_regexp=".*is configured to use SCRAM-SHA authentication mode, the user "
-                       "must be specified using 'mongodbUserRef'",
+            "must be specified using 'mongodbUserRef'",
         )
 
     def test_fix_om(self, ops_manager: MongoDBOpsManager, oplog_user: MongoDBUser):

mongodb-community-operator/pkg/kube/podtemplatespec/podspec_template_test.go

@@ -4,7 +4,6 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/assert"
-	"k8s.io/utils/ptr"
 
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -442,12 +441,6 @@ func TestMergeContainer(t *testing.T) {
 			},
 		},
 		ReadinessProbe: otherDefaultContainer.ReadinessProbe,
-		SecurityContext: &corev1.SecurityContext{
-			AllowPrivilegeEscalation: ptr.To(false),
-			Capabilities: &corev1.Capabilities{
-				Drop: []corev1.Capability{"ALL"},
-			},
-		},
 	}
 	assert.Equal(t, secondExpected, mergedSpec.Spec.Containers[2])
 }