ci: fix windows tests with .exe suffix

.github/actions/cli-tests/action.yml

@@ -53,10 +53,6 @@ runs:
       working-directory: cli-tests
       run: |
         npm install
-        which solc
-        which zksolc
-        solc --version
-        zksolc --version
         npx jest --ci --reporters=default --reporters=jest-junit
 
     - name: Upload results Linux

.github/actions/install-solc/action.yml

@@ -46,17 +46,17 @@ runs:
               SOLC_DOWNLOAD_FILENAME="solc-macosx-amd64-${SOLC_VERSION}"
               ;;
             Windows*)
+              WIN_SUFFIX=".exe"
               SOLC_DOWNLOAD_FILENAME="solc-windows-amd64-${SOLC_VERSION}.exe"
               ;;
           esac
+          OUTPUT="${OUTPUT}${WIN_SUFFIX}"
           curl --location -o "${OUTPUT}" "${SOLC_URL}/${SOLC_VERSION}/${SOLC_DOWNLOAD_FILENAME}" && \
           chmod a+x "${OUTPUT}"
-          [ ${SOLC_VERSION} = ${{ inputs.solc-version-latest }} ] && cp "${OUTPUT}" "${{ inputs.output-dir }}/${{ inputs.solc-default-name }}" && cp "${OUTPUT}" "${{ inputs.solc-default-name }}"
+          [ ${SOLC_VERSION} = ${{ inputs.solc-version-latest }} ] && cp "${OUTPUT}" "${{ inputs.output-dir }}/${{ inputs.solc-default-name }}${WIN_SUFFIX}"
         done
         echo "${PWD}/${{ inputs.output-dir }}" >> "${GITHUB_PATH}"
-
-        # Tweak
-        echo "${PWD}" >> "${GITHUB_PATH}"
+        ls -la ${PWD}/${{ inputs.output-dir }}
 
     - name: Install upstream solc on Windows
       if: runner.os == 'Windows'
@@ -69,6 +69,6 @@ runs:
           curl --location -o "${SOLC_DOWNLOAD}.zip" "${SOLC_URL}/${SOLC_DOWNLOAD}.zip"
           unzip -d ${SOLC_DOWNLOAD} "${SOLC_DOWNLOAD}.zip"
           SOLC_SHORT_VERSION=$(echo "${SOLC_VERSION}" | cut -d+ -f1 | tr -d v)
-          mv "${SOLC_DOWNLOAD}/solc.exe" "${{ inputs.output-dir }}/${{ inputs.solc-default-name }}-${SOLC_SHORT_VERSION}"
+          mv "${SOLC_DOWNLOAD}/solc.exe" "${{ inputs.output-dir }}/${{ inputs.solc-default-name }}-${SOLC_SHORT_VERSION}.exe"
         done
-        ls -la "${PWD}/${{ inputs.output-dir }}"
+        ls -la ${PWD}/${{ inputs.output-dir }}

.github/workflows/test.yaml

@@ -35,18 +35,18 @@ jobs:
       fail-fast: false # finalize testing of all targets even if one failed
       matrix:
         include:
-          # - name: "MacOS x86"
-          #   runner: macos-12-large
-          # - name: "MacOS arm64"
-          #   runner: [self-hosted, macOS, ARM64]
-          # - name: "Linux x86"
-          #   runner: matterlabs-ci-runner
-          #   image: matterlabs/llvm_runner:ubuntu22-llvm17-latest
-          #   target: "x86_64-unknown-linux-musl"
-          # - name: "Linux ARM64"
-          #   runner: matterlabs-ci-runner-arm
-          #   image: matterlabs/llvm_runner:ubuntu22-llvm17-latest
-          #   target: "aarch64-unknown-linux-musl"
+          - name: "MacOS x86"
+            runner: macos-12-large
+          - name: "MacOS arm64"
+            runner: [self-hosted, macOS, ARM64]
+          - name: "Linux x86"
+            runner: matterlabs-ci-runner
+            image: matterlabs/llvm_runner:ubuntu22-llvm17-latest
+            target: "x86_64-unknown-linux-musl"
+          - name: "Linux ARM64"
+            runner: matterlabs-ci-runner-arm
+            image: matterlabs/llvm_runner:ubuntu22-llvm17-latest
+            target: "aarch64-unknown-linux-musl"
           - name: "Windows"
             runner: windows-2022-github-hosted-16core
     runs-on: ${{ matrix.runner }}

src/tests/mod.rs

