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

[IO-1331][internal] Introduce E2E tests module #631

Merged
merged 2 commits into from
Jul 13, 2023
Merged

[IO-1331][internal] Introduce E2E tests module #631

merged 2 commits into from
Jul 13, 2023

Conversation

owencjones
Copy link
Contributor

Problem

No provision for E2E tests in darwin-py

Solution

Added configured pytest module.

Changelog

Added pytest module
Added pytest.ini fule
Added minor docs.

@linear
Copy link

linear bot commented Jul 11, 2023

IO-1331 Create E2E testing module in darwin-py

Create a simple, largely empty module, setup not to run when pytest is normally run, using pytest config. Include an example test which doesn't actually test anything (this can be used for checking it runs or not)

Acceptance criteria

  • There is an E2E module in darwin-py
  • It's a fully functioning pytest set
  • It's not to run unless directed to separately (e.g. pytest command at command line doesn't run this direction, but something like pytest e2e does,

@@ -0,0 +1,2 @@
[pytest]
addopts = --ignore=e2e_tests
Copy link
Contributor

Choose a reason for hiding this comment

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

So the intention here is that it'll only be run via Github Actions (or if you want to manually test)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Exactly, yeah

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You can run pytest e2e_tests and it will run them, but still run pytest on your local machine

@owencjones owencjones merged commit 547ca51 into master Jul 13, 2023
@owencjones owencjones deleted the io-1331 branch July 13, 2023 13:18
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