forked from sonic-net/sonic-utilities
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Improve show acl commands (sonic-net#2667)
* Add status for ACL_TABLE and ACL_RULE in STATE_DB
- Loading branch information
1 parent
3d24b00
commit 793b14a
Showing
9 changed files
with
213 additions
and
15 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,95 @@ | ||
| import os | ||
| import pytest | ||
| from click.testing import CliRunner | ||
|
|
||
| import acl_loader.main as acl_loader_show | ||
| from acl_loader import * | ||
| from acl_loader.main import * | ||
| from importlib import reload | ||
|
|
||
| root_path = os.path.dirname(os.path.abspath(__file__)) | ||
| modules_path = os.path.dirname(root_path) | ||
| scripts_path = os.path.join(modules_path, "scripts") | ||
|
|
||
|
|
||
| @pytest.fixture() | ||
| def setup_teardown_single_asic(): | ||
| os.environ["PATH"] += os.pathsep + scripts_path | ||
| os.environ["UTILITIES_UNIT_TESTING"] = "2" | ||
| os.environ["UTILITIES_UNIT_TESTING_TOPOLOGY"] = "" | ||
| yield | ||
| os.environ["UTILITIES_UNIT_TESTING"] = "0" | ||
|
|
||
|
|
||
| @pytest.fixture(scope="class") | ||
| def setup_teardown_multi_asic(): | ||
| os.environ["PATH"] += os.pathsep + scripts_path | ||
| os.environ["UTILITIES_UNIT_TESTING"] = "2" | ||
| os.environ["UTILITIES_UNIT_TESTING_TOPOLOGY"] = "multi_asic" | ||
| from .mock_tables import mock_multi_asic_3_asics | ||
| reload(mock_multi_asic_3_asics) | ||
| from .mock_tables import dbconnector | ||
| dbconnector.load_namespace_config() | ||
| yield | ||
| os.environ["UTILITIES_UNIT_TESTING"] = "0" | ||
| os.environ["UTILITIES_UNIT_TESTING_TOPOLOGY"] = "" | ||
| from .mock_tables import mock_single_asic | ||
| reload(mock_single_asic) | ||
|
|
||
|
|
||
| class TestShowACLSingleASIC(object): | ||
| def test_show_acl_table(self, setup_teardown_single_asic): | ||
| runner = CliRunner() | ||
| aclloader = AclLoader() | ||
| context = { | ||
| "acl_loader": aclloader | ||
| } | ||
| result = runner.invoke(acl_loader_show.cli.commands['show'].commands['table'], ['DATAACL_5'], obj=context) | ||
| assert result.exit_code == 0 | ||
| # We only care about the third line, which contains the 'Active' | ||
| result_top = result.output.split('\n')[2] | ||
| expected_output = "DATAACL_5 L3 Ethernet124 DATAACL_5 ingress Active" | ||
| assert result_top == expected_output | ||
|
|
||
| def test_show_acl_rule(self, setup_teardown_single_asic): | ||
| runner = CliRunner() | ||
| aclloader = AclLoader() | ||
| context = { | ||
| "acl_loader": aclloader | ||
| } | ||
| result = runner.invoke(acl_loader_show.cli.commands['show'].commands['rule'], ['DATAACL_5'], obj=context) | ||
| assert result.exit_code == 0 | ||
| # We only care about the third line, which contains the 'Active' | ||
| result_top = result.output.split('\n')[2] | ||
| expected_output = "DATAACL_5 RULE_1 9999 FORWARD IP_PROTOCOL: 126 Active" | ||
| assert result_top == expected_output | ||
|
|
||
|
|
||
| class TestShowACLMultiASIC(object): | ||
| def test_show_acl_table(self, setup_teardown_multi_asic): | ||
| runner = CliRunner() | ||
| aclloader = AclLoader() | ||
| context = { | ||
| "acl_loader": aclloader | ||
| } | ||
| result = runner.invoke(acl_loader_show.cli.commands['show'].commands['table'], ['DATAACL_5'], obj=context) | ||
| assert result.exit_code == 0 | ||
| # We only care about the third line, which contains the 'Active' | ||
| result_top = result.output.split('\n')[2] | ||
| expected_output = "DATAACL_5 L3 Ethernet124 DATAACL_5 ingress {'asic0': 'Active', 'asic2': 'Active'}" | ||
| assert result_top == expected_output | ||
|
|
||
| def test_show_acl_rule(self, setup_teardown_multi_asic): | ||
| runner = CliRunner() | ||
| aclloader = AclLoader() | ||
| context = { | ||
| "acl_loader": aclloader | ||
| } | ||
| result = runner.invoke(acl_loader_show.cli.commands['show'].commands['rule'], ['DATAACL_5'], obj=context) | ||
| assert result.exit_code == 0 | ||
| # We only care about the third line, which contains the 'Active' | ||
| result_top = result.output.split('\n')[2] | ||
| expected_output = "DATAACL_5 RULE_1 9999 FORWARD IP_PROTOCOL: 126 {'asic0': 'Active', 'asic2': 'Active'}" | ||
| assert result_top == expected_output | ||
|
|
||
|
|