[Asterisk-code-review] Add global pluggable modules. (testsuite[master])

Mark Michelson asteriskteam at digium.com
Fri Jan 29 08:09:43 CST 2016


Hello Kevin Harwell, Anonymous Coward #1000019, Matt Jordan, Joshua Colp,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/1803

to look at the new patch set (#8).

Change subject: Add global pluggable modules.
......................................................................

Add global pluggable modules.

This gives the test runner the ability to load pluggable modules from
the global test-config.yaml in addition to the test-specific
test-config.yaml.

The reason for this is that there are certain pluggable modules that
have the following attributes:
1) They have configuration that is specific to the environment, rather
than to the test itself
2) They cause an overall change in behavior to how a test runs, but the
test itself does not need to be aware of this change.
3) They should be applied to loaded during a large number of tests.

The only pluggable module that fits these criteria at the moment is the
pluggable module that converts a test with static configuration files
into a realtime test. By putting this pluggable module in the global
test-config.yaml file, it allows for tests being run to automatically be
converted into realtime tests without the need for altering any of the
tests themselves.

If a global pluggable module is configured, it will be loaded for all
tests that are run. It is likely, therefore, that test runs that use
global pluggable modules will also specify a certain tag. This allows
for test runs where they can allow tests to opt into the run.

Change-Id: Ieec1b1cfa48cadab108c4ab65122ce36ab697e4e
---
M lib/python/asterisk/test_case.py
M lib/python/asterisk/test_runner.py
M test-config.yaml
M tests/channels/pjsip/configuration/happy_config/test-config.yaml
4 files changed, 62 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/03/1803/8
-- 
To view, visit https://gerrit.asterisk.org/1803
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ieec1b1cfa48cadab108c4ab65122ce36ab697e4e
Gerrit-PatchSet: 8
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list