ASSERT-KTH / spork

AST-based structured merge tool for Java, fully Git compatible https://doi.org/10.1109/TSE.2022.3143766
MIT License
49 stars 6 forks source link

graalvm: Run native tests #511

Open wetneb opened 5 months ago

wetneb commented 5 months ago

This makes it possible to run tests via native image with mvn -P native test.

What I find interesting is that it does show some failing tests, but not the same ones as the inputs where @slarse noticed that the outputs were different in https://github.com/ASSERT-KTH/spork/pull/481#issuecomment-1913688408.

For instance, on my machine the method_visibility_left_empty test passes in mvn -P native test, but the output of the native binary is different from the expected one. I have strictly no idea why!

codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.25%. Comparing base (ba76361) to head (3b8b85b). Report is 5 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #511 +/- ## ============================================ - Coverage 82.70% 82.25% -0.45% Complexity 363 363 ============================================ Files 43 43 Lines 1769 1719 -50 Branches 303 303 ============================================ - Hits 1463 1414 -49 + Misses 180 179 -1 Partials 126 126 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

slarse commented 5 months ago

Hi @wetneb,

I'm going to have to get back to you on this one. My main machine has a malfunctioning thermal sensor and keeps shutting down, and my backup doesn't have GraalVM configured (and probably not enough RAM to build the native image anyway).

Wanna try this out before a merge :)

slarse commented 3 months ago

I've finally gotten my main development machine back in working order. It needed two rounds of repairs because the first repair broke another thing :s

In any case, I will revisit this during the weekend and hopefully get it merged. Sorry about the wait.