Skip to content

Add first unit and integration tests#8

Open
Dispader wants to merge 1 commit intoUMN-PeopleSoft:masterfrom
Dispader:add_first_tests
Open

Add first unit and integration tests#8
Dispader wants to merge 1 commit intoUMN-PeopleSoft:masterfrom
Dispader:add_first_tests

Conversation

@Dispader
Copy link
Contributor

This commit splits the unit tests from the integration tests, and adds one of each by introducing a Config object capable of testing the loading of configuration properties.

This object could in turn be introduced into the IBMonitorSvc initialization method using the integration test.

This commit splits the unit tests from the integration tests, and adds
one of each by introducing a Config object capable of testing the
loading of configuration properties.  This object could in turn be
introduced into the IBMonitorSvc initialization method using the
integration test.
@Dispader
Copy link
Contributor Author

@carl0426 Hey, man— I'd be happy to give the YAML config a shot, iffn we can agree the config tests in this Pull are at least minimally valid. What d'ya say?

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.

1 participant