Closed cyrille-artho closed 4 months ago
@cyrille-artho Can I work on this?
Yes, please do so. It is good to work on this first, as it will make testing and validating any subsequent PRs easier.
@cyrille-artho I have written YAML file for workflow. On my local machine, all the tests are working well and passing, but while running the workflow all tests are failing. Can you please tell what can be an error?
Getting same error for every test : on_the_fly.StringTest > testToLowerCase FAILED java.lang.AssertionError at StringTest.java:115 on_the_fly.StringTest > testGetBytes FAILED java.lang.AssertionError at StringTest.java:95 on_the_fly.StringTest > testSplit FAILED
build ouput: https://github.com/Harsh4902/jpf-nhandler/actions/runs/8508077814/job/23301052281
I saw two potential problems:
touch $HOME/.jpf/site.propeties;
, but the command is not needed because you use >
in the next line.jpf-core
has to be "side by side" with jpf-nhandler
. You clone jpf-core
inside the project rather than in its parent directory.Okay, I am looking into it. Thanks.
It unfortunately currently fails on the GitHub CI.
Should I revert it?
No need. It's strange that the build status in your repo was green and it fails on this repo, though.
Yeah, that blown my mind also.
The first warning in the build shows multiple extensions being set in ~/.jpf/site.properties
:
WARNING: in the site.properties file, the property "extensions" has
more than one value. However, it suffices to use
extensions = ${jpf-core}
You use jpf-numeric and jpf-apropos in addition to nhandler. I think these are indeed not needed; my own ~/.jpf/site.properties
includes only jpf-core.
In fact, if you only want to develop/test nhandler but not use it otherwise, you don't even have to add nhandler itself to site.properties
. I think a simple one will do:
mkdir $HOME/.jpf;
echo "jpf-core = ${HOME}/work/jpf-nhandler/jpf-core" >> $HOME/.jpf/site.properties;
echo "extensions=${jpf-core}" >> $HOME/.jpf/site.properties;
Note that you're supposed to use the variable for the extension and not its value, although I don't think that causes a problem per se. Try the new, more lightweight, configuration and see what happens. If this does not fix the problem, at least it removes possible causes.
https://github.com/javapathfinder/jpf-nhandler/pull/10 Hopefully this PR will work
It passes now, thanks! I assume that the extra extension inclusions caused problems because the extensions were not available (even though they were not needed either).
It passes now, thanks! I assume that the extra extension inclusions caused problems because the extensions were not available (even though they were not needed either).
Yeah, possibly.
Even though this project gets only infrequent updates, a CI build/test (as a GitHub action) should be configured. The action in
jpf-core
can be used as a template.