uPortal-Project / uPortal-start

CLI tools for implementing uPortal, beginning with version 5.0.0
Apache License 2.0
19 stars 53 forks source link

Include playwrightRun in uPortal-start CI #574

Open cbeach47 opened 2 years ago

cbeach47 commented 2 years ago

Is your feature request related to a problem? Please describe. Currently, the CI flow only initializes and starts uPortal, but doesn't access the portal.

Describe the solution you'd like Have the playwright scripts be run as part of the CI tests, to provide greater confidence the PR will not introduce regressions.

Describe alternatives you've considered playwrightRun can be launched locally, and for now, is an alright workaround, but it's another manual step on a single environment.

Additional context This was attempted in the initial Playwright PR (see below), however, only in the CI environment, the login flow failed to log the user in. There was a single occurrence of this behavior locally, but subsequent attempts have not been able to replicate the issue.

History in https://github.com/uPortal-Project/uPortal-start/pull/571 .

cbeach47 commented 2 years ago

Checking with the community if folks have experienced this kind of issue before - https://groups.google.com/u/1/a/apereo.org/g/uportal-dev/c/uk1Prh00hew