Skip to content

Commit

Permalink
Fix test case timing issue
Browse files Browse the repository at this point in the history
  • Loading branch information
liuh-80 committed Sep 29, 2024
1 parent df84a4e commit e8e6a70
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
3 changes: 3 additions & 0 deletions tests/dash/dash_db.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
from dvslib.dvs_common import wait_for_result
import typing
import pytest
import time

from dash_api.appliance_pb2 import *
from dash_api.vnet_pb2 import *
Expand Down Expand Up @@ -58,9 +59,11 @@ def __setitem__(self, key: str, pairs: typing.Union[dict, list, tuple]):
for k, v in pairs:
pairs_str.append((to_string(k), to_string(v)))
self.set(key, pairs_str)
time.sleep(3)

def __delitem__(self, key: str):
self.delete(str(key))
time.sleep(3)


class Table(swsscommon.Table):
Expand Down
7 changes: 7 additions & 0 deletions tests/dash/test_dash_acl.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,10 @@

import pytest

import logging
logging.basicConfig(level=logging.INFO)
logger = logging.getLogger(__name__)

DVS_ENV = ["HWSKU=DPU-2P"]
NUM_PORTS = 2

Expand Down Expand Up @@ -87,10 +91,12 @@ def __setitem__(self, key: str, pairs: Union[dict, list, tuple]):
pairs_str.append((to_string(k), to_string(v)))
self.table.set(key, pairs_str)
self.keys.add(key)
time.sleep(3)

def __delitem__(self, key: str):
self.table.delete(str(key))
self.keys.discard(key)
time.sleep(3)

def get_keys(self):
return self.keys
Expand Down Expand Up @@ -242,6 +248,7 @@ def remove_acl_rule(self, group_id, rule_id):
def create_acl_group(self, group_id, ip_version):
pb = AclGroup()
pb.ip_version = IpVersion.IP_VERSION_IPV4
logger.warning("create_acl_group: {}='{}'".format(str(group_id), pb.SerializeToString()))
self.app_dash_acl_group_table[str(group_id)] = {"pb": pb.SerializeToString()}

def remove_acl_group(self, group_id):
Expand Down

0 comments on commit e8e6a70

Please sign in to comment.