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

update sample with mocha test runner to jebtrains' new gradle plugin #690

Merged
merged 7 commits into from
Nov 12, 2020

Conversation

robstoll
Copy link
Owner


I confirm that I have read the Contributor Agreements v1.0, agree to be bound on them and confirm that my contribution is compliant.

@robstoll robstoll changed the title New kotlin js plugin update sample with mocha test runner to jebtrains' new gradle plugin Nov 10, 2020
@codecov
Copy link

codecov bot commented Nov 10, 2020

Codecov Report

Merging #690 (607ef0d) into master (2727afc) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             master     #690    +/-   ##
==========================================
  Coverage     87.37%   87.37%            
- Complexity       16      121   +105     
==========================================
  Files           904      904            
  Lines          8161     8161            
  Branches        376      376            
==========================================
  Hits           7131     7131            
  Misses          953      953            
  Partials         77       77            
Flag Coverage Δ Complexity Δ
bbc 72.45% <ø> (ø) 0.00 <ø> (ø)
bc 56.94% <ø> (ø) 0.00 <ø> (ø)
current 58.54% <ø> (ø) 0.00 <ø> (ø)
current_windows 49.24% <ø> (ø) 0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 2727afc...607ef0d. Read the comment docs.

@robstoll robstoll linked an issue Nov 10, 2020 that may be closed by this pull request
@robstoll robstoll force-pushed the new-kotlin-js-plugin branch 2 times, most recently from c981b95 to 003c4b6 Compare November 10, 2020 22:13
chadmarchand and others added 6 commits November 11, 2020 22:53
We can further simplify it when we use the testSetup.kt file instead of
the gradle task. In this case we do not require commonjs any more and
we can also get rid of the NpxTask (as JetBrains' plugin seems to set
it up already).
It is very unlikely someone uses JS with jasmine as jetbrains has
setup everything with mocha
@robstoll robstoll force-pushed the new-kotlin-js-plugin branch 3 times, most recently from e0581ef to 31d3cc1 Compare November 11, 2020 22:27
because of a Windows Server 2019 issue (or at least github action issue)
@robstoll robstoll merged commit 19d7687 into master Nov 12, 2020
@robstoll robstoll deleted the new-kotlin-js-plugin branch November 12, 2020 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update js samples to use the new org.jetbrains.kotlin.js gradle plugin
2 participants