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

Add MessageRegistryFile to validation, small RedfishObject tweaks, mild refactoring #438

Closed
wants to merge 15 commits into from

Conversation

tomasg2012
Copy link
Contributor

@tomasg2012 tomasg2012 commented Nov 12, 2021

Many changes to refactor where functions are located

Otherwise, refactoring of URITree Validation to validate any URIs given, not just links.

TODO: Wait for PR #439 to clear

mraineri and others added 14 commits November 12, 2021 14:37
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
…lder than 3.8

Signed-off-by: Mike Raineri <michael.raineri@dell.com>
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
Signed-off-by: Mike Raineri <michael.raineri@dell.com>
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
The tool can now check URIs on any service at and above version 1.6.0,
testing all @odata.id possible on each Resource.

Now tests headers to check if Capabilities is properly applied to the
Allow header.

fix: MinVersion is now lessthan or equal

tweak: Nav properties now have RedfishType.IsNav

fix: Fix modified results using success rather than result

...

Changed Error to Warn on PATCH/PUT header

Renamed variables, made URI checking less strict

Loosened URI pattern regex, Pound sign check for fragments

tweak/fix: Change fulltype variable, proper check for incorrect non-dictionary RedfishObject values

fix: Fixed DynamicProperties being incorrectly pulled from schema
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
Signed-off-by: Tomas <tomas.g.2012@gmail.com>
@tomasg2012
Copy link
Contributor Author

Closing this since signing/merging from last week broke the branch...

@tomasg2012 tomasg2012 closed this Nov 19, 2021
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.

None yet

2 participants