Passed
Pull Request — master (#49)
by
unknown
03:07
created

TestMain.test_traces_twice()   B

Complexity

Conditions 1

Size

Total Lines 58
Code Lines 39

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 39
nop 3
dl 0
loc 58
rs 8.9439
c 0
b 0
f 0

How to fix   Long Method   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
"""Module to test the main napp file."""
2
import json
3
from unittest import TestCase
4
from unittest.mock import patch, MagicMock
5
6
from kytos.core.interface import Interface
7
from kytos.lib.helpers import (
8
    get_interface_mock,
9
    get_switch_mock,
10
    get_controller_mock,
11
    get_link_mock,
12
    get_test_client,
13
)
14
15
16
# pylint: disable=too-many-public-methods, too-many-lines
17
class TestMain(TestCase):
18
    """Test the Main class."""
19
20
    def setUp(self):
21
        """Execute steps before each tests.
22
23
        Set the server_name_url_url from amlight/sdntrace_cp
24
        """
25
        self.server_name_url = "http://localhost:8181/api/amlight/sdntrace_cp"
26
27
        # The decorator run_on_thread is patched, so methods that listen
28
        # for events do not run on threads while tested.
29
        # Decorators have to be patched before the methods that are
30
        # decorated with them are imported.
31
        patch("kytos.core.helpers.run_on_thread", lambda x: x).start()
32
        # pylint: disable=import-outside-toplevel
33
        from napps.amlight.sdntrace_cp.main import Main
34
35
        self.napp = Main(get_controller_mock())
36
37
    @staticmethod
38
    def get_napp_urls(napp):
39
        """Return the amlight/sdntrace_cp urls.
40
41
        The urls will be like:
42
43
        urls = [
44
            (options, methods, url)
45
        ]
46
47
        """
48
        controller = napp.controller
49
        controller.api_server.register_napp_endpoints(napp)
50
51
        urls = []
52
        for rule in controller.api_server.app.url_map.iter_rules():
53
            options = {}
54
            for arg in rule.arguments:
55
                options[arg] = f"[{0}]".format(arg)
56
57
            if f"{napp.username}/{napp.name}" in str(rule):
58
                urls.append((options, rule.methods, f"{str(rule)}"))
59
60
        return urls
61
62
    def test_verify_api_urls(self):
63
        """Verify all APIs registered."""
64
65
        expected_urls = [
66
            (
67
                {},
68
                {"OPTIONS", "HEAD", "PUT"},
69
                "/api/amlight/sdntrace_cp/trace/ ",
70
            ),
71
            (
72
                {},
73
                {"OPTIONS", "HEAD", "PUT"},
74
                "/api/amlight/sdntrace_cp/traces/ ",
75
            ),
76
        ]
77
        urls = self.get_napp_urls(self.napp)
78
        self.assertEqual(len(expected_urls), len(urls))
79
80
    @patch("napps.amlight.sdntrace_cp.main.Main.match_and_apply")
81
    def test_trace_step(self, mock_flow_match):
82
        """Test trace_step success result."""
83
        mock_flow_match.return_value = ["1"], ["entries"], 1
84
        switch = get_switch_mock("00:00:00:00:00:00:00:01", 0x04)
85
86
        mock_interface = Interface("interface A", 1, MagicMock())
87
        mock_interface.address = "00:00:00:00:00:00:00:01"
88
89
        iface1 = get_interface_mock(
90
            "", 1, get_switch_mock("00:00:00:00:00:00:00:01")
91
        )
92
        iface2 = get_interface_mock(
93
            "", 2, get_switch_mock("00:00:00:00:00:00:00:02")
94
        )
95
        mock_interface.link = get_link_mock(iface1, iface2)
96
        mock_interface.link.endpoint_a.port_number = 1
97
        mock_interface.link.endpoint_a.port_number = 2
98
99
        # Patch for utils.find_endpoint
100
        switch.get_interface_by_port_no.return_value = mock_interface
101
102
        entries = MagicMock()
103
104
        stored_flows = {
105
            "flow": {
106
                "table_id": 0,
107
                "cookie": 84114964,
108
                "hard_timeout": 0,
109
                "idle_timeout": 0,
110
                "priority": 10,
111
            },
112
            "flow_id": 1,
113
            "state": "installed",
114
            "switch": "00:00:00:00:00:00:00:01",
115
        }
116
117
        stored_flows_arg = {
118
            "00:00:00:00:00:00:00:01": [stored_flows]
119
        }
120
121
        result = self.napp.trace_step(switch, entries, stored_flows_arg)
122
123
        mock_flow_match.assert_called_once()
124
        self.assertEqual(
125
            result,
126
            {
127
                "dpid": "00:00:00:00:00:00:00:01",
128
                "in_port": 2,
129
                "out_port": 1,
130
                "entries": ["entries"],
131
            },
132
        )
133
134
    @patch("napps.amlight.sdntrace_cp.main.Main.match_and_apply")
135
    def test_trace_step__no_endpoint(self, mock_flow_match):
136
        """Test trace_step without endpoints available for switch/port."""
137
        mock_flow_match.return_value = ["1"], ["entries"], 1
138
        switch = get_switch_mock("00:00:00:00:00:00:00:01", 0x04)
139
140
        mock_interface = Interface("interface A", 1, MagicMock())
141
        mock_interface.address = "00:00:00:00:00:00:00:01"
142
        mock_interface.link = None
143
144
        # Patch for utils.find_endpoint
145
        switch.get_interface_by_port_no.return_value = mock_interface
146
147
        entries = MagicMock()
148
149
        stored_flows = {
150
            "flow": {
151
                "table_id": 0,
152
                "cookie": 84114964,
153
                "hard_timeout": 0,
154
                "idle_timeout": 0,
155
                "priority": 10,
156
            },
157
            "flow_id": 1,
158
            "state": "installed",
159
            "switch": "00:00:00:00:00:00:00:01",
160
        }
161
162
        stored_flows_arg = {
163
            "00:00:00:00:00:00:00:01": [stored_flows]
164
        }
165
166
        result = self.napp.trace_step(switch, entries, stored_flows_arg)
167
168
        mock_flow_match.assert_called_once()
169
        self.assertEqual(result, {"entries": ["entries"], "out_port": 1})
170
171
    def test_trace_step__no_flow(self):
172
        """Test trace_step without flows for the switch."""
173
        switch = get_switch_mock("00:00:00:00:00:00:00:01")
174
        entries = MagicMock()
175
176
        stored_flows = {
177
            "flow": {
178
                "table_id": 0,
179
                "cookie": 84114964,
180
                "hard_timeout": 0,
181
                "idle_timeout": 0,
182
                "priority": 10,
183
            },
184
            "flow_id": 1,
185
            "state": "installed",
186
            "switch": "00:00:00:00:00:00:00:01",
187
        }
188
189
        stored_flows_arg = {
190
            "00:00:00:00:00:00:00:01": [stored_flows]
191
        }
192
193
        result = self.napp.trace_step(switch, entries, stored_flows_arg)
194
        assert result is None
195
196
    @patch("napps.amlight.sdntrace_cp.main.Main.trace_step")
197
    def test_tracepath(self, mock_trace_step):
198
        """Test tracepath with success result."""
199
        eth = {"dl_vlan": 100}
200
        dpid = {"dpid": "00:00:00:00:00:00:00:01", "in_port": 1}
201
        switch = {"switch": dpid, "eth": eth}
202
        entries = {"trace": switch}
203
        mock_trace_step.return_value = {
204
            "dpid": "00:00:00:00:00:00:00:02",
205
            "in_port": 2,
206
            "out_port": 3,
207
            "entries": entries,
208
        }
209
        stored_flows = {
210
            "flow": {
211
                "table_id": 0,
212
                "cookie": 84114964,
213
                "hard_timeout": 0,
214
                "idle_timeout": 0,
215
                "priority": 10,
216
            },
217
            "flow_id": 1,
218
            "state": "installed",
219
            "switch": "00:00:00:00:00:00:00:01",
220
        }
221
222
        stored_flows_arg = {
223
            "00:00:00:00:00:00:00:01": [stored_flows]
224
        }
225
226
        switch_01 = get_switch_mock("00:00:00:00:00:00:00:01", 0x04)
227
        switch_01.is_enabled.return_value = True
228
229
        self.napp.controller.switches = {
230
            "00:00:00:00:00:00:00:01": switch_01
231
        }
232
233
        result = self.napp.tracepath(
234
                                        entries["trace"]["switch"],
235
                                        stored_flows_arg
236
                                    )
237
238
        assert result[0]["in"]["dpid"] == "00:00:00:00:00:00:00:01"
239
        assert result[0]["in"]["port"] == 1
240
        assert result[0]["in"]["type"] == "starting"
241
242
        assert result[1]["in"]["dpid"] == "00:00:00:00:00:00:00:02"
243
        assert result[1]["in"]["port"] == 2
244
        assert result[1]["in"]["type"] == "trace"
245
246
    def test_has_loop(self):
247
        """Test has_loop to detect a tracepath with loop."""
248
        trace_result = [
249
            {
250
                "in": {
251
                    "dpid": "00:00:00:00:00:00:00:01",
252
                    "port": 2,
253
                },
254
                "out": {
255
                    "port": 1,
256
                },
257
            },
258
            {
259
                "in": {
260
                    "dpid": "00:00:00:00:00:00:00:03",
261
                    "port": 2,
262
                },
263
                "out": {
264
                    "port": 1,
265
                },
266
            },
267
            {
268
                "in": {
269
                    "dpid": "00:00:00:00:00:00:00:03",
270
                    "port": 3,
271
                },
272
                "out": {
273
                    "port": 1,
274
                },
275
            },
276
            {
277
                "in": {
278
                    "dpid": "00:00:00:00:00:00:00:03",
279
                    "port": 3,
280
                },
281
                "out": {
282
                    "port": 1,
283
                },
284
            },
285
        ]
286
        trace_step = {
287
            "dpid": "00:00:00:00:00:00:00:03",
288
            "port": 3,
289
        }
290
291
        result = self.napp.has_loop(trace_step, trace_result)
292
293
        self.assertTrue(result)
294
295
    def test_has_loop__fail(self):
296
        """Test has_loop to detect a tracepath with loop."""
297
        trace_result = [
298
            {
299
                "in": {
300
                    "dpid": "00:00:00:00:00:00:00:01",
301
                    "port": 2,
302
                },
303
                "out": {
304
                    "port": 1,
305
                },
306
            },
307
            {
308
                "in": {
309
                    "dpid": "00:00:00:00:00:00:00:02",
310
                    "port": 2,
311
                },
312
                "out": {
313
                    "port": 1,
314
                },
315
            },
316
        ]
317
        trace_step = {
318
            "dpid": "00:00:00:00:00:00:00:03",
319
            "port": 2,
320
        }
321
322
        result = self.napp.has_loop(trace_step, trace_result)
323
324
        self.assertFalse(result)
325
326
    @patch("napps.amlight.sdntrace_cp.main.settings")
327
    def test_update_circuits(self, mock_settings):
328
        """Test update_circuits event listener with success."""
329
        mock_settings.FIND_CIRCUITS_IN_FLOWS = True
330
331
        self.napp.automate = MagicMock()
332
        self.napp.automate.find_circuits = MagicMock()
333
334
        self.napp.update_circuits()
335
336
        self.napp.automate.find_circuits.assert_called_once()
337
338
    @patch("napps.amlight.sdntrace_cp.main.settings")
339
    def test_update_circuits__no_settings(self, mock_settings):
340
        """Test update_circuits event listener without
341
        settings option enabled."""
342
        mock_settings.FIND_CIRCUITS_IN_FLOWS = False
343
344
        self.napp.automate = MagicMock()
345
        self.napp.automate.find_circuits = MagicMock()
346
347
        self.napp.update_circuits()
348
349
        self.napp.automate.find_circuits.assert_not_called()
350
351
    @patch("napps.amlight.sdntrace_cp.utils.get_stored_flows")
352
    @patch("napps.amlight.sdntrace_cp.utils.requests")
353
    def test_trace(self, mock_request_get, mock_stored_flows):
354
        """Test trace rest call."""
355
        api = get_test_client(get_controller_mock(), self.napp)
356
        url = f"{self.server_name_url}/trace/"
357
358
        payload = {
359
            "trace": {
360
                "switch": {
361
                    "dpid": "00:00:00:00:00:00:00:01",
362
                    "in_port": 1
363
                    },
364
                "eth": {"dl_vlan": 100},
365
            }
366
        }
367
        stored_flows = {
368
                "flow": {
369
                    "table_id": 0,
370
                    "cookie": 84114964,
371
                    "hard_timeout": 0,
372
                    "idle_timeout": 0,
373
                    "priority": 10,
374
                },
375
                "flow_id": 1,
376
                "state": "installed",
377
                "switch": "00:00:00:00:00:00:00:01",
378
        }
379
        mock_stored_flows.return_value = {
380
            "00:00:00:00:00:00:00:01": [stored_flows]
381
        }
382
        mock_json = MagicMock()
383
        mock_json.json.return_value = {
384
            "00:00:00:00:00:00:00:01": [stored_flows]
385
        }
386
        mock_request_get.get.return_value = mock_json
387
388
        response = api.put(
389
            url, data=json.dumps(payload), content_type="application/json"
390
        )
391
        current_data = json.loads(response.data)
392
        result = current_data["result"]
393
394
        assert result[0]["dpid"] == "00:00:00:00:00:00:00:01"
395
        assert result[0]["port"] == 1
396
        assert result[0]["type"] == "starting"
397
        assert result[0]["vlan"] == 100
398
399
    @patch("napps.amlight.sdntrace_cp.utils.get_stored_flows")
400
    @patch("napps.amlight.sdntrace_cp.utils.requests")
401
    def test_get_traces(self, mock_request_get, mock_stored_flows):
402
        """Test traces rest call."""
403
        api = get_test_client(get_controller_mock(), self.napp)
404
        url = f"{self.server_name_url}/traces/"
405
406
        payload = [{
407
            "trace": {
408
                "switch": {
409
                    "dpid": "00:00:00:00:00:00:00:01",
410
                    "in_port": 1
411
                    },
412
                "eth": {"dl_vlan": 100},
413
            }
414
        }]
415
416
        stored_flow = {
417
                "id": 1,
418
                "table_id": 0,
419
                "cookie": 84114964,
420
                "hard_timeout": 0,
421
                "idle_timeout": 0,
422
                "priority": 10,
423
        }
424
425
        mock_stored_flows.return_value = {
426
            "00:00:00:00:00:00:00:01": [stored_flow]
427
        }
428
        mock_json = MagicMock()
429
        mock_json.json.return_value = {
430
            "00:00:00:00:00:00:00:01": [stored_flow]
431
        }
432
        mock_request_get.get.return_value = mock_json
433
        response = api.put(
434
            url, data=json.dumps(payload), content_type="application/json"
435
        )
436
        current_data = json.loads(response.data)
437
        result1 = current_data["00:00:00:00:00:00:00:01"]
438
439
        assert result1[0][0]["dpid"] == "00:00:00:00:00:00:00:01"
440
        assert result1[0][0]["port"] == 1
441
        assert result1[0][0]["type"] == "starting"
442
        assert result1[0][0]["vlan"] == 100
443
444 View Code Duplication
    @patch("napps.amlight.sdntrace_cp.utils.get_stored_flows")
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated in your project.
Loading history...
445
    @patch("napps.amlight.sdntrace_cp.utils.requests")
446
    def test_traces(self, mock_request_get, mock_stored_flows):
447
        """Test traces rest call for two traces with different switches."""
448
        api = get_test_client(get_controller_mock(), self.napp)
449
        url = f"{self.server_name_url}/traces/"
450
451
        payload = [
452
            {
453
                "trace": {
454
                    "switch": {
455
                        "dpid": "00:00:00:00:00:00:00:01",
456
                        "in_port": 1
457
                        },
458
                    "eth": {"dl_vlan": 100},
459
                }
460
            },
461
            {
462
                "trace": {
463
                    "switch": {
464
                        "dpid": "00:00:00:00:00:00:00:02",
465
                        "in_port": 1},
466
                    "eth": {"dl_vlan": 100},
467
                }
468
            }
469
        ]
470
471
        stored_flow = {
472
                "id": 1,
473
                "table_id": 0,
474
                "cookie": 84114964,
475
                "hard_timeout": 0,
476
                "idle_timeout": 0,
477
                "priority": 10,
478
        }
479
480
        mock_stored_flows.return_value = {
481
            "00:00:00:00:00:00:00:01": [stored_flow]
482
        }
483
        mock_json = MagicMock()
484
        mock_json.json.return_value = {
485
            "00:00:00:00:00:00:00:01": [stored_flow]
486
        }
487
        mock_request_get.get.return_value = mock_json
488
489
        response = api.put(
490
            url, data=json.dumps(payload), content_type="application/json"
491
        )
492
        current_data = json.loads(response.data)
493
        result1 = current_data["00:00:00:00:00:00:00:01"]
494
        result2 = current_data["00:00:00:00:00:00:00:02"]
495
496
        assert result1[0][0]["dpid"] == "00:00:00:00:00:00:00:01"
497
        assert result1[0][0]["port"] == 1
498
        assert result1[0][0]["type"] == "starting"
499
        assert result1[0][0]["vlan"] == 100
500
501
        assert result2[0][0]["dpid"] == "00:00:00:00:00:00:00:02"
502
        assert result2[0][0]["port"] == 1
503
        assert result2[0][0]["type"] == "starting"
504
        assert result2[0][0]["vlan"] == 100
505
506 View Code Duplication
    @patch("napps.amlight.sdntrace_cp.utils.get_stored_flows")
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated in your project.
Loading history...
507
    @patch("napps.amlight.sdntrace_cp.utils.requests")
508
    def test_traces_same_switch(self, mock_request_get, mock_stored_flows):
509
        """Test traces rest call for two traces with samw switches."""
510
        api = get_test_client(get_controller_mock(), self.napp)
511
        url = f"{self.server_name_url}/traces/"
512
513
        payload = [
514
            {
515
                "trace": {
516
                    "switch": {
517
                        "dpid": "00:00:00:00:00:00:00:01",
518
                        "in_port": 1
519
                    },
520
                    "eth": {"dl_vlan": 100},
521
                }
522
            },
523
            {
524
                "trace": {
525
                    "switch": {
526
                        "dpid": "00:00:00:00:00:00:00:01",
527
                        "in_port": 2
528
                    },
529
                    "eth": {"dl_vlan": 100},
530
                }
531
            }
532
        ]
533
534
        stored_flow = {
535
                "id": 1,
536
                "table_id": 0,
537
                "cookie": 84114964,
538
                "hard_timeout": 0,
539
                "idle_timeout": 0,
540
                "priority": 10,
541
        }
542
543
        mock_stored_flows.return_value = {
544
            "00:00:00:00:00:00:00:01": [stored_flow]
545
        }
546
        mock_json = MagicMock()
547
        mock_json.json.return_value = {
548
            "00:00:00:00:00:00:00:01": [stored_flow]
549
        }
550
        mock_request_get.get.return_value = mock_json
551
552
        response = api.put(
553
            url, data=json.dumps(payload), content_type="application/json"
554
        )
555
        current_data = json.loads(response.data)
556
        result = current_data["00:00:00:00:00:00:00:01"]
557
558
        assert len(current_data) == 1
559
560
        assert result[0][0]["dpid"] == "00:00:00:00:00:00:00:01"
561
        assert result[0][0]["port"] == 1
562
        assert result[0][0]["type"] == "starting"
563
        assert result[0][0]["vlan"] == 100
564
565
        assert result[1][0]["dpid"] == "00:00:00:00:00:00:00:01"
566
        assert result[1][0]["port"] == 2
567
        assert result[1][0]["type"] == "starting"
568
        assert result[1][0]["vlan"] == 100
569
570
    @patch("napps.amlight.sdntrace_cp.utils.get_stored_flows")
571
    @patch("napps.amlight.sdntrace_cp.utils.requests")
572
    def test_traces_twice(self, mock_request_get, mock_stored_flows):
573
        """Test traces rest call for two equal traces."""
574
        api = get_test_client(get_controller_mock(), self.napp)
575
        url = f"{self.server_name_url}/traces/"
576
577
        payload = [
578
            {
579
                "trace": {
580
                    "switch": {
581
                        "dpid": "00:00:00:00:00:00:00:01",
582
                        "in_port": 1
583
                        },
584
                    "eth": {"dl_vlan": 100},
585
                }
586
            },
587
            {
588
                "trace": {
589
                    "switch": {
590
                        "dpid": "00:00:00:00:00:00:00:01",
591
                        "in_port": 1
592
                        },
593
                    "eth": {"dl_vlan": 100},
594
                }
595
            }
596
        ]
597
        stored_flow = {
598
                "id": 1,
599
                "table_id": 0,
600
                "cookie": 84114964,
601
                "hard_timeout": 0,
602
                "idle_timeout": 0,
603
                "priority": 10,
604
        }
605
606
        mock_stored_flows.return_value = {
607
            "00:00:00:00:00:00:00:01": [stored_flow]
608
        }
609
        mock_json = MagicMock()
610
        mock_json.json.return_value = {
611
            "00:00:00:00:00:00:00:01": [stored_flow]
612
        }
613
        mock_request_get.get.return_value = mock_json
614
615
        response = api.put(
616
            url, data=json.dumps(payload), content_type="application/json"
617
        )
618
        current_data = json.loads(response.data)
619
        result = current_data["00:00:00:00:00:00:00:01"]
620
621
        assert len(current_data) == 1
622
        assert len(result) == 1
623
624
        assert result[0][0]["dpid"] == "00:00:00:00:00:00:00:01"
625
        assert result[0][0]["port"] == 1
626
        assert result[0][0]["type"] == "starting"
627
        assert result[0][0]["vlan"] == 100
628