Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DEV] Add Redfish API tests for more predictable behavior on different hardware #33

Closed
davidallendj opened this issue Jun 26, 2024 · 0 comments · Fixed by #42
Closed
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed priority High priority issue or task

Comments

@davidallendj
Copy link
Collaborator

davidallendj commented Jun 26, 2024

Short Description
After using Magellan with different systems with different Redfish versions and implementations, it has become apparent that Magellan needs a way of ensuring predictable behavior. This is especially needed when testing on new systems where it would easier to see when and where there is a failure.

For example, one system may provide EthernetInterfaces under the /redfish/v1/Systems/EthernetInterfaces endpoint while another system provides it via the /redfish/v1/Managers/EthernetInterfaces endpoint. This real scenario required handling each endpoint differently, and would be able to be added and caught by tests if the tool is ran on other hardware with a similar Redfish API.

Definition of Done
This issue is considered completed once there are tests for testing Redfish's API to provide some guarantees. These tests can simply be HTTP requests that assert specific output and does not require any other sophisticated features.

Additional context
This issue is related to #31, but strictly addresses adding unit tests for Redfish API predictability.

@davidallendj davidallendj added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed priority High priority issue or task labels Jun 26, 2024
@davidallendj davidallendj linked a pull request Jul 17, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed priority High priority issue or task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant