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.

Sign up for GitHub

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

Jump to bottom

chore: rename "integration-tests" to "e2e" #6315

Merged
merged 4 commits into from
May 27, 2018
Merged

chore: rename "integration-tests" to "e2e" #6315

merged 4 commits into from
May 27, 2018

Conversation

thymikee
Copy link
Collaborator

Summary

Renaming integration-tests to e2e. Life's so much simpler now :D

cc @captbaritone

Test plan

CI green

@codecov-io
Copy link

codecov-io commented May 27, 2018 •
edited
Loading

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6315   +/-   ##
=======================================
  Coverage   63.64%   63.64%           
=======================================
  Files         227      227           
  Lines        8638     8638           
  Branches        3        3           
=======================================
  Hits         5498     5498           
  Misses       3139     3139           
  Partials        1        1
Impacted Files Coverage Δ
...bel-plugin-jest-hoist/__test_modules__/Unmocked.js 100% <ø> (ø)
...e/coverage-report/cached-duplicates/b/Identical.js 100% <ø> (ø)
e2e/coverage-transform-instrumented/covered.js 100% <ø> (ø)
e2e/babel-plugin-jest-hoist/__test_modules__/b.js 0% <ø> (ø)
e2e/babel-plugin-jest-hoist/mock-file.js 100% <ø> (ø)
...e/coverage-report/cached-duplicates/a/Identical.js 100% <ø> (ø)
e2e/coverage-remapping/covered.ts 85.71% <ø> (ø)
e2e/babel-plugin-jest-hoist/__test_modules__/c.js 100% <ø> (ø)
...babel-plugin-jest-hoist/__test_modules__/Mocked.js 0% <ø> (ø)
e2e/babel-plugin-jest-hoist/__test_modules__/a.js 0% <ø> (ø)
... and 5 more

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 a43fd6c...3a1c710. Read the comment docs.

SimenB
SimenB approved these changes May 27, 2018
View reviewed changes
Copy link
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

Since CI's happy, I'm happy

rickhanlonii
rickhanlonii approved these changes May 27, 2018
View reviewed changes
Copy link
Member

@rickhanlonii rickhanlonii left a comment

Choose a reason for hiding this comment

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

Beauty

@captbaritone
Copy link
Contributor

Love it!

@SimenB
Copy link
Member

SimenB commented May 27, 2018

@thymikee conflicts 😅

@thymikee
Merge remote-tracking branch 'upstream/master' into feat/e2e
96710d7 
* upstream/master:
  normalize icons and slashes across OSes (jestjs#6310)
  fix: toMatchObject throws TypeError when a source property is null (jestjs#6313)
@thymikee thymikee mentioned this pull request May 27, 2018
Merged
@thymikee thymikee changed the title chore: rename integration-tests -> e2e chore: rename "integration-tests" to "e2e" May 27, 2018
@SimenB SimenB merged commit 9310e9c into jestjs:master May 27, 2018
This was referenced May 31, 2018
Merged
Open
Merged
Open
Merged
Open
Merged
Merged
@dependencies dependencies bot mentioned this pull request Nov 1, 2018
Closed
@thymikee thymikee deleted the feat/e2e branch March 16, 2019 10:56
@snyk-bot snyk-bot mentioned this pull request Mar 17, 2020
Merged
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mattphillips mattphillips mattphillips approved these changes

@rickhanlonii rickhanlonii rickhanlonii approved these changes

@SimenB SimenB SimenB approved these changes

Assignees
No one assigned
Labels
cla signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@thymikee @codecov-io @captbaritone @SimenB @rickhanlonii @mattphillips @facebook-github-bot