@@ -46,7 +46,13 @@ pub fn build_solidity(
     let _ = crate::process::EXECUTABLE.set(PathBuf::from(crate::r#const::DEFAULT_EXECUTABLE_NAME));
 
     let solc_compiler = SolcCompiler::new(
-        format!("{}-{}", SolcCompiler::DEFAULT_EXECUTABLE_NAME, solc_version).as_str(),
+        format!(
+            "{}-{}{}",
+            SolcCompiler::DEFAULT_EXECUTABLE_NAME,
+            solc_version,
+            std::env::consts::EXE_SUFFIX,
+        )
+        .as_str(),
     )?;
 
     let solc_input = SolcStandardJsonInput::try_from_solidity_sources(
@@ -132,7 +138,13 @@ pub fn build_solidity_and_detect_missing_libraries(
     let _ = crate::process::EXECUTABLE.set(PathBuf::from(crate::r#const::DEFAULT_EXECUTABLE_NAME));
 
     let solc_compiler = SolcCompiler::new(
-        format!("{}-{}", SolcCompiler::DEFAULT_EXECUTABLE_NAME, solc_version).as_str(),
+        format!(
+            "{}-{}{}",
+            SolcCompiler::DEFAULT_EXECUTABLE_NAME,
+            solc_version,
+            std::env::consts::EXE_SUFFIX,
+        )
+        .as_str(),
     )?;
 
     let solc_input = SolcStandardJsonInput::try_from_solidity_sources(
@@ -375,7 +387,13 @@ pub fn check_solidity_message(
     check_dependencies(Some(solc_version));
 
     let solc_compiler = SolcCompiler::new(
-        format!("{}-{}", SolcCompiler::DEFAULT_EXECUTABLE_NAME, solc_version).as_str(),
+        format!(
+            "{}-{}{}",
+            SolcCompiler::DEFAULT_EXECUTABLE_NAME,
+            solc_version,
+            std::env::consts::EXE_SUFFIX,
+        )
+        .as_str(),
     )?;
     if skip_for_zksync_edition && solc_compiler.version.l2_revision.is_some() {
         return Ok(true);
@@ -425,9 +443,10 @@ fn check_dependencies(solc_version: Option<&semver::Version>) {
     let mut executables = vec![crate::r#const::DEFAULT_EXECUTABLE_NAME.to_owned()];
     if let Some(solc_version) = solc_version {
         executables.push(format!(
-            "{}-{}",
+            "{}-{}{}",
             SolcCompiler::DEFAULT_EXECUTABLE_NAME,
-            solc_version
+            solc_version,
+            std::env::consts::EXE_SUFFIX,
         ));
     }
     for executable in executables.into_iter() {

src/tests/standard_json.rs

@@ -4,6 +4,7 @@
 
 #![cfg(test)]
 
+use std::env::consts::EXE_SUFFIX;
 use std::path::PathBuf;
 
 use crate::solc::standard_json::input::Input as SolcStandardJsonInput;
@@ -42,9 +43,10 @@ fn standard_json_yul_default_validated() {
     .expect("Standard JSON reading error");
     let solc_compiler = SolcCompiler::new(
         format!(
-            "{}-{}",
+            "{}-{}{}",
             SolcCompiler::DEFAULT_EXECUTABLE_NAME,
-            SolcCompiler::LAST_SUPPORTED_VERSION
+            SolcCompiler::LAST_SUPPORTED_VERSION,
+            EXE_SUFFIX
         )
         .as_str(),
     )
@@ -102,9 +104,10 @@ fn standard_json_yul_default_urls_validated() {
     .expect("Standard JSON reading error");
     let solc_compiler = SolcCompiler::new(
         format!(
-            "{}-{}",
+            "{}-{}{}",
             SolcCompiler::DEFAULT_EXECUTABLE_NAME,
-            SolcCompiler::LAST_SUPPORTED_VERSION
+            SolcCompiler::LAST_SUPPORTED_VERSION,
+            EXE_SUFFIX
         )
         .as_str(),
     )
@@ -162,9 +165,10 @@ fn standard_json_yul_eravm_validated() {
     .expect("Standard JSON reading error");
     let solc_compiler = SolcCompiler::new(
         format!(
-            "{}-{}",
+            "{}-{}{}",
             SolcCompiler::DEFAULT_EXECUTABLE_NAME,
-            SolcCompiler::LAST_SUPPORTED_VERSION
+            SolcCompiler::LAST_SUPPORTED_VERSION,
+            EXE_SUFFIX
         )
         .as_str(),
     )
@@ -222,9 +226,10 @@ fn standard_json_yul_eravm_urls_validated() {
     .expect("Standard JSON reading error");
     let solc_compiler = SolcCompiler::new(
         format!(
-            "{}-{}",
+            "{}-{}{}",
             SolcCompiler::DEFAULT_EXECUTABLE_NAME,
-            SolcCompiler::LAST_SUPPORTED_VERSION
+            SolcCompiler::LAST_SUPPORTED_VERSION,
+            EXE_SUFFIX
         )
         .as_str(),
     )