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

For #6208 - Sync int tests: fix path for showing the html report file #6209

Merged

Conversation

isabelrios
Copy link
Contributor


Pull Request checklist

  • Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)

Copy link

@nojunpark nojunpark left a comment

Choose a reason for hiding this comment

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

Hmm, test failed because it was inconclusive. This is probably the test lab error. Perhaps rerunning it might fix this.

@isabelrios
Copy link
Contributor Author

Hmm, test failed because it was inconclusive. This is probably the test lab error. Perhaps rerunning it might fix this.

oh this should not affect the ui tests, I'm re-running them, hopefully they will be green again

@isabelrios isabelrios changed the title For #6208 - Sync Int Tests fix path for showing the html report file For #6208 - Sync int tests: fix path for showing the html report file Oct 23, 2019
@codecov-io
Copy link

codecov-io commented Oct 23, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #6209   +/-   ##
=========================================
  Coverage     14.69%   14.69%           
  Complexity      319      319           
=========================================
  Files           262      262           
  Lines         10723    10723           
  Branches       1558     1558           
=========================================
  Hits           1576     1576           
  Misses         9021     9021           
  Partials        126      126

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 885e816...238e95e. Read the comment docs.

@sblatz sblatz merged commit 210e358 into mozilla-mobile:master Oct 23, 2019
mcarare pushed a commit to mcarare/fenix that referenced this pull request Oct 29, 2019
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.

None yet

4 participants