Skip to content

Commit 38ce9cb

Browse files
wen587wangxin
authored andcommitted
[override_config_table] ignore non user config table in empty input compare (#7159)
Summary: Bypass non user config table in empty intput test. What is the motivation for this PR? The empty input compare failed in FLEX_COUNTER_TABLE comparison. This specific table will have derivative change after load_minigraph. We can skip it as it is not user config. How did you do it? Bypass non user config table in empty input compare. How did you verify/test it? Local test.
1 parent 17d4e94 commit 38ce9cb

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

tests/override_config_table/test_override_config_table.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,13 @@ def load_minigraph_with_golden_empty_input(duthost):
103103
reload_minigraph_with_golden_config(duthost, empty_input)
104104

105105
current_config = get_running_config(duthost)
106-
pytest_assert(initial_config == current_config,
107-
"Running config differs.")
106+
for table in initial_config:
107+
if table in NON_USER_CONFIG_TABLES:
108+
continue
109+
pytest_assert(
110+
initial_config[table] == current_config[table],
111+
"empty input compare fail! {}".format(table)
112+
)
108113

109114

110115
def load_minigraph_with_golden_partial_config(duthost):

0 commit comments

Comments
 (0)