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

Switch to firefox dev edition #5779

Merged
merged 2 commits into from Jul 30, 2018
Merged

Switch to firefox dev edition #5779

merged 2 commits into from Jul 30, 2018

Conversation

willdurand
Copy link
Member

@willdurand willdurand commented Jul 30, 2018

Refs mozilla/addons#12140


This PR changes the Firefox version from nightly to dev edition in order
to get a green feedback again. The issue is related to a "menu" button
not being found anymore. All FF versions < 63 are not affected.

I added a job for nightly so that we can keep an eye on it, but this job
won't fail the build at least.

@willdurand willdurand force-pushed the travis-ff-dev branch 2 times, most recently from ffbca94 to 8199876 Compare July 30, 2018 14:26
@codecov-io
Copy link

codecov-io commented Jul 30, 2018

Codecov Report

Merging #5779 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    mozilla/addons-frontend#5779   +/-   ##
=======================================
  Coverage   97.25%   97.25%           
=======================================
  Files         228      228           
  Lines        5678     5678           
  Branches     1093     1093           
=======================================
  Hits         5522     5522           
  Misses        136      136           
  Partials       20       20

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 26a264e...62ec7d0. Read the comment docs.

@willdurand willdurand requested a review from kumar303 July 30, 2018 14:37
Copy link
Contributor

@kumar303 kumar303 left a comment

Choose a reason for hiding this comment

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

I think we should make the change more temporary

.travis.yml Outdated
addons:
firefox: latest-nightly
hosts: example.com
env: ALLOWED_TO_FAIL=1 TOXENV=discopane-ui-tests GECKODRIVER=0.20.1 MOZ_HEADLESS=1 PATH=./node_modules/.bin:$PATH
Copy link
Contributor

Choose a reason for hiding this comment

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

If this is allowed to fail, no one will care about it. I don't think it will be helpful to add an additional stage here.

Copy link
Member Author

Choose a reason for hiding this comment

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

Well, I will care about it. Otherwise, how do we know if this gets fixed on Nightly?

Copy link
Contributor

Choose a reason for hiding this comment

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

Otherwise, how do we know if this gets fixed on Nightly?

The person looking into fixing the tests for Nightly would be running it locally.

But, sure, that's fine if you want to leave in a new stage temporarily while someone works on a fix.

.travis.yml Outdated
@@ -55,7 +55,7 @@ jobs:
- stage:
<<: *tox
addons:
firefox: latest-nightly
firefox: latest-dev
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 is fine as a temporary fix. I would like to see latest-nightly commented out with a link to a bug for re-enabling it. We should unblock master then investigate why it's failing on Nightly and try to get back on Nightly.

Copy link
Contributor

@kumar303 kumar303 left a comment

Choose a reason for hiding this comment

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

r+wc

If the new stage is a temporary thing that we will remove when we fix the tests for Nightly then I'm fine with leaving it in. I just didn't want it to be a permanent solution.

.travis.yml Outdated
@@ -55,7 +55,7 @@ jobs:
- stage:
<<: *tox
addons:
firefox: latest-nightly
firefox: latest-dev
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a bug on file for putting the tests back on Nightly? I'd like to see a comment here with that bug link.

Copy link
Member Author

Choose a reason for hiding this comment

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

@willdurand
Copy link
Member Author

There is a new issue on master but unrelated to this patch, merging...

@willdurand willdurand merged commit 0e2aaa0 into master Jul 30, 2018
@willdurand willdurand deleted the travis-ff-dev branch July 30, 2018 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants