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

Experiment/create ephemeral UI test accts #1898

Draft
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

andrewleith
Copy link
Member

Summary | Résumé

This is an experiment to use ephemeral test account aliases in order to increase concurrency potential for UI tests. It still uses a single gmail inbox, but now can create an unlimited amount of aliases on demand so that if tests are running in parallel they won't try to login to the same account in the same environment.

Test instructions | Instructions pour tester la modification

Sequential steps (1., 2., 3., ...) that describe how to test this change. This
will help a developer test things out without too much detective work. Also,
include any environmental setup steps that aren't in the normal README steps
and/or any time-based elements that this requires.


Étapes consécutives (1., 2., 3., …) qui décrivent la façon de tester la
modification. Elles aideront les développeurs à faire des tests sans avoir à
jouer au détective. Veuillez aussi inclure toutes les étapes de configuration
de l’environnement qui ne font pas partie des étapes normales dans le fichier
README et tout élément temporel requis.

tests_cypress/cypress/Notify/NotifyAPI.js Fixed Show fixed Hide fixed
@@ -3,6 +3,7 @@
import config from "../../config";

import LoginPage from "../Notify/Admin/Pages/LoginPage";
import { Admin } from "../Notify/NotifyAPI";

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note test

Unused import Admin.
Copy link

Copy link

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