Skip to content

fix(e2e): stub pingIdentity and email config for plugin-sanity - #5242

Open
nickboldt wants to merge 2 commits into
mainfrom
RHIDP-13884-plugin-sanity-config-stubs
Open

fix(e2e): stub pingIdentity and email config for plugin-sanity#5242
nickboldt wants to merge 2 commits into
mainfrom
RHIDP-13884-plugin-sanity-config-stubs

Conversation

@nickboldt

Copy link
Copy Markdown
Member

Summary

  • Add dummy catalog.providers.pingIdentityOrg and notifications.processors.email stubs to app-config.plugin-sanity.yaml so cluster-free plugin-sanity can boot when the catalog index enables those modules.
  • Matches existing harness pattern (.invalid hosts / dummy credentials); email uses debug-only stream transport.

Test plan

  • plugin-sanity job reaches readiness 200 (no timeout on config.webServer)
  • Backend no longer logs missing apiPath / email.sender startup errors
  • Scheduled E2E Cluster-free on main recovers after merge

Generated-by: cursor

Ref: https://redhat.atlassian.net/browse/RHIDP-13884

Catalog index next enables these modules without required init
config, so cluster-free plugin-sanity stays on readiness 503.

https://redhat.atlassian.net/browse/RHIDP-13884

Generated-by: cursor
Co-authored-by: Cursor <cursoragent@cursor.com>
@nickboldt

Copy link
Copy Markdown
Member Author

this PR seeks to fix a known test failure that prevented me from merging #5226 (comment) until I was able to determine the test was NOT blocking my change.

@nickboldt
nickboldt requested a review from gustavolira August 11, 2026 17:38
@github-actions

Copy link
Copy Markdown
Contributor

The container image build workflow finished with status: cancelled.

@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.96%. Comparing base (f959fd9) to head (f0cf5c2).
⚠️ Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5242      +/-   ##
==========================================
- Coverage   63.69%   59.96%   -3.74%     
==========================================
  Files         123      111      -12     
  Lines        2424     2198     -226     
  Branches      577      546      -31     
==========================================
- Hits         1544     1318     -226     
  Misses        878      878              
  Partials        2        2              
Flag Coverage Δ
rhdh 59.96% <ø> (-3.74%) ⬇️
Components Coverage Δ
Backend plugins ∅ <ø> (∅)
Backend app 66.66% <ø> (ø)
Frontend app 58.89% <ø> (ø)
Plugin utils ∅ <ø> (∅)
Dynamic plugins utils ∅ <ø> (∅)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f959fd9...f0cf5c2. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@github-actions

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open 7 days with no activity. Remove stale label or comment or this will be closed in 21 days.

@github-actions github-actions Bot added Stale and removed Stale labels Aug 19, 2026
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.

2 participants