GH-3852: Fixing Fuseki UI test:e2e:win32#3853
Conversation
|
The change breaks (doesn't run tests) for the windows github workflow that admittedly uses bash for starting it. I think it fails during startup, and then skips all the tests. By default the github-runner run action on windows uses powershell if a shell isn't defined, and you can chose between powershell, cmd and bash, when configuring run actions (while now it currently runs bash -c "command" from powershell.) I think I tried getting it to run with cmd first, and then ended up with using bash, because the solution that worked locally in cmd didn't work because the options for it (SET) are already defined. See https://github.com/actions/runner-images/blob/main/images/windows/Windows2025-Readme.md for reference on the runner. Which of powershell or cmd should the change work for? I did a quick check on deleting "bash -c" and running it on powershell by default But that also had the same issue of the tests not running in GH-action. Trying to use |
@OyvindLGjesdal Thanks, for your explanation. Unfortunately I cannot reproduce the GitLabRunner behaviour on my machine. My IDE should also use bash...
|
|
I get a for me not so helpful error message: (edit, I think it just is missing 33 from test:e2e:win -> test:e2e:win32, running again) not sure if the action is browsable: https://github.com/OyvindLGjesdal/jena/actions/runs/24278345266/job/70896002093 from a local copy of your branch https://github.com/OyvindLGjesdal/jena/tree/GH-3852-fixing-fuseki-ui-test_e2e_win32 |
|
Yes, the updated e2e from you runs the action and tests successfully 👍 It also works without updating the github workflow windows file, so this change should be enough. |
c5efce5 to
9ab9871
Compare
Fixed "test:e2e:win32" in jena-fuseki2/jena-fuseki-ui/package.json:
SET /a (FUSEKI_PORT=FUSEKI_PORT ^ 3030)
--> produced: missing brackets
${FUSEKI_PORT}
--> is unix syntax, not windows
Switched from cmd to powershell
9ab9871 to
5b923dc
Compare
GitHub issue resolved #3852
Pull request Description:
Fixed "test:e2e:win32" in jena-fuseki2/jena-fuseki-ui/package.json:
SET /a (FUSEKI_PORT=FUSEKI_PORT ^ 3030)--> produced: missing brackets
${FUSEKI_PORT}--> is unix syntax, not windows
Same with
PORT.By submitting this pull request, I acknowledge that I am making a contribution to the Apache Software Foundation under the terms and conditions of the Contributor's Agreement.
See the Apache Jena "Contributing" guide.