Skip to content

Parallelize Test Suite Execution #348

Closed
tahmed-dev wants to merge 2 commits intosonic-net:masterfrom
tahmed-dev:tahmed/octans-int
Closed

Parallelize Test Suite Execution #348
tahmed-dev wants to merge 2 commits intosonic-net:masterfrom
tahmed-dev:tahmed/octans-int

Conversation

@tahmed-dev
Copy link
Contributor

  • Add pytest-xdist to testing dependencies
  • Change MockConfigDb.CONFIG_DB default from None to {} to prevent TypeError in setUp methods that assign to CONFIG_DB[table]
  • Fix caclmgrd test files to use correct swsscommon import: change "import swsscommon" to "from swsscommon import swsscommon" so swsscommon.ConfigDBConnector correctly resolves to the submodule
  • Add missing ConfigDBConnector mock in generate_internal_docker_ip and namespace_docker_ip tests
  • Add tests/caclmgrd/conftest.py with autouse fixture that initializes DEVICE_METADATA and FEATURE tables in MockConfigDb.CONFIG_DB, required by ControlPlaneAclManager.init

Signed-off-by: Tamer Ahmed tamerahmed@microsoft.com

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 17, 2026

CLA Missing ID CLA Not Signed

@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Tamer Ahmed <tamerahmed@microsoft.com>
…t compat

- Change MockConfigDb.CONFIG_DB default from None to {} to prevent
  TypeError in setUp methods that assign to CONFIG_DB[table]
- Fix caclmgrd test files to use correct swsscommon import: change
  "import swsscommon" to "from swsscommon import swsscommon" so
  swsscommon.ConfigDBConnector correctly resolves to the submodule
- Add missing ConfigDBConnector mock in generate_internal_docker_ip
  and namespace_docker_ip tests
- Add tests/caclmgrd/conftest.py with autouse fixture that initializes
  DEVICE_METADATA and FEATURE tables in MockConfigDb.CONFIG_DB,
  required by ControlPlaneAclManager.__init__

Fixes 18 of 24 test failures (remaining 6 are pre-existing SWIG
type mismatches in ldap/radius/tacacs tests).

Signed-off-by: Tamer Ahmed <tamerahmed@microsoft.com>
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tahmed-dev tahmed-dev closed this Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants