Skip to content

Conversation

@sebastian-aranda
Copy link
Contributor

This PR add playwright configurations to run e2e (end-to-end) testing. These changes are particularly focused on stress testing to be able to assess on load into our different services by running different cases. Documentation about how to run tests is added as well.

@sebastian-aranda sebastian-aranda force-pushed the tickets/OSW-943 branch 12 times, most recently from 3b7c566 to f057e79 Compare December 17, 2025 20:28
@sebastian-aranda sebastian-aranda force-pushed the tickets/OSW-943 branch 5 times, most recently from f6377e1 to 2f886c0 Compare December 18, 2025 20:55
Copy link
Contributor

@Vebop Vebop left a comment

Choose a reason for hiding this comment

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

This looks good to me but I would prefer if Eman has time to review as well since she started looking into this before! Thanks for setting this all up !

Copy link
Contributor

@emanehab99 emanehab99 left a comment

Choose a reason for hiding this comment

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

Nice work, @sebastian-aranda! I liked the gif/mp4 demo for getting the token. The docs is very comprehensive too.
There is only one typo to fix.

That would be a good base to extend our testing suite and run it in the CI. Maybe then we might need to move these tests to a separate directory as these won't run on the CI.

/* global process */
import { test, expect } from "@playwright/test";

const NIGHTLY_DIGEST_URL = "https://base-lsp.lsst.codes/nightlydigest";
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should point to usdf-dev?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the catch! Regarding moving the folder I added an exception parameter to dismiss these tests in the CI jobs 😉 .

@sebastian-aranda sebastian-aranda force-pushed the tickets/OSW-943 branch 2 times, most recently from 4fc0009 to 7a40d4c Compare December 30, 2025 16:48
@sebastian-aranda sebastian-aranda merged commit f643cca into develop Dec 30, 2025
3 checks passed
@sebastian-aranda sebastian-aranda deleted the tickets/OSW-943 branch December 30, 2025 17:30
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.

4 participants