Skip to content

Commit 50df705

Browse files
paigerube14paigerube14
committed
krkn_config test removal
1 parent 6283c98 commit 50df705

File tree

3 files changed

+8
-37
lines changed

3 files changed

+8
-37
lines changed

‎tests/test_pvc_scenario_plugin.py‎

Lines changed: 8 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,6 @@ def test_run_missing_namespace(self):
255255
result = self.plugin.run(
256256
run_uuid="test-uuid",
257257
scenario=scenario_path,
258-
krkn_config={},
259258
lib_telemetry=mock_telemetry,
260259
scenario_telemetry=mock_scenario_telemetry,
261260
)
@@ -279,7 +278,6 @@ def test_run_missing_pvc_and_pod_name(self):
279278
result = self.plugin.run(
280279
run_uuid="test-uuid",
281280
scenario=scenario_path,
282-
krkn_config={},
283281
lib_telemetry=mock_telemetry,
284282
scenario_telemetry=mock_scenario_telemetry,
285283
)
@@ -306,7 +304,6 @@ def test_run_pod_not_found(self):
306304
result = self.plugin.run(
307305
run_uuid="test-uuid",
308306
scenario=scenario_path,
309-
krkn_config={},
310307
lib_telemetry=mock_telemetry,
311308
scenario_telemetry=mock_scenario_telemetry,
312309
)
@@ -340,7 +337,6 @@ def test_run_pvc_not_found_for_pod(self):
340337
result = self.plugin.run(
341338
run_uuid="test-uuid",
342339
scenario=scenario_path,
343-
krkn_config={},
344340
lib_telemetry=mock_telemetry,
345341
scenario_telemetry=mock_scenario_telemetry,
346342
)
@@ -395,7 +391,6 @@ def test_run_invalid_fill_percentage(self):
395391
result = self.plugin.run(
396392
run_uuid="test-uuid",
397393
scenario=scenario_path,
398-
krkn_config={},
399394
lib_telemetry=mock_telemetry,
400395
scenario_telemetry=mock_scenario_telemetry,
401396
)
@@ -404,8 +399,7 @@ def test_run_invalid_fill_percentage(self):
404399
self.assertEqual(result, 1)
405400

