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 an end to end test that checks breadcrumbs #786

Merged
merged 3 commits into from
Sep 11, 2024

Conversation

MatMoore
Copy link
Contributor

@MatMoore MatMoore commented Sep 5, 2024

Currently we don't have any tests that run end to end using a real Datahub backend.

This PR adds one to test that breadcrumbs are rendered correctly. #674

Note: this introduces a CI dependency on the dev datahub instance, so it's possible that we could break that environment and block CI. However, CI is independent of the dev Find MoJ data instance.

As there is no test isolation, the test makes some assumptions about what data is populated:

  • there is a subject area for Bold
  • there is at least one table in it (from CaDeT)
  • the CaDeT ingestion ran as expected, and populated database entities

@MatMoore MatMoore force-pushed the fmd-674-end-to-end-test branch from 6fa271c to 9f5b009 Compare September 5, 2024 10:26
These can be used for end to end tests as well as Find MoJ data tests.
@MatMoore MatMoore force-pushed the fmd-674-end-to-end-test branch from 9f5b009 to 3c75520 Compare September 5, 2024 10:35
this requires github actions to be able to access the dev datahub
instance.
@MatMoore MatMoore force-pushed the fmd-674-end-to-end-test branch from 3c75520 to cd473b4 Compare September 5, 2024 10:43
@MatMoore MatMoore marked this pull request as ready for review September 5, 2024 10:46
@MatMoore MatMoore requested a review from a team as a code owner September 5, 2024 10:46
@MatMoore MatMoore changed the title Add an end to end test Add an end to end test that checks breadcrumbs Sep 5, 2024
murdo-moj
murdo-moj previously approved these changes Sep 10, 2024
@MatMoore MatMoore force-pushed the fmd-674-end-to-end-test branch from fdee59a to c53fc74 Compare September 11, 2024 11:04
@MatMoore MatMoore merged commit e79ff16 into main Sep 11, 2024
8 checks passed
@MatMoore MatMoore deleted the fmd-674-end-to-end-test branch September 11, 2024 13:02
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