Skip to content

Commit

Permalink
[dualtor]: Fix expect_db_values arguments order (#3095)
Browse files Browse the repository at this point in the history
Signed-off-by: Lawrence Lee <[email protected]>
  • Loading branch information
theasianpianist authored Mar 4, 2021
1 parent 3b432d6 commit 884f25c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/common/dualtor/control_plane_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ def _dump_db(duthost, db, key_pattern):
return json.loads(lines[0])


def expect_db_values(duthost, db, intf_names='all', state, health=None):
def expect_db_values(duthost, db, state, health=None, intf_names='all'):
"""
Query db on `tor_host` and check if the mux-related fields match the expected values.
Expand All @@ -52,9 +52,9 @@ def expect_db_values(duthost, db, intf_names='all', state, health=None):
Args:
duthost: DUT host object (needs to be passed by calling function from duthosts fixture)
db: Database number to check. Should be either 0 for APP_DB or 6 for STATE_DB
intf_names: A list of the PORTNAME to check in each table, or 'all' (by default) to check all MUX_CABLE interfaces
state: The expected value for each of the `state` fields in both tables
health: The expected value for the `state` field in the MUX_LINKMGR_TABLE table (only needed for STATE_DB)
intf_names: A list of the PORTNAME to check in each table, or 'all' (by default) to check all MUX_CABLE interfaces
Raises:
AssertionError if th mux cable fields don't match the given states.
Expand Down

0 comments on commit 884f25c

Please sign in to comment.