fix(e2e): add the missing assertion [ci-skip] [skip build]

[josephca]

Description

Add a missing assertion to fix renovate PR - more strict Lint checkup is added by the updated version.

E2E Tests / TSC, ESLint, ShellCheck and Prettier (pull_request)

Which issue(s) does this PR fix

• Fixes chore(deps): update linters (non-major) #4338

PR acceptance criteria

Please make sure that the following steps are complete:

• GitHub Actions are completed and successful
• Unit Tests are updated and passing
• E2E Tests are updated and passing
• Documentation is updated if necessary (requirement for new features)
• Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

[github-actions]

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

• quay.io/rhdh-community/rhdh:pr-4724
• quay.io/rhdh-community/rhdh:pr-4724-b3de3a6c

[gustavolira]

/lgtm

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.56%. Comparing base (d326407) to head (b3de3a6).

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #4724       +/-   ##
===========================================
+ Coverage   40.98%   69.56%   +28.58%     
===========================================
  Files         119      109       -10     
  Lines        2223     4705     +2482     
  Branches      537      511       -26     
===========================================
+ Hits          911     3273     +2362     
- Misses       1307     1432      +125     
+ Partials        5        0        -5     

Flag	Coverage Δ
install-dynamic-plugins	92.44% <ø> (?)
rhdh	38.75% <ø> (-2.23%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d326407...b3de3a6. 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.

[openshift-ci]

New changes are detected. LGTM label has been removed.

[github-actions]

The container image build and publish workflows were skipped (either due to [skip-build] tag or no relevant changes with existing image).

[github-actions]

The container image build and publish workflows were skipped (either due to [skip-build] tag or no relevant changes with existing image).

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[josephca]

/test e2e-ocp-helm

[josephca]

/test e2e-ocp-helm

[openshift-ci]

@josephca: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-ocp-helm	31344aa	link	true	/test e2e-ocp-helm

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[github-actions]

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.