Skip to content

chore: sync Redfish artifacts from redfish-tools pr-5#982

Merged
DevipriyaS17 merged 4 commits into
redfishfrom
automation/redfish-sync-pr-5
May 18, 2026
Merged

chore: sync Redfish artifacts from redfish-tools pr-5#982
DevipriyaS17 merged 4 commits into
redfishfrom
automation/redfish-sync-pr-5

Conversation

@DevipriyaS17
Copy link
Copy Markdown
Contributor

Automated sync of generated Redfish artifacts from redfish-tools.

Source:

Generated and synced files:

  • redfish/openapi/redfish-openapi.yaml
  • redfish/internal/controller/http/v1/generated/spec.gen.go
  • redfish/internal/controller/http/v1/generated/types.gen.go
  • redfish/internal/controller/http/v1/generated/server.gen.go
  • redfish/internal/controller/http/v1/handler/metadata.xml

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.22%. Comparing base (536f67a) to head (6e33e82).

Additional details and impacted files
@@             Coverage Diff             @@
##           redfish     #982      +/-   ##
===========================================
+ Coverage    40.99%   41.22%   +0.22%     
===========================================
  Files          152      152              
  Lines        14330    14345      +15     
===========================================
+ Hits          5875     5914      +39     
+ Misses        7882     7858      -24     
  Partials       573      573              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…erated

- Add 6 tests for SOL consent action handlers (CancelSolConsent, RequestSolConsent, SubmitSolConsentCode)
- Add comprehensive tests for convertStateToGenerated covering all 12 state values plus error cases
- Tests verify correct enum mappings and invalid state handling
- Extract validation logic into helper function to reduce cognitive complexity
- Add t.Helper() call to properly mark test helper function
- Add blank line before return statement per lint requirements
@DevipriyaS17 DevipriyaS17 merged commit 921de61 into redfish May 18, 2026
21 checks passed
@DevipriyaS17 DevipriyaS17 deleted the automation/redfish-sync-pr-5 branch May 18, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants