Adds test_redfish_credential_validation which sets the redfish credentials and checks if the charm is funtioning as intended. This is necessary for hardware dependent tests since the charm only goes to active state once the correct redfish credentials are configured (assuming redfish is present on the underlying machine).
test_build_and_deploy is broken up into smaller tests so that they are easier to manage. Also modifies logic in conftest.py so that all these tests are executed whether they're hw dependent or independent.
test_redfish_metrics has also been modified since we are setting the redfish credentials as part of the earlier test. The resource based tests (test_resource_in_correct_location, test_wrong_resource_attached and test_resource_clean_up) have been moved towards the end of TestCharmWithHW so that all the metrics based tests can be grouped together in the beginning.
Finally, a retry mechanism has been added to test_metrics_available as it was found while testing that it takes some time for the endpoint to be ready with the metrics.
Adds
test_redfish_credential_validation
which sets the redfish credentials and checks if the charm is funtioning as intended. This is necessary for hardware dependent tests since the charm only goes to active state once the correct redfish credentials are configured (assuming redfish is present on the underlying machine).test_build_and_deploy
is broken up into smaller tests so that they are easier to manage. Also modifies logic in conftest.py so that all these tests are executed whether they're hw dependent or independent.test_redfish_metrics
has also been modified since we are setting the redfish credentials as part of the earlier test. The resource based tests (test_resource_in_correct_location
,test_wrong_resource_attached
andtest_resource_clean_up
) have been moved towards the end ofTestCharmWithHW
so that all the metrics based tests can be grouped together in the beginning.Finally, a retry mechanism has been added to
test_metrics_available
as it was found while testing that it takes some time for the endpoint to be ready with the metrics.Note: Requires #192 to be merged.