Skip to content

Commit e7667d6

Browse files
committed
test: use parent to access other resources clients
1 parent 22dd8a9 commit e7667d6

File tree

10 files changed

+57
-57
lines changed

10 files changed

+57
-57
lines changed

tests/unit/actions/test_client.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ def test_get_by_id(
8989
request_mock.return_value = generic_action
9090
action = actions_client.get_by_id(1)
9191
request_mock.assert_called_with(url="/resource/actions/1", method="GET")
92-
assert action._client == actions_client._client.actions
92+
assert action._client == actions_client._parent.actions
9393
assert action.id == 1
9494
assert action.command == "stop_server"
9595

@@ -118,11 +118,11 @@ def test_get_list(
118118
action1 = actions[0]
119119
action2 = actions[1]
120120

121-
assert action1._client == actions_client._client.actions
121+
assert action1._client == actions_client._parent.actions
122122
assert action1.id == 1
123123
assert action1.command == "start_server"
124124

125-
assert action2._client == actions_client._client.actions
125+
assert action2._client == actions_client._parent.actions
126126
assert action2.id == 2
127127
assert action2.command == "stop_server"
128128

@@ -148,11 +148,11 @@ def test_get_all(
148148
action1 = actions[0]
149149
action2 = actions[1]
150150

151-
assert action1._client == actions_client._client.actions
151+
assert action1._client == actions_client._parent.actions
152152
assert action1.id == 1
153153
assert action1.command == "start_server"
154154

155-
assert action2._client == actions_client._client.actions
155+
assert action2._client == actions_client._parent.actions
156156
assert action2.id == 2
157157
assert action2.command == "stop_server"
158158

@@ -171,7 +171,7 @@ def test_get_by_id(
171171
request_mock.return_value = generic_action
172172
action = actions_client.get_by_id(1)
173173
request_mock.assert_called_with(url="/actions/1", method="GET")
174-
assert action._client == actions_client._client.actions
174+
assert action._client == actions_client._parent.actions
175175
assert action.id == 1
176176
assert action.command == "stop_server"
177177

@@ -199,11 +199,11 @@ def test_get_list(
199199
action1 = actions[0]
200200
action2 = actions[1]
201201

202-
assert action1._client == actions_client._client.actions
202+
assert action1._client == actions_client._parent.actions
203203
assert action1.id == 1
204204
assert action1.command == "start_server"
205205

206-
assert action2._client == actions_client._client.actions
206+
assert action2._client == actions_client._parent.actions
207207
assert action2.id == 2
208208
assert action2.command == "stop_server"
209209

@@ -228,10 +228,10 @@ def test_get_all(
228228
action1 = actions[0]
229229
action2 = actions[1]
230230

231-
assert action1._client == actions_client._client.actions
231+
assert action1._client == actions_client._parent.actions
232232
assert action1.id == 1
233233
assert action1.command == "start_server"
234234

235-
assert action2._client == actions_client._client.actions
235+
assert action2._client == actions_client._parent.actions
236236
assert action2.id == 2
237237
assert action2.command == "stop_server"

tests/unit/certificates/test_client.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ def test_actions_get_by_id(
358358
request_mock.assert_called_with(url="/certificates/actions/13", method="GET")
359359

360360
assert isinstance(action, BoundAction)
361-
assert action._client == certificates_client._client.actions
361+
assert action._client == certificates_client._parent.actions
362362
assert action.id == 13
363363
assert action.command == "change_protection"
364364

@@ -382,7 +382,7 @@ def test_actions_get_list(
382382

383383
assert len(actions) == 1
384384
assert isinstance(actions[0], BoundAction)
385-
assert actions[0]._client == certificates_client._client.actions
385+
assert actions[0]._client == certificates_client._parent.actions
386386
assert actions[0].id == 13
387387
assert actions[0].command == "change_protection"
388388

@@ -403,6 +403,6 @@ def test_actions_get_all(
403403

404404
assert len(actions) == 1
405405
assert isinstance(actions[0], BoundAction)
406-
assert actions[0]._client == certificates_client._client.actions
406+
assert actions[0]._client == certificates_client._parent.actions
407407
assert actions[0].id == 13
408408
assert actions[0].command == "change_protection"

tests/unit/firewalls/test_client.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -361,7 +361,7 @@ def test_get_actions_list(
361361
assert len(actions) == 1
362362
assert isinstance(actions[0], BoundAction)
363363

364-
assert actions[0]._client == firewalls_client._client.actions
364+
assert actions[0]._client == firewalls_client._parent.actions
365365
assert actions[0].id == 13
366366
assert actions[0].command == "set_firewall_rules"
367367

@@ -556,7 +556,7 @@ def test_actions_get_by_id(
556556
request_mock.assert_called_with(url="/firewalls/actions/13", method="GET")
557557

558558
assert isinstance(action, BoundAction)
559-
assert action._client == firewalls_client._client.actions
559+
assert action._client == firewalls_client._parent.actions
560560
assert action.id == 13
561561
assert action.command == "set_firewall_rules"
562562

@@ -580,7 +580,7 @@ def test_actions_get_list(
580580

581581
assert len(actions) == 1
582582
assert isinstance(actions[0], BoundAction)
583-
assert actions[0]._client == firewalls_client._client.actions
583+
assert actions[0]._client == firewalls_client._parent.actions
584584
assert actions[0].id == 13
585585
assert actions[0].command == "set_firewall_rules"
586586

@@ -601,6 +601,6 @@ def test_actions_get_all(
601601

602602
assert len(actions) == 1
603603
assert isinstance(actions[0], BoundAction)
604-
assert actions[0]._client == firewalls_client._client.actions
604+
assert actions[0]._client == firewalls_client._parent.actions
605605
assert actions[0].id == 13
606606
assert actions[0].command == "set_firewall_rules"

tests/unit/floating_ips/test_client.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ def test_get_actions(
366366
assert len(actions) == 1
367367
assert isinstance(actions[0], BoundAction)
368368

369-
assert actions[0]._client == floating_ips_client._client.actions
369+
assert actions[0]._client == floating_ips_client._parent.actions
370370
assert actions[0].id == 13
371371
assert actions[0].command == "assign_floating_ip"
372372

@@ -511,7 +511,7 @@ def test_actions_get_by_id(
511511
request_mock.assert_called_with(url="/floating_ips/actions/13", method="GET")
512512

513513
assert isinstance(action, BoundAction)
514-
assert action._client == floating_ips_client._client.actions
514+
assert action._client == floating_ips_client._parent.actions
515515
assert action.id == 13
516516
assert action.command == "assign_floating_ip"
517517

@@ -535,7 +535,7 @@ def test_actions_get_list(
535535

536536
assert len(actions) == 1
537537
assert isinstance(actions[0], BoundAction)
538-
assert actions[0]._client == floating_ips_client._client.actions
538+
assert actions[0]._client == floating_ips_client._parent.actions
539539
assert actions[0].id == 13
540540
assert actions[0].command == "assign_floating_ip"
541541

@@ -556,6 +556,6 @@ def test_actions_get_all(
556556

557557
assert len(actions) == 1
558558
assert isinstance(actions[0], BoundAction)
559-
assert actions[0]._client == floating_ips_client._client.actions
559+
assert actions[0]._client == floating_ips_client._parent.actions
560560
assert actions[0].id == 13
561561
assert actions[0].command == "assign_floating_ip"

tests/unit/images/test_client.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ def test_get_actions_list(
311311
assert len(actions) == 1
312312
assert isinstance(actions[0], BoundAction)
313313

314-
assert actions[0]._client == images_client._client.actions
314+
assert actions[0]._client == images_client._parent.actions
315315
assert actions[0].id == 13
316316
assert actions[0].command == "change_protection"
317317

@@ -391,7 +391,7 @@ def test_actions_get_by_id(
391391
request_mock.assert_called_with(url="/images/actions/13", method="GET")
392392

393393
assert isinstance(action, BoundAction)
394-
assert action._client == images_client._client.actions
394+
assert action._client == images_client._parent.actions
395395
assert action.id == 13
396396
assert action.command == "change_protection"
397397

@@ -415,7 +415,7 @@ def test_actions_get_list(
415415

416416
assert len(actions) == 1
417417
assert isinstance(actions[0], BoundAction)
418-
assert actions[0]._client == images_client._client.actions
418+
assert actions[0]._client == images_client._parent.actions
419419
assert actions[0].id == 13
420420
assert actions[0].command == "change_protection"
421421

@@ -436,6 +436,6 @@ def test_actions_get_all(
436436

437437
assert len(actions) == 1
438438
assert isinstance(actions[0], BoundAction)
439-
assert actions[0]._client == images_client._client.actions
439+
assert actions[0]._client == images_client._parent.actions
440440
assert actions[0].id == 13
441441
assert actions[0].command == "change_protection"

tests/unit/load_balancers/test_client.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -630,7 +630,7 @@ def test_actions_get_by_id(
630630
request_mock.assert_called_with(url="/load_balancers/actions/13", method="GET")
631631

632632
assert isinstance(action, BoundAction)
633-
assert action._client == load_balancers_client._client.actions
633+
assert action._client == load_balancers_client._parent.actions
634634
assert action.id == 13
635635
assert action.command == "change_protection"
636636

@@ -654,7 +654,7 @@ def test_actions_get_list(
654654

655655
assert len(actions) == 1
656656
assert isinstance(actions[0], BoundAction)
657-
assert actions[0]._client == load_balancers_client._client.actions
657+
assert actions[0]._client == load_balancers_client._parent.actions
658658
assert actions[0].id == 13
659659
assert actions[0].command == "change_protection"
660660

@@ -675,6 +675,6 @@ def test_actions_get_all(
675675

676676
assert len(actions) == 1
677677
assert isinstance(actions[0], BoundAction)
678-
assert actions[0]._client == load_balancers_client._client.actions
678+
assert actions[0]._client == load_balancers_client._parent.actions
679679
assert actions[0].id == 13
680680
assert actions[0].command == "change_protection"

tests/unit/networks/test_client.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -521,7 +521,7 @@ def test_get_actions_list(
521521
assert len(actions) == 1
522522
assert isinstance(actions[0], BoundAction)
523523

524-
assert actions[0]._client == networks_client._client.actions
524+
assert actions[0]._client == networks_client._parent.actions
525525
assert actions[0].id == 13
526526
assert actions[0].command == "add_subnet"
527527

@@ -743,7 +743,7 @@ def test_actions_get_by_id(
743743
request_mock.assert_called_with(url="/networks/actions/13", method="GET")
744744

745745
assert isinstance(action, BoundAction)
746-
assert action._client == networks_client._client.actions
746+
assert action._client == networks_client._parent.actions
747747
assert action.id == 13
748748
assert action.command == "add_subnet"
749749

@@ -767,7 +767,7 @@ def test_actions_get_list(
767767

768768
assert len(actions) == 1
769769
assert isinstance(actions[0], BoundAction)
770-
assert actions[0]._client == networks_client._client.actions
770+
assert actions[0]._client == networks_client._parent.actions
771771
assert actions[0].id == 13
772772
assert actions[0].command == "add_subnet"
773773

@@ -788,6 +788,6 @@ def test_actions_get_all(
788788

789789
assert len(actions) == 1
790790
assert isinstance(actions[0], BoundAction)
791-
assert actions[0]._client == networks_client._client.actions
791+
assert actions[0]._client == networks_client._parent.actions
792792
assert actions[0].id == 13
793793
assert actions[0].command == "add_subnet"

tests/unit/primary_ips/test_client.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -390,7 +390,7 @@ def test_actions_get_by_id(
390390
request_mock.assert_called_with(url="/primary_ips/actions/13", method="GET")
391391

392392
assert isinstance(action, BoundAction)
393-
assert action._client == primary_ips_client._client.actions
393+
assert action._client == primary_ips_client._parent.actions
394394
assert action.id == 13
395395
assert action.command == "assign_primary_ip"
396396

@@ -414,7 +414,7 @@ def test_actions_get_list(
414414

415415
assert len(actions) == 1
416416
assert isinstance(actions[0], BoundAction)
417-
assert actions[0]._client == primary_ips_client._client.actions
417+
assert actions[0]._client == primary_ips_client._parent.actions
418418
assert actions[0].id == 13
419419
assert actions[0].command == "assign_primary_ip"
420420

@@ -435,6 +435,6 @@ def test_actions_get_all(
435435

436436
assert len(actions) == 1
437437
assert isinstance(actions[0], BoundAction)
438-
assert actions[0]._client == primary_ips_client._client.actions
438+
assert actions[0]._client == primary_ips_client._parent.actions
439439
assert actions[0].id == 13
440440
assert actions[0].command == "assign_primary_ip"

0 commit comments

Comments
 (0)