@@ -84,22 +84,19 @@ def test_appdb_pod_template_containers(self, ops_manager: MongoDBOpsManager):
8484
8585 assert appdb_sts .spec .template .spec .service_account_name == APPDB_SA_NAME
8686
87- found_agent_container = False
88- found_sidecar_container = False
89- for container in appdb_sts .spec .template .spec .containers :
90- if container .name == "mongodb-agent" :
91- found_agent_container = True
92- appdb_agent_container = container
93- assert appdb_agent_container .resources .limits ["cpu" ] == "750m"
94- assert appdb_agent_container .resources .limits ["memory" ] == "850M"
95- elif container .name == "appdb-sidecar" :
96- found_sidecar_container = True
97- assert container .image == "busybox"
98- assert container .command == ["sleep" ]
99- assert container .args == ["infinity" ]
100-
101- assert found_agent_container , "mongodb-agent container not found"
102- assert found_sidecar_container , "appdb-sidecar container not found"
87+ containers_by_name = {container .name : container for container in appdb_sts .spec .template .spec .containers }
88+
89+ assert "mongodb-agent" in containers_by_name , "mongodb-agent container not found"
90+ assert "appdb-sidecar" in containers_by_name , "appdb-sidecar container not found"
91+
92+ appdb_agent_container = containers_by_name ["mongodb-agent" ]
93+ assert appdb_agent_container .resources .limits ["cpu" ] == "750m"
94+ assert appdb_agent_container .resources .limits ["memory" ] == "850M"
95+
96+ appdb_sidecar_container = containers_by_name ["appdb-sidecar" ]
97+ assert appdb_sidecar_container .image == "busybox"
98+ assert appdb_sidecar_container .command == ["sleep" ]
99+ assert appdb_sidecar_container .args == ["infinity" ]
103100
104101 def test_appdb_persistence (self , ops_manager : MongoDBOpsManager , namespace : str ):
105102 # appdb pod volume claim template
0 commit comments