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-1334][internal] E2E Testing config #636

Merged
merged 3 commits into from
Jul 17, 2023
Merged

[IO-1334][internal] E2E Testing config #636

merged 3 commits into from
Jul 17, 2023

Conversation

owencjones
Copy link
Contributor

Problem

E2E Tests had no config solution

Solution

Added a conftest.py file that incoporated environment variables, with optional dotenv for use on local machines. Also added a fixture that is global, and session bound (runs once per test session, not once per test) that allows for these values to be included in tests easily. Have also added custom E2E exceptions that the config code can raise so that if our system raises an error we foresee in config, we can easily spot these in CI.

NB: Have deliberately not included the staging URL in the defaults, as it would go into our public code if I did, and possibly shouldn't.

Changelog

e2e_tests/conftest.py
e2e_tests/exceptions.py
e2e_tests/pytest.ini
e2e_tests/test_example.py

  • Added test configuration file with hook to accept and parse config, and fixture to share it
  • Added custom exceptions for E2E tests
  • Added to example test to offer green that this works.

@owencjones owencjones changed the title Io 1334 [IO-1334][internal] E2E Testing config Jul 17, 2023
@linear
Copy link

linear bot commented Jul 17, 2023

IO-1334 Test setup configuration

Setup the new E2E test system to be easily pointed at any server we have, and configure it to point at staging.darwin.com for now.

@owencjones owencjones merged commit c6f7e7a into master Jul 17, 2023
@Nathanjp91 Nathanjp91 deleted the io_1334 branch November 8, 2023 10:42
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