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

[JVM] Make 'make test' work again #4355

Merged
merged 1 commit into from
Dec 18, 2021
Merged

[JVM] Make 'make test' work again #4355

merged 1 commit into from
Dec 18, 2021

Conversation

usev6
Copy link
Contributor

@usev6 usev6 commented May 2, 2021

As described in #4191
make test didn't work for a long time on the JVM backend
and gives a lot of errors at the moment.

With the changes from this PR (and one fix in NQP, that will
arrive with the next bump) make test runs fine again.
That should help to notice regressions in a timely manner.

@usev6
Copy link
Contributor Author

usev6 commented Aug 22, 2021

Most of tests for Telemetry are passing now (#4354 has been resolved).

The tests for NativeCall (and some others) are still failing.

Since there is a conflict as well, I'll convert this to a draft, will take a closer look and probably force-push an updated version.

@usev6 usev6 marked this pull request as draft August 22, 2021 18:20
(Also, un-todo one test that passes on the JVM backend.)

The affected tests fail for a variety of reasons. The idea is to look
at the underlying problems one by one -- but with being able to run
'make test' while working on the fixes.
@usev6 usev6 marked this pull request as ready for review December 17, 2021 21:09
@usev6
Copy link
Contributor Author

usev6 commented Dec 17, 2021

With these changes I get a successful run of "gmake test":

$ ./rakudo-j -v
Welcome to Rakudo™ v2021.10-149-g0851a253a.
Implementing the Raku® Programming Language v6.d.
Built on JVM.
$ time TEST_JOBS=1 gmake test
[...]
All tests successful.
Files=106, Tests=1452, 2531 wallclock secs ( 0.60 usr  0.32 sys + 4592.73 cusr 238.69 csys = 4832.34 CPU)
Result: PASS

real    42m12.508s
user    76m36.009s
sys     3m59.211s

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.

2 participants