Fix windows integration tests and reenable in CI

[ikappaki]

Hi,

can you please review patch to fix the MS-Windows integration tests, and re-enable them on CI.

Changes include:

• Escaping backslashes in Windows paths when embedding them in JSON.
• Using the Windows command prompt to launch the server.

I didn't think it was necessary to open an issue for this and update the changelog.

Thanks

• I added a entry in changelog under unreleased section.

[ericdallo]

That's awesome!! windows support help is very welcome! c/c @afucher

[ericdallo]

FYI @ikappaki, @afucher is working on some improvements of ECA for windows too, but I guess it affects only eca-vscode

[ikappaki]

The main run for the Windows integration tests failed while downloading dependencies. This is likely a glitch on the Windows runner when handling many downloads concurrently, although it returned a path access denied exception, which is unusual. Could you please restart the job and see if it passes?

Downloading: com/fasterxml/jackson/dataformat/jackson-dataformat-cbor/2.18.3/jackson-dataformat-cbor-2.18.3.pom from central
Execution error (AccessDeniedException) at sun.nio.fs.WindowsException/translateToIOException (WindowsException.java:89).
C:\Users\runneradmin\.m2\repository\org\ow2\asm\asm-bom\9.8\asm-bom-9.8.pom.8594740676423826778.tmp -> C:\Users\runneradmin\.m2\repository\org\ow2\asm\asm-bom\9.8\asm-bom-9.8.pom

