Skip to content
This repository has been archived by the owner on Sep 4, 2023. It is now read-only.

Introducing end-to-end tests #106

Merged
merged 6 commits into from
Feb 17, 2022
Merged

Introducing end-to-end tests #106

merged 6 commits into from
Feb 17, 2022

Conversation

andrenatal
Copy link
Contributor

Fixes #49

This basically clones latest m-c, integrates the extension and the mochitest files into it, run them and report if there's failures. @abhi-agg Take a look at the mochistest to see what's being tested. I believe that's enough.

We want to test every PR and push to main, but also every night, so we could be warned of any introduced breakage from Nightly.

@andrenatal andrenatal requested a review from abhi-agg February 16, 2022 10:45
@andrenatal
Copy link
Contributor Author

There's a few run examples here https://github.com/andrenatal/firefox-translations/actions, both for failures as for succeeded ones.

Comment on lines 19 to 22
/*
* the infobar is not triggered first time the page is loaded due some caching
* so we need to reload the tab in order to have it summoned
*/
Copy link
Collaborator

Choose a reason for hiding this comment

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

the infobar is not triggered first time the page is loaded due some caching

Is this a bug that we need to resolve later?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This happens only in the mochistest framework. We fixed something similar in the production xpi here #69 though

@andrenatal andrenatal requested a review from abhi-agg February 16, 2022 17:37
@andrenatal andrenatal changed the title Introducing end-to-end tests [WIP] Introducing end-to-end tests Feb 16, 2022
@andrenatal
Copy link
Contributor Author

I'll add tests for outbound-translations

@andrenatal andrenatal changed the title [WIP] Introducing end-to-end tests Introducing end-to-end tests Feb 17, 2022
@andrenatal
Copy link
Contributor Author

Ok, this contains tests for outbound translations already. @abhi-agg r?

Copy link
Collaborator

@abhi-agg abhi-agg left a comment

Choose a reason for hiding this comment

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

Awesome work @andrenatal 👍🏾

@andrenatal andrenatal merged commit 3821e90 into mozilla:main Feb 17, 2022
@andrenatal
Copy link
Contributor Author

Thanks @abhi-agg

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorporate end-to-end integration tests
2 participants