search

kytos-ng / kytos-end-to-end-tests

Kytos End-to-End Tests
0 stars 10 forks source link

Enhance `kytos_stats` #253

Closed gretelliz closed 10 months ago

gretelliz commented 11 months ago

Closes #243

Summary

Add test_036_table_1_active_count_update test to create some traffic on table 1, and make sure that the stats are accurate.

Update test_015_packet_count and test_020_bytes_count tests to avoid trying to find a non-empty flow, instead a flow is installed and query it later for stats.

gretelliz commented 11 months ago

@viniarck , Thank you.

viniarck commented 10 months ago

@gretelliz check out with @italovalcy if this can already land, thanks.

gretelliz commented 10 months ago

Thank you, @italovalcy. Good suggestion for incorporating dataplane checks.

gretelliz commented 10 months ago

@gretelliz looks like this PR is ready to land since you've also addressed @italovalcy's feedback. Please re-run this test suite if it's still passing that feel free to merge.

tests/test_e2e_70_kytos_stats.py ........                                [100%]

=============================== warnings summary ===============================
test_e2e_70_kytos_stats.py: 17 warnings
  /usr/lib/python3/dist-packages/mininet/node.py:1121: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
    return ( StrictVersion( cls.OVSVersion ) <

test_e2e_70_kytos_stats.py: 17 warnings
  /usr/lib/python3/dist-packages/mininet/node.py:1122: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
    StrictVersion( '1.10' ) )

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
------------------------------- start/stop times -------------------------------
================== 8 passed, 34 warnings in 467.41s (0:07:47) ==================