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 test for Minion dashboard #5571

Merged
merged 2 commits into from Apr 11, 2024
Merged

Conversation

Martchus
Copy link
Contributor

Just check whether the dashboard is accessible from the menu and do a small
smoke test. This should already catch issues caused by severe JavaScript
errors like https://progress.opensuse.org/issues/158805.

@Martchus
Copy link
Contributor Author

Considering the test failure we maybe need to merge #5572 first.

@okurz
Copy link
Member

okurz commented Apr 11, 2024

@Mergifyio rebase

* Use our current coding style for quoting
* Avoid useless comments
Just check whether the dashboard is accessible from the menu and do a small
smoke test. This should already catch issues caused by severe JavaScript
errors like https://progress.opensuse.org/issues/158805.
Copy link
Contributor

mergify bot commented Apr 11, 2024

rebase

✅ Branch has been successfully rebased

Copy link

codecov bot commented Apr 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.39%. Comparing base (00e78c3) to head (48cda39).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5571   +/-   ##
=======================================
  Coverage   98.39%   98.39%           
=======================================
  Files         392      392           
  Lines       38234    38244   +10     
=======================================
+ Hits        37621    37631   +10     
  Misses        613      613           

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

@mergify mergify bot merged commit 42c8473 into os-autoinst:master Apr 11, 2024
42 checks passed
@Martchus Martchus deleted the minion-test branch April 11, 2024 10:36
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

4 participants