406401
@patch("krkn.scenario_plugins.pvc.pvc_scenario_plugin.time.sleep")
407-
@patch("krkn.scenario_plugins.pvc.pvc_scenario_plugin.cerberus.publish_kraken_status")
408-
def test_run_success_with_fallocate(self, mock_publish, mock_sleep):
402+
def test_run_success_with_fallocate(self, mock_sleep):
409403
"""Test successful run using fallocate"""
410404
with tempfile.TemporaryDirectory() as temp_dir:
411405
scenario_config = {
@@ -460,7 +454,6 @@ def test_run_success_with_fallocate(self, mock_publish, mock_sleep):
460454
result = self.plugin.run(
461455
run_uuid="test-uuid",
462456
scenario=scenario_path,
463-
krkn_config={},
464457
lib_telemetry=mock_telemetry,
465458
scenario_telemetry=mock_scenario_telemetry,
466459
)
@@ -469,8 +462,7 @@ def test_run_success_with_fallocate(self, mock_publish, mock_sleep):
469462
mock_sleep.assert_called_once_with(1)
470463

471464
@patch("krkn.scenario_plugins.pvc.pvc_scenario_plugin.time.sleep")
472-
@patch("krkn.scenario_plugins.pvc.pvc_scenario_plugin.cerberus.publish_kraken_status")
473-
def test_run_success_with_dd(self, mock_publish, mock_sleep):
465+
def test_run_success_with_dd(self, mock_sleep):
474466
"""Test successful run using dd when fallocate is not available"""
475467
with tempfile.TemporaryDirectory() as temp_dir:
476468
scenario_config = {
@@ -525,7 +517,6 @@ def test_run_success_with_dd(self, mock_publish, mock_sleep):
525517
result = self.plugin.run(
526518
run_uuid="test-uuid",
527519
scenario=scenario_path,
528-
krkn_config={},
529520
lib_telemetry=mock_telemetry,
530521
scenario_telemetry=mock_scenario_telemetry,
531522
)
@@ -582,7 +573,6 @@ def test_run_no_binary_available(self):
582573
result = self.plugin.run(
583574
run_uuid="test-uuid",
584575
scenario=scenario_path,
585-
krkn_config={},
586576
lib_telemetry=mock_telemetry,
587577
scenario_telemetry=mock_scenario_telemetry,
588578
)
@@ -597,7 +587,6 @@ def test_run_file_not_found(self):
597587
result = self.plugin.run(
598588
run_uuid="test-uuid",
599589
scenario="/non/existent/path.yaml",
600-
krkn_config={},
601590
lib_telemetry=mock_telemetry,
602591
scenario_telemetry=mock_scenario_telemetry,
603592
)
@@ -659,14 +648,12 @@ def test_run_both_pvc_and_pod_name_provided(self):
659648
mock_scenario_telemetry = MagicMock()
660649

661650
with patch("krkn.scenario_plugins.pvc.pvc_scenario_plugin.time.sleep"):
662-
with patch("krkn.scenario_plugins.pvc.pvc_scenario_plugin.cerberus.publish_kraken_status"):
663-
result = self.plugin.run(
664-
run_uuid="test-uuid",
665-
scenario=scenario_path,
666-
krkn_config={},
667-
lib_telemetry=mock_telemetry,
668-
scenario_telemetry=mock_scenario_telemetry,
669-
)
651+
result = self.plugin.run(
652+
run_uuid="test-uuid",
653+
scenario=scenario_path,
654+
lib_telemetry=mock_telemetry,
655+
scenario_telemetry=mock_scenario_telemetry,
656+
)
670657

671658
self.assertEqual(result, 0)
672659
# get_pod_info should be called with "actual-pod-from-pvc", not "ignored-pod"
@@ -707,7 +694,6 @@ def test_run_pvc_name_only_no_pods_associated(self):
707694
result = self.plugin.run(
708695
run_uuid="test-uuid",
709696
scenario=scenario_path,
710-
krkn_config={},
711697
lib_telemetry=mock_telemetry,
712698
scenario_telemetry=mock_scenario_telemetry,
713699
)
@@ -770,7 +756,6 @@ def test_run_file_creation_failed(self):
770756
result = self.plugin.run(
771757
run_uuid="test-uuid",
772758
scenario=scenario_path,
773-
krkn_config={},
774759
lib_telemetry=mock_telemetry,
775760
scenario_telemetry=mock_scenario_telemetry,
776761
)

‎tests/test_service_hijacking_scenario_plugin.py‎

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,6 @@ def test_run_successful(self):
185185
result = plugin.run(
186186
run_uuid=str(uuid.uuid4()),
187187
scenario=scenario_file,
188-
krkn_config={},
189188
lib_telemetry=mock_lib_telemetry,
190189
scenario_telemetry=mock_scenario_telemetry,
191190
)
@@ -215,7 +214,6 @@ def test_run_service_not_found(self):
215214
result = plugin.run(
216215
run_uuid=str(uuid.uuid4()),
217216
scenario=scenario_file,
218-
krkn_config={},
219217
lib_telemetry=mock_lib_telemetry,
220218
scenario_telemetry=mock_scenario_telemetry,
221219
)
@@ -246,7 +244,6 @@ def test_run_patch_service_failed(self):
246244
result = plugin.run(
247245
run_uuid=str(uuid.uuid4()),
248246
scenario=scenario_file,
249-
krkn_config={},
250247
lib_telemetry=mock_lib_telemetry,
251248
scenario_telemetry=mock_scenario_telemetry,
252249
)
@@ -277,7 +274,6 @@ def test_run_restore_service_failed(self):
277274
result = plugin.run(
278275
run_uuid=str(uuid.uuid4()),
279276
scenario=scenario_file,
280-
krkn_config={},
281277
lib_telemetry=mock_lib_telemetry,
282278
scenario_telemetry=mock_scenario_telemetry,
283279
)
@@ -309,7 +305,6 @@ def test_run_with_numeric_port(self):
309305
result = plugin.run(
310306
run_uuid=str(uuid.uuid4()),
311307
scenario=scenario_file,
312-
krkn_config={},
313308
lib_telemetry=mock_lib_telemetry,
314309
scenario_telemetry=mock_scenario_telemetry,
315310
)
@@ -343,7 +338,6 @@ def test_run_with_named_port(self):
343338
result = plugin.run(
344339
run_uuid=str(uuid.uuid4()),
345340
scenario=scenario_file,
346-
krkn_config={},
347341
lib_telemetry=mock_lib_telemetry,
348342
scenario_telemetry=mock_scenario_telemetry,
349343
)
@@ -370,7 +364,6 @@ def test_run_exception_handling(self):
370364
result = plugin.run(
371365
run_uuid=str(uuid.uuid4()),
372366
scenario=scenario_file,
373-
krkn_config={},
374367
lib_telemetry=mock_lib_telemetry,
375368
scenario_telemetry=mock_scenario_telemetry,
376369
)
@@ -399,7 +392,6 @@ def test_run_unprivileged_mode(self):
399392
result = plugin.run(
400393
run_uuid=str(uuid.uuid4()),
401394
scenario=scenario_file,
402-
krkn_config={},
403395
lib_telemetry=mock_lib_telemetry,
404396
scenario_telemetry=mock_scenario_telemetry,
405397
)

‎tests/test_syn_flood_scenario_plugin.py‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,6 @@ def test_run_successful_with_target_service(self):
294294
result = plugin.run(
295295
run_uuid=str(uuid.uuid4()),
296296
scenario=scenario_file,
297-
krkn_config={},
298297
lib_telemetry=mock_lib_telemetry,
299298
scenario_telemetry=mock_scenario_telemetry,
300299
)
@@ -330,7 +329,6 @@ def test_run_successful_with_label_selector(self):
330329
result = plugin.run(
331330
run_uuid=str(uuid.uuid4()),
332331
scenario=scenario_file,
333-
krkn_config={},
334332
lib_telemetry=mock_lib_telemetry,
335333
scenario_telemetry=mock_scenario_telemetry,
336334
)
@@ -359,7 +357,6 @@ def test_run_service_not_found(self):
359357
result = plugin.run(
360358
run_uuid=str(uuid.uuid4()),
361359
scenario=scenario_file,
362-
krkn_config={},
363360
lib_telemetry=mock_lib_telemetry,
364361
scenario_telemetry=mock_scenario_telemetry,
365362
)
@@ -385,7 +382,6 @@ def test_run_multiple_pods(self):
385382
result = plugin.run(
386383
run_uuid=str(uuid.uuid4()),
387384
scenario=scenario_file,
388-
krkn_config={},
389385
lib_telemetry=mock_lib_telemetry,
390386
scenario_telemetry=mock_scenario_telemetry,
391387
)
@@ -411,7 +407,6 @@ def test_run_exception_handling(self):
411407
result = plugin.run(
412408
run_uuid=str(uuid.uuid4()),
413409
scenario=scenario_file,
414-
krkn_config={},
415410
lib_telemetry=mock_lib_telemetry,
416411
scenario_telemetry=mock_scenario_telemetry,
417412
)
@@ -437,7 +432,6 @@ def test_run_waits_for_pods_to_finish(self):
437432
result = plugin.run(
438433
run_uuid=str(uuid.uuid4()),
439434
scenario=scenario_file,
440-
krkn_config={},
441435
lib_telemetry=mock_lib_telemetry,
442436
scenario_telemetry=mock_scenario_telemetry,
443437
)

0 commit comments

Comments
 (0)