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

[Test] Setup autotest infrastructure #23

Merged
merged 21 commits into from
Jan 9, 2025

Conversation

Fabricio-ESP
Copy link
Collaborator

@Fabricio-ESP Fabricio-ESP commented Dec 9, 2024

Description

This PR adds the automated tests infrastructure of the EIM GUI.
The initial test only launches the application and checks the application is running and the version printed on the page footer.

2 Additional test worflow are added:
Express installation (single click test automation process)
Expert Installation (Step by step flow using basic settings)

TODO:
Setup CI workflow

Related

Testing


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

Copy link

github-actions bot commented Dec 20, 2024

Test Results

 6 files  12 suites   20s ⏱️
 5 tests  3 ✅ 0 💤 2 ❌
38 runs  34 ✅ 0 💤 4 ❌

For more details on these failures, see this check.

Results for commit 800c460.

♻️ This comment has been updated with latest results.

@Fabricio-ESP Fabricio-ESP force-pushed the EIM-61-autotest-infrastructure branch from 242863c to 800c460 Compare January 6, 2025 09:32
Copy link
Collaborator

@Hahihula Hahihula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As an initial version of the tests LGTM. It would be nice to read the version from cargo.toml and not have it hardcoded, but that's nitpicking.
Also please squash the commits on merge time.

@Fabricio-ESP Fabricio-ESP merged commit ee1160c into master Jan 9, 2025
8 of 10 checks passed
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