Skip to content

ScalaJS incompatible issue #101

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

Closed
giovannicandido opened this issue Mar 16, 2015 · 5 comments
Closed

ScalaJS incompatible issue #101

giovannicandido opened this issue Mar 16, 2015 · 5 comments

Comments

@giovannicandido
Copy link

In a project with both scala and scalajs the coverage broke tests. I think is because scalajs generate intermediate classes but not sure
Project based on https://github.com/vmunier/play-with-scalajs-example

Error:

Compiling 1 Scala source to /Users/giovanni/Projetos/atende/monografias/shared/.js/target/scala-2.11/classes...
[info] Compiling 1 Scala source to /Users/giovanni/Projetos/atende/monografias/shared/.jvm/target/scala-2.11/classes...
[info] Cleaning datadir [/Users/giovanni/Projetos/atende/monografias/shared/.jvm/target/scala-2.11/scoverage-data]
[info] Beginning coverage instrumentation
[info] Instrumentation completed [11 statements]
[info] Cleaning datadir [/Users/giovanni/Projetos/atende/monografias/shared/.js/target/scala-2.11/scoverage-data]
[info] Beginning coverage instrumentation
[info] Wrote instrumentation file [/Users/giovanni/Projetos/atende/monografias/shared/.jvm/target/scala-2.11/scoverage-data/scoverage.coverage.xml]
[info] Will write measurement data to [/Users/giovanni/Projetos/atende/monografias/shared/.jvm/target/scala-2.11/scoverage-data]
[info] Instrumentation completed [11 statements]
[info] Wrote instrumentation file [/Users/giovanni/Projetos/atende/monografias/shared/.js/target/scala-2.11/scoverage-data/scoverage.coverage.xml]
[info] Will write measurement data to [/Users/giovanni/Projetos/atende/monografias/shared/.js/target/scala-2.11/scoverage-data]
[info] Compiling 6 Scala sources and 1 Java source to /Users/giovanni/Projetos/atende/monografias/server/target/scala-2.11/classes...
[info] Compiling 5 Scala sources to /Users/giovanni/Projetos/atende/monografias/client/target/scala-2.11/classes...
[info] Cleaning datadir [/Users/giovanni/Projetos/atende/monografias/client/target/scala-2.11/scoverage-data]
[info] Beginning coverage instrumentation
[info] Instrumentation completed [26 statements]
[info] Wrote instrumentation file [/Users/giovanni/Projetos/atende/monografias/client/target/scala-2.11/scoverage-data/scoverage.coverage.xml]
[info] Will write measurement data to [/Users/giovanni/Projetos/atende/monografias/client/target/scala-2.11/scoverage-data]
[info] Cleaning datadir [/Users/giovanni/Projetos/atende/monografias/server/target/scala-2.11/scoverage-data]
[info] Beginning coverage instrumentation
[info] Instrumentation completed [342 statements]
[info] Wrote instrumentation file [/Users/giovanni/Projetos/atende/monografias/server/target/scala-2.11/scoverage-data/scoverage.coverage.xml]
[info] Will write measurement data to [/Users/giovanni/Projetos/atende/monografias/server/target/scala-2.11/scoverage-data]
[info] Fast optimizing /Users/giovanni/Projetos/atende/monografias/client/target/scala-2.11/monografias-fastopt.js
[error] Referring to non-existent class Lscoverage_Invoker$
[error]   called from LmonoJs_MonoApp$.main__V
[error]   called from sjs_js_JSApp$class.$$js$exported$meth$main__sjs_js_JSApp__O
[error]   called from LmonoJs_MonoApp$.$$js$exported$meth$main__O
[error]   called from LmonoJs_MonoApp$.main
[error]   exported to JavaScript with @JSExport
[error] involving instantiated classes:
[error]   LmonoJs_MonoApp$
[error] Referring to non-existent method Lscoverage_Invoker$.invoked__I__T__V
@VladUreche
Copy link

cc @sjrd, he might want to be aware of this.

@sjrd
Copy link

sjrd commented Apr 4, 2015

@ssidorenko is working on Scala.js support for scoverage.

@holograph
Copy link

Seconded, I've run into much the same problem with Accord (see Travis output).

I'll look for a way to disable scoverage on the Scala.js projects as an interim workaround.

@sjrd
Copy link

sjrd commented Jul 15, 2015

There's a PR at scoverage/scalac-scoverage-plugin#118 which was never commented on by the maintainers of scoverage ...

holograph added a commit to wix-incubator/accord that referenced this issue Jul 27, 2015
newca12 added a commit to newca12/ProvingGround that referenced this issue May 4, 2017
newca12 added a commit to newca12/ProvingGround that referenced this issue May 4, 2017
@ckipp01
Copy link
Member

ckipp01 commented Apr 28, 2021

Modern scoverage has support for scalajs. If you have any issues, please do re-report.

@ckipp01 ckipp01 closed this as completed Apr 28, 2021
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

No branches or pull requests

5 participants