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 basic e2e testing for i18n #3461

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Conversation

tdonohue
Copy link
Member

@tdonohue tdonohue commented Oct 16, 2024

Description

In testing the many dependencies related PRs from dependabot, I've realized that a common test that I perform manually (in some scenarios) is to simply check if i18n translations are still working (by using the globe in the UI header)

So, rather than do that manually, here's an e2e test that verifies the basics by checking that we can switch from English to German and back (and that everything works).

Instructions for Reviewers

  • This only involves e2e tests, so if the tests succeed, then this can be merged.

@tdonohue tdonohue added i18n / l10n Internationalisation and localisation, related to message catalogs 1 APPROVAL pull request only requires a single approval to merge testing framework Related specifically to Unit or Integration (e2e) Tests port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release labels Oct 16, 2024
@tdonohue tdonohue added this to the 9.0 milestone Oct 16, 2024
@tdonohue
Copy link
Member Author

Merging immediately as this is a test that helps with dependabot PRs (as I've seen some dependency updates impact i18n in odd ways in the past)

@tdonohue tdonohue merged commit 34b9124 into DSpace:main Oct 17, 2024
13 checks passed
@dspace-bot
Copy link
Contributor

Backport failed for dspace-7_x, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin dspace-7_x
git worktree add -d .worktree/backport-3461-to-dspace-7_x origin/dspace-7_x
cd .worktree/backport-3461-to-dspace-7_x
git switch --create backport-3461-to-dspace-7_x
git cherry-pick -x a74cd848feeabe380026f504f0e70f6de134fd65

@dspace-bot
Copy link
Contributor

Successfully created backport PR for dspace-8_x:

@tdonohue tdonohue removed port to dspace-7_x This PR needs to be ported to `dspace-7_x` branch for next bug-fix release port to dspace-8_x This PR needs to be ported to `dspace-8_x` branch for next bug-fix release labels Oct 17, 2024
@tdonohue
Copy link
Member Author

Ported to 7.x in #3468

@tdonohue tdonohue deleted the i18n_e2e_test branch October 17, 2024 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge i18n / l10n Internationalisation and localisation, related to message catalogs testing framework Related specifically to Unit or Integration (e2e) Tests
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants