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

chore: migrate src/__tests__ to test #1156

Merged
merged 3 commits into from
Oct 20, 2021
Merged

chore: migrate src/__tests__ to test #1156

merged 3 commits into from
Oct 20, 2021

Conversation

eladb
Copy link
Contributor

@eladb eladb commented Oct 20, 2021

Move all test code from src/__tests__ to test. This means that now tests are executed without a separate compilation step.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Elad Ben-Israel added 2 commits October 20, 2021 10:06
Move all test code from `src/__tests__` to `test`. This means that now tests are executed without a separate compilation step.
@eladb eladb requested review from Chriscbr and iliapolo October 20, 2021 07:10
@mergify mergify bot added the contribution/core ⚙️ used by automation label Oct 20, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #1156 (8a23c6c) into main (377966e) will increase coverage by 6.66%.
The diff coverage is 85.31%.

❗ Current head 8a23c6c differs from pull request most recent head a2f9edc. Consider uploading reports for the commit a2f9edc to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1156      +/-   ##
==========================================
+ Coverage   78.46%   85.13%   +6.66%     
==========================================
  Files          88      114      +26     
  Lines        3181     4439    +1258     
  Branches      809     1185     +376     
==========================================
+ Hits         2496     3779    +1283     
+ Misses        683      660      -23     
+ Partials        2        0       -2     
Impacted Files Coverage Δ
src/clobber.ts 100.00% <ø> (ø)
src/github/index.ts 100.00% <ø> (ø)
src/java/junit.ts 100.00% <ø> (ø)
src/java/maven-compile.ts 100.00% <ø> (ø)
src/java/maven-packaging.ts 100.00% <ø> (ø)
src/java/pom.ts 93.84% <ø> (ø)
src/object-file.ts 100.00% <ø> (+2.38%) ⬆️
src/python/pytest.ts 100.00% <ø> (ø)
src/python/setuptools.ts 100.00% <ø> (ø)
src/typescript-typedoc.ts 20.00% <0.00%> (-13.34%) ⬇️
... and 107 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 63f89f0...a2f9edc. Read the comment docs.

@mergify mergify bot merged commit 160df80 into main Oct 20, 2021
@mergify mergify bot deleted the benisrae/src-test branch October 20, 2021 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core ⚙️ used by automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants