Run some checks on windows

.gitattributes

@@ -0,0 +1 @@
+*.* text eol=lf

.github/workflows/ci.yml

@@ -11,7 +11,10 @@ on:
 jobs:
   test-forge-unit-and-integration:
     name: Test Forge / Unit and Integration Tests
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
     steps:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
@@ -23,58 +26,62 @@ jobs:
       - run: cargo test --release integration -p forge
 
   build-test-forge-e2e-nextest-archive:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
     steps:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
       - uses: Swatinem/rust-cache@82a92a6e8fbeee089604da2575dc567ae9ddeaab
       - name: Install nextest
         uses: taiki-e/install-action@nextest
       - name: Build and archive tests
-        run: cargo nextest archive --release -p forge --archive-file nextest-archive.tar.zst
+        run: cargo nextest archive --release -p forge --archive-file 'nextest-archive-${{ matrix.os }}.tar.zst'
       - name: Upload archive to workflow
         uses: actions/upload-artifact@v4
         with:
-          name: nextest-archive
-          path: nextest-archive.tar.zst
+          name: nextest-archive-${{ matrix.os }}
+          path: nextest-archive-${{ matrix.os }}.tar.zst
 
   test-forge-e2e:
     name: Test Forge / E2E Tests
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
     needs: build-test-forge-e2e-nextest-archive
     strategy:
+      fail-fast: false
       matrix:
         partition: [ 1, 2, 3, 4, 5, 6, 7, 8 ]
+        os: [ ubuntu-latest, windows-latest ]
     steps:
       - name: Extract branch name
         if: github.event_name != 'pull_request'
         run: echo "BRANCH_NAME=$(echo ${GITHUB_REF#refs/heads/})" >> $GITHUB_ENV
+        shell: bash
 
       - name: Extract branch name on pull request
         if: github.event_name == 'pull_request'
         run: echo "BRANCH_NAME=$(echo $GITHUB_HEAD_REF)" >> $GITHUB_ENV
+        shell: bash
 
       - name: Extract repo name and owner
         if: github.event_name != 'pull_request'
         run: echo "REPO_NAME=$(echo ${{ github.repository }}.git)" >> $GITHUB_ENV
+        shell: bash
 
       - name: Extract repo name and owner on pull request
         if: github.event_name == 'pull_request'
         run: echo "REPO_NAME=$(echo ${{ github.event.pull_request.head.repo.full_name }}.git)" >> $GITHUB_ENV
+        shell: bash
 
       - name: Print repo name
         run: echo 'The repo name is' $REPO_NAME
+        shell: bash
 
       - name: Get branch name
         run: echo 'The branch name is' $BRANCH_NAME
-
-      - name: Install cairo-profiler
-        run: |
-          curl -L https://raw.githubusercontent.com/software-mansion/cairo-profiler/main/scripts/install.sh | sh
-
-      - name: Install cairo-coverage
-        run: |
-          curl -L https://raw.githubusercontent.com/software-mansion/cairo-coverage/main/scripts/install.sh | sh
+        shell: bash
 
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
@@ -84,9 +91,9 @@ jobs:
       - uses: taiki-e/install-action@nextest
       - uses: actions/download-artifact@v4
         with:
-          name: nextest-archive
+          name: nextest-archive-${{ matrix.os }}
       - name: nextest partition ${{ matrix.partition }}/8
-        run: cargo nextest run --partition 'count:${{ matrix.partition }}/8' --archive-file 'nextest-archive.tar.zst' e2e
+        run: cargo nextest run --no-fail-fast --partition 'count:${{ matrix.partition }}/8' --archive-file 'nextest-archive-${{ matrix.os }}.tar.zst' e2e
 
   test-scarb-2-8-3:
     name: Test scarb 2.8.3
@@ -104,8 +111,13 @@ jobs:
         run: |
           curl -L https://raw.githubusercontent.com/software-mansion/cairo-coverage/main/scripts/install.sh | sh
 
+      - name: Install cairo-profiler
+        run: |
+          curl -L https://raw.githubusercontent.com/software-mansion/cairo-profiler/main/scripts/install.sh | sh
+
       - run: cargo test --package forge --features scarb_2_8_3 --test main e2e::coverage
       - run: cargo test --package forge --features scarb_2_8_3 --test main e2e::backtrace
+      - run: cargo test --package forge --features scarb_2_8_3 --test main e2e::build_profile
 
   test-coverage-error:
     name: Test coverage error
@@ -188,15 +200,24 @@ jobs:
 
   test-cast:
     name: Test Cast
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
     steps:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@7b1c307e0dcbda6122208f10795a713336a9b35a
         with:
           toolchain: stable
       - uses: Swatinem/rust-cache@82a92a6e8fbeee089604da2575dc567ae9ddeaab
-      - name: Install starknet-devnet-rs
-        run: ./scripts/install_devnet.sh
+      - name: Install starknet-devnet-rs on Linux/Macos
+        if: runner.os != 'Windows'
+        run: |
+          ./scripts/install_devnet.sh
+      - name: Install starknet-devnet-rs on Windows
+        if: runner.os == 'Windows'
+        run: |
+          powershell .\scripts\install_devnet.ps1
       - uses: software-mansion/setup-scarb@v1
         with:
           scarb-version: "2.7.0"
@@ -229,7 +250,10 @@ jobs:
 
   test-scarb-api:
     name: Test Scarb Api
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
     steps:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable

Cargo.toml

@@ -124,3 +124,4 @@ serde_path_to_error = "0.1.16"
 wiremock = "0.6.0"
 const-hex = "1.14.0"
 indicatif = "0.17.9"
+sanitize-filename = "0.6.0"

crates/forge-runner/Cargo.toml

@@ -55,3 +55,4 @@ shared = { path = "../shared" }
 universal-sierra-compiler-api = { path = "../universal-sierra-compiler-api" }
 fs4.workspace = true
 which.workspace = true
+sanitize-filename.workspace = true

crates/forge-runner/src/build_trace_data.rs

@@ -307,7 +307,7 @@ fn build_profiler_trace_entry(value: &RelocatedTraceEntry) -> ProfilerTraceEntry
 }
 
 pub fn save_trace_data(
-    test_name: &String,
+    test_name: &str,
     trace_data: &VersionedProfilerCallTrace,
 ) -> Result<PathBuf> {
     let serialized_trace =

crates/forge-runner/src/lib.rs

@@ -68,9 +68,10 @@ pub fn maybe_save_trace_and_profile(
     }) = result
     {
         if execution_data_to_save.is_vm_trace_needed() {
-            let trace_path = save_trace_data(name, trace_data)?;
+            let name = sanitize_filename::sanitize(name.replace("::", "_"));
+            let trace_path = save_trace_data(&name, trace_data)?;
             if execution_data_to_save.profile {
-                run_profiler(name, &trace_path, &execution_data_to_save.additional_args)?;
+                run_profiler(&name, &trace_path, &execution_data_to_save.additional_args)?;
             }
             return Ok(Some(trace_path));
         }

crates/forge/tests/e2e/build_profile.rs

@@ -2,33 +2,35 @@ use super::common::runner::{setup_package, test_runner};
 use forge_runner::profiler_api::PROFILE_DIR;
 
 #[test]
+#[cfg_attr(not(feature = "scarb_2_8_3"), ignore)]
 fn simple_package_build_profile() {
     let temp = setup_package("simple_package");
 
     test_runner(&temp).arg("--build-profile").assert().code(1);
 
     assert!(temp
         .join(PROFILE_DIR)
-        .join("simple_package::tests::test_fib.pb.gz")
+        .join("simple_package_tests_test_fib.pb.gz")
         .is_file());
     assert!(!temp
         .join(PROFILE_DIR)
-        .join("simple_package_integrationtest::test_simple::test_failing.pb.gz")
+        .join("simple_package_integrationtest_test_simple_test_failing.pb.gz")
         .is_file());
     assert!(!temp
         .join(PROFILE_DIR)
-        .join("simple_package::tests::ignored_test.pb.gz")
+        .join("simple_package_tests_ignored_test.pb.gz")
         .is_file());
     assert!(temp
         .join(PROFILE_DIR)
-        .join("simple_package_integrationtest::ext_function_test::test_simple.pb.gz")
+        .join("simple_package_integrationtest_ext_function_test_test_simple.pb.gz")
         .is_file());
 
     // Check if it doesn't crash in case some data already exists
     test_runner(&temp).arg("--build-profile").assert().code(1);
 }
 
 #[test]
+#[cfg_attr(not(feature = "scarb_2_8_3"), ignore)]
 fn simple_package_build_profile_and_pass_args() {
     let temp = setup_package("simple_package");
 

crates/forge/tests/e2e/build_trace_data.rs

@@ -16,24 +16,24 @@ fn simple_package_save_trace() {
 
     assert!(temp
         .join(TRACE_DIR)
-        .join("simple_package::tests::test_fib.json")
+        .join("simple_package_tests_test_fib.json")
         .exists());
     assert!(!temp
         .join(TRACE_DIR)
-        .join("simple_package_integrationtest::test_simple::test_failing.json")
+        .join("simple_package_integrationtest_test_simple_test_failing.json")
         .exists());
     assert!(!temp
         .join(TRACE_DIR)
-        .join("simple_package::tests::ignored_test.json")
+        .join("simple_package_tests_ignored_test.json")
         .exists());
     assert!(temp
         .join(TRACE_DIR)
-        .join("simple_package_integrationtest::ext_function_test::test_simple.json")
+        .join("simple_package_integrationtest_ext_function_test_test_simple.json")
         .exists());
 
     let trace_data = fs::read_to_string(
         temp.join(TRACE_DIR)
-            .join("simple_package_integrationtest::ext_function_test::test_simple.json"),
+            .join("simple_package_integrationtest_ext_function_test_test_simple.json"),
     )
     .unwrap();
 
@@ -56,7 +56,7 @@ fn trace_has_contract_and_function_names() {
 
     let trace_data = fs::read_to_string(
         temp.join(TRACE_DIR)
-            .join("trace_info_integrationtest::test_trace::test_trace.json"),
+            .join("trace_info_integrationtest_test_trace_test_trace.json"),
     )
     .unwrap();
 
@@ -103,7 +103,7 @@ fn trace_has_cairo_execution_info() {
 
     let trace_data = fs::read_to_string(
         temp.join(TRACE_DIR)
-            .join("trace_info_integrationtest::test_trace::test_trace.json"),
+            .join("trace_info_integrationtest_test_trace_test_trace.json"),
     )
     .unwrap();
 
@@ -145,7 +145,7 @@ fn trace_has_deploy_with_no_constructor_phantom_nodes() {
 
     let trace_data = fs::read_to_string(
         temp.join(TRACE_DIR)
-            .join("trace_info_integrationtest::test_trace::test_trace.json"),
+            .join("trace_info_integrationtest_test_trace_test_trace.json"),
     )
     .unwrap();
 

crates/forge/tests/e2e/io_operations.rs

@@ -21,7 +21,7 @@ fn file_reading() {
         [FAIL] file_reading_integrationtest::test::json_non_existent
 
         Failure data:
-            "No such file or directory [..]"
+            [..]
 
         [FAIL] file_reading_integrationtest::test::invalid_json
 
@@ -31,7 +31,7 @@ fn file_reading() {
         [FAIL] file_reading_integrationtest::test::non_existent
 
         Failure data:
-            "No such file or directory [..]"
+            "[..] file [..]"
 
         [FAIL] file_reading_integrationtest::test::non_ascii
 

crates/forge/tests/e2e/mod.rs

@@ -11,7 +11,9 @@ mod coverage;
 mod docs_snippets_validation;
 mod env;
 mod features;
+#[cfg(not(target_os = "windows"))]
 mod fork_warning;
+#[cfg(not(target_os = "windows"))]
 mod forking;
 mod fuzzing;
 mod io_operations;

crates/forge/tests/e2e/running.rs

@@ -1045,7 +1045,7 @@ fn catch_runtime_errors() {
                 use snforge_std::fs::{FileTrait, read_txt};
 
                 #[test]
-                #[should_panic(expected: "No such file or directory (os error 2)")]
+                #[should_panic(expected: "file")]
                 fn catch_no_such_file() {
                     let file = FileTrait::new("no_way_this_file_exists");
                     let content = read_txt(@file);

crates/forge/tests/e2e/trace_resources.rs

@@ -65,7 +65,7 @@ fn assert_resources_for_test(
 
 fn deserialize_call_trace(test_name: &str, temp_dir: &TempDir) -> VersionedProfilerCallTrace {
     let trace_data = fs::read_to_string(temp_dir.join(TRACE_DIR).join(format!(
-        "trace_resources_tests::{test_name}::{test_name}.json"
+        "trace_resources_tests_{test_name}_{test_name}.json"
     )))
     .unwrap();
     serde_json::from_str(&trace_data).expect("Failed to parse call trace")