Skip to content

Conversation

@jsjiang
Copy link
Collaborator

@jsjiang jsjiang commented Jul 30, 2025

@sfisher Hi Scott,
Here are the major changes:

  • create a Docker image for running EZID UI and functional tests
  • define a GitHub workflow to deploy the test container and run EZID tests as a GitHub action

Added files:

  • .env: used to trigger GitHub action
  • ezid-tests.yml: define GitHub workflow
  • docker-compose.yml: define the seleniarm/standalone-chromium and EZID test containers
  • Dockerfile: to build a Docker image for EZID tests
  • ezid_ui_tests_docker.py: Docker version EZID UI tests derived from ezid_ui_tests.py. The major change was to use a remote Selenium server instead of a local installed one.
  • run_ezid_tests.sh: a wrapper to combine functional tests (defined in verify_ezid_status.py) and UI tests (defined in `ezid_ui_tests_docker.py" in one script

Please take a look and let me know if you have questions.

Thank you

Jing

@sfisher sfisher self-requested a review August 28, 2025 17:19
Copy link
Contributor

@sfisher sfisher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested out both doing docker compose manually with setting the environment variable and using the run_ezid_tests.sh script. Both worked well for me.

At first I had forgotten if this was to run tests against local code for checking functionality or to check what was on the servers, but then I remembered. It ran well for me and I tried both dev and stage.

This isn't running against pull requests to EZID, also, is it?

Feel free to merge and it seems useful!

@jsjiang
Copy link
Collaborator Author

jsjiang commented Aug 28, 2025

@sfisher Thank you Scott for reviewing and testing.
It is not running against pull requests to EZID. Update the .env file will trigger the test to run as a GitHub action in the ezid-ops-scripts repo.

Jing

@jsjiang jsjiang merged commit 1b41bd1 into main Aug 28, 2025
1 check passed
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.

3 participants