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

[Backport nixos-24.05] docs/ioc: expand Integration tests tutorial #137

Merged
merged 8 commits into from
Aug 2, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 2, 2024

Bot-based backport to nixos-24.05, triggered by a label in #134.

Because the test is added by default, and run by default in GitLab CI,
this means that by default the test will fail.

With this change, the test fails in 10 seconds instead of 15 minutes,
leaving more time for other jobs.

(cherry picked from commit 02fee06)
The simulator was rewritten in Lewis, which changed its default port to
9999, and is now case-sensitive.

(cherry picked from commit a5f45fa)
Renamed to "Adding integration tests to your IOC", because an
"Integration tests" user guide is planned, and to make it a bit clearer
that it is a continuation of the StreamDevice tutorial.

Previously the tutorial was a mix between a guide and tutorial, not
really guided enough.

Now the tutorial builds upon the StreamDevice tutorial, and shows a
complete path to have a fully functional integration test.

Also introduce how to *run* tests, both non-interactively and
interactively.

Added a quick reference of useful Python test functions.

The tutorial was also updated due to some changes in the default
template, and some renamed options.

(cherry picked from commit b46224c)
Nix will show an error running tests if KVM is not present.

(cherry picked from commit a0e3f8a)
@minijackson minijackson merged commit 010c9be into nixos-24.05 Aug 2, 2024
@minijackson minijackson deleted the backport/134-to-nixos-24.05 branch August 2, 2024 14:13
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.

1 participant