Full report at:
C:\Users\RUNNER~1\AppData\Local\Temp\clojure-14743379622849296428.edn
----- Error --------------------------------------------------------------------
Type:     clojure.lang.ExceptionInfo
Message:  
Data:     {:proc #object[java.lang.ProcessImpl 0x2f32632c "Process[pid=6172, exitValue=1]"], :exit 1, :in #object[java.lang.ProcessBuilder$NullOutputStream 0x45ca2c7b "java.lang.ProcessBuilder$NullOutputStream@45ca2c7b"], :out #object[java.lang.ProcessBuilder$NullInputStream 0x4f32b04d "java.lang.ProcessBuilder$NullInputStream@4f32b04d"], :err #object[java.lang.ProcessBuilder$NullInputStream 0x4f32b04d "java.lang.ProcessBuilder$NullInputStream@4f32b04d"], :prev nil, :cmd ["C:\\hostedtoolcache\\windows\\Java_Adopt_jdk\\24.0.2-12\\x64\\bin\\java.exe" "-XX:-OmitStackTraceInFastThrow" "-Dclojure.basis=.cpcache\\B0C8293BB89CAFC37577281659075D43.basis" "-classpath" "resources;.;C:\\Users\\runneradmin\\.m2\\repository\\babashka\\fs\\0.5.22\\fs-0.5.22.jar;C:\\Users\\runneradmin\\.m2\\repository\\babashka\\process\\0.6.23\\process-0.6.23.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\github\\ericdallo\\deps-bin\\1.0.0\\deps-bin-1.0.0.jar;C:\\Users\\runneradmin\\.gitlibs\\libs\\io.github.clojure\\tools.build\\573711ea65f7d324a6a6cdd3ac20cb30bba47e48\\src\\main\\clojure;C:\\Users\\runneradmin\\.gitlibs\\libs\\io.github.clojure\\tools.build\\573711ea65f7d324a6a6cdd3ac20cb30bba47e48\\src\\main\\resources;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\clojure\\1.12.0\\clojure-1.12.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\slipset\\deps-deploy\\0.2.2\\deps-deploy-0.2.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\clj-zip-meta\\clj-zip-meta\\0.1.3\\clj-zip-meta-0.1.3.jar;C:\\Users\\runneradmin\\.m2\\repository\\de\\ubercode\\clostache\\clostache\\1.4.0\\clostache-1.4.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\me\\raynes\\fs\\1.4.6\\fs-1.4.6.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\logging\\log4j\\log4j-core\\2.17.2\\log4j-core-2.17.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\slf4j\\slf4j-simple\\1.7.36\\slf4j-simple-1.7.36.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.deps\\0.22.1492\\tools.deps-0.22.1492.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.namespace\\1.5.0\\tools.namespace-1.5.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\core.specs.alpha\\0.4.74\\core.specs.alpha-0.4.74.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\spec.alpha\\0.5.238\\spec.alpha-0.5.238.jar;C:\\Users\\runneradmin\\.m2\\repository\\clj-commons\\pomegranate\\1.2.23\\pomegranate-1.2.23.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-settings\\3.9.4\\maven-settings-3.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-settings-builder\\3.9.4\\maven-settings-builder-3.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\slf4j\\slf4j-nop\\2.0.9\\slf4j-nop-2.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\sonatype\\plexus\\plexus-sec-dispatcher\\1.4\\plexus-sec-dispatcher-1.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\s3-wagon-private\\s3-wagon-private\\1.3.5\\s3-wagon-private-1.3.5.jar;C:\\Users\\runneradmin\\.m2\\repository\\funcool\\octet\\1.1.2\\octet-1.1.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\core.incubator\\0.1.2\\core.incubator-0.1.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\commons\\commons-compress\\1.8\\commons-compress-1.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\logging\\log4j\\log4j-api\\2.17.2\\log4j-api-2.17.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\cognitect\\aws\\api\\0.8.723\\api-0.8.723.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\cognitect\\aws\\endpoints\\871.2.29.39\\endpoints-871.2.29.39.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\cognitect\\aws\\s3\\871.2.29.35\\s3-871.2.29.35.jar;C:\\Users\\runneradmin\\.m2\\repository\\javax\\inject\\javax.inject\\1\\javax.inject-1.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-core\\3.8.8\\maven-core-3.8.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-resolver-provider\\3.8.8\\maven-resolver-provider-3.8.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\data.xml\\0.2.0-alpha9\\data.xml-0.2.0-alpha9.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.cli\\1.1.230\\tools.cli-1.1.230.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.gitlibs\\2.6.206\\tools.gitlibs-2.6.206.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\eclipse\\jetty\\jetty-client\\9.4.56.v20240826\\jetty-client-9.4.56.v20240826.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\eclipse\\jetty\\jetty-http\\9.4.56.v20240826\\jetty-http-9.4.56.v20240826.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\java.classpath\\1.1.0\\java.classpath-1.1.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\httpcomponents\\httpclient\\4.5.14\\httpclient-4.5.14.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\httpcomponents\\httpcore\\4.4.16\\httpcore-4.4.16.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-api\\1.9.4\\maven-resolver-api-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-connector-basic\\1.9.4\\maven-resolver-connector-basic-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-impl\\1.9.4\\maven-resolver-impl-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-spi\\1.9.4\\maven-resolver-spi-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-transport-file\\1.9.4\\maven-resolver-transport-file-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-transport-http\\1.9.4\\maven-resolver-transport-http-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-transport-wagon\\1.9.4\\maven-resolver-transport-wagon-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-util\\1.9.4\\maven-resolver-util-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\wagon\\wagon-http\\3.5.3\\wagon-http-3.5.3.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\wagon\\wagon-provider-api\\3.5.3\\wagon-provider-api-3.5.3.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\wagon\\wagon-ssh\\3.5.3\\wagon-ssh-3.5.3.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\tcrawley\\dynapath\\1.1.0\\dynapath-1.1.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-utils\\3.5.1\\plexus-utils-3.5.1.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-builder-support\\3.9.4\\maven-builder-support-3.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-interpolation\\1.26\\plexus-interpolation-1.26.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-sec-dispatcher\\2.0\\plexus-sec-dispatcher-2.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\slf4j\\slf4j-api\\2.0.9\\slf4j-api-2.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\sonatype\\plexus\\plexus-cipher\\1.4\\plexus-cipher-1.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\amazonaws\\aws-java-sdk-s3\\1.12.49\\aws-java-sdk-s3-1.12.49.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\amazonaws\\aws-java-sdk-sts\\1.12.49\\aws-java-sdk-sts-1.12.49.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\fasterxml\\jackson\\core\\jackson-core\\2.12.4\\jackson-core-2.12.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\fasterxml\\jackson\\core\\jackson-databind\\2.12.4\\jackson-databind-2.12.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\springframework\\build\\aws-maven\\4.8.0.RELEASE\\aws-maven-4.8.0.RELEASE.jar;C:\\Users\\runneradmin\\.m2\\repository\\io\\netty\\netty-buffer\\4.1.30.Final\\netty-buffer-4.1.30.Final.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\tukaani\\xz\\1.5\\xz-1.5.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\core.async\\1.7.701\\core.async-1.7.701.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\data.json\\2.5.1\\data.json-2.5.1.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.logging\\1.3.0\\tools.logging-1.3.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\google\\inject\\guice\\4.2.2\\guice-4.2.2-no_aop.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-artifact\\3.8.8\\maven-artifact-3.8.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-model\\3.8.8\\maven-model-3.8.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-model-builder\\3.8.8\\maven-model-builder-3.8.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-plugin-api\\3.8.8\\maven-plugin-api-3.8.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\maven-repository-metadata\\3.8.8\\maven-repository-metadata-3.8.8.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\shared\\maven-shared-utils\\3.3.4\\maven-shared-utils-3.3.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-classworlds\\2.6.0\\plexus-classworlds-2.6.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-component-annotations\\2.1.0\\plexus-component-annotations-2.1.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\eclipse\\sisu\\org.eclipse.sisu.inject\\0.3.5\\org.eclipse.sisu.inject-0.3.5.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\eclipse\\sisu\\org.eclipse.sisu.plexus\\0.3.5\\org.eclipse.sisu.plexus-0.3.5.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\eclipse\\jetty\\jetty-io\\9.4.56.v20240826\\jetty-io-9.4.56.v20240826.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\eclipse\\jetty\\jetty-util\\9.4.56.v20240826\\jetty-util-9.4.56.v20240826.jar;C:\\Users\\runneradmin\\.m2\\repository\\commons-logging\\commons-logging\\1.2\\commons-logging-1.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\commons\\commons-lang3\\3.12.0\\commons-lang3-3.12.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\resolver\\maven-resolver-named-locks\\1.9.4\\maven-resolver-named-locks-1.9.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\slf4j\\jcl-over-slf4j\\1.7.36\\jcl-over-slf4j-1.7.36.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\wagon\\wagon-http-shared\\3.5.3\\wagon-http-shared-3.5.3.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch\\0.1.55\\jsch-0.1.55.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch.agentproxy.connector-factory\\0.0.9\\jsch.agentproxy.connector-factory-0.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch.agentproxy.jsch\\0.0.9\\jsch.agentproxy.jsch-0.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\apache\\maven\\wagon\\wagon-ssh-common\\3.5.3\\wagon-ssh-common-3.5.3.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-interactivity-api\\1.1\\plexus-interactivity-api-1.1.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-cipher\\2.0\\plexus-cipher-2.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\amazonaws\\aws-java-sdk-core\\1.12.49\\aws-java-sdk-core-1.12.49.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\amazonaws\\aws-java-sdk-kms\\1.12.49\\aws-java-sdk-kms-1.12.49.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\amazonaws\\jmespath-java\\1.12.49\\jmespath-java-1.12.49.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\fasterxml\\jackson\\core\\jackson-annotations\\2.12.4\\jackson-annotations-2.12.4.jar;C:\\Users\\runneradmin\\.m2\\repository\\ch\\qos\\logback\\logback-classic\\1.0.12\\logback-classic-1.0.12.jar;C:\\Users\\runneradmin\\.m2\\repository\\io\\netty\\netty-common\\4.1.30.Final\\netty-common-4.1.30.Final.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.analyzer.jvm\\1.3.1\\tools.analyzer.jvm-1.3.1.jar;C:\\Users\\runneradmin\\.m2\\repository\\aopalliance\\aopalliance\\1.0\\aopalliance-1.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\javax\\annotation\\javax.annotation-api\\1.2\\javax.annotation-api-1.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch.agentproxy.core\\0.0.9\\jsch.agentproxy.core-0.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch.agentproxy.pageant\\0.0.9\\jsch.agentproxy.pageant-0.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch.agentproxy.sshagent\\0.0.9\\jsch.agentproxy.sshagent-0.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch.agentproxy.usocket-jna\\0.0.9\\jsch.agentproxy.usocket-jna-0.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\jcraft\\jsch.agentproxy.usocket-nc\\0.0.9\\jsch.agentproxy.usocket-nc-0.0.9.jar;C:\\Users\\runneradmin\\.m2\\repository\\com\\fasterxml\\jackson\\dataformat\\jackson-dataformat-cbor\\2.12.3\\jackson-dataformat-cbor-2.12.3.jar;C:\\Users\\runneradmin\\.m2\\repository\\commons-codec\\commons-codec\\1.15\\commons-codec-1.15.jar;C:\\Users\\runneradmin\\.m2\\repository\\joda-time\\joda-time\\2.8.1\\joda-time-2.8.1.jar;C:\\Users\\runneradmin\\.m2\\repository\\software\\amazon\\ion\\ion-java\\1.0.2\\ion-java-1.0.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\ch\\qos\\logback\\logback-core\\1.0.12\\logback-core-1.0.12.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\core.memoize\\1.1.266\\core.memoize-1.1.266.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.analyzer\\1.2.0\\tools.analyzer-1.2.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\tools.reader\\1.5.0\\tools.reader-1.5.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\ow2\\asm\\asm\\9.2\\asm-9.2.jar;C:\\Users\\runneradmin\\.m2\\repository\\net\\java\\dev\\jna\\jna\\4.1.0\\jna-4.1.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\net\\java\\dev\\jna\\jna-platform\\4.1.0\\jna-platform-4.1.0.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\codehaus\\plexus\\plexus-container-default\\1.0-alpha-9-stable-1\\plexus-container-default-1.0-alpha-9-stable-1.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\core.cache\\1.1.234\\core.cache-1.1.234.jar;C:\\Users\\runneradmin\\.m2\\repository\\classworlds\\classworlds\\1.1-alpha-2\\classworlds-1.1-alpha-2.jar;C:\\Users\\runneradmin\\.m2\\repository\\junit\\junit\\3.8.1\\junit-3.8.1.jar;C:\\Users\\runneradmin\\.m2\\repository\\org\\clojure\\data.priority-map\\1.2.0\\data.priority-map-1.2.0.jar;C:\\Users\\runneradmin\\.deps.clj\\1.12.0.1530\\ClojureTools\\exec.jar" "clojure.main" "-m" "clojure.run.exec" "prod-cli"], :type :babashka.process/error}
Location: /D:/a/eca/eca/scripts/make.clj:29:7

[ericdallo]

Just did! but I think that happens frequently for windows for some reason, would be nice if we find a way to fix if that keeps happening

EDIT:
it failed

[ikappaki]

I think i found a work around as per #180. It's difficult to say what's the root cause exactly.