Fix fork caches on windows (#3113)

<!-- Reference any GitHub issues resolved by this PR -->

Closes #2303

## Introduced changes

<!-- A brief description of the changes -->

- Fixed fork caches on windows.
- Restored forking related tests on windows

## Checklist

<!-- Make sure all of these are complete -->

- [x] Linked relevant issue
- [x] Updated relevant documentation
- [x] Added relevant tests
- [x] Performed self-review of the code
- [x] Added changes to `CHANGELOG.md`

Author: cptartur

CHANGELOG.md

@@ -20,6 +20,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - gas is now reported using resource bounds triplet (l1_gas, l1_data_gas and l2_gas)
 - `available_gas` now accepts named arguments denoting resource bounds (eg #[available_gas(l1_gas: 1, l1_data_gas: 2, l2_gas: 3)])
 
+#### Fixed
+
+- Bug with file locking that prevented forking from working on Windows
+
 ### Cast
 
 #### Added

crates/cheatnet/src/forking/cache.rs

@@ -11,8 +11,8 @@ use starknet_api::state::StorageKey;
 use starknet_types_core::felt::Felt;
 use std::collections::HashMap;
 use std::fs;
-use std::fs::OpenOptions;
-use std::io::{Read, Write};
+use std::fs::{File, OpenOptions};
+use std::io::{Read, Seek, Write};
 use std::string::ToString;
 use url::Url;
 
@@ -99,6 +99,18 @@ impl Drop for ForkCache {
     }
 }
 
+trait FileTruncateExtension {
+    fn clear(&mut self) -> Result<()>;
+}
+
+impl FileTruncateExtension for File {
+    fn clear(&mut self) -> Result<()> {
+        self.set_len(0).context("Failed to truncate file")?;
+        self.rewind().context("Failed to rewind file")?;
+        Ok(())
+    }
+}
+
 impl ForkCache {
     pub(crate) fn load_or_new(
         url: &Url,
@@ -134,15 +146,17 @@ impl ForkCache {
     fn save(&self) {
         let mut file = OpenOptions::new()
             .write(true)
+            .read(true)
             .create(true)
             .truncate(false)
             .open(&self.cache_file)
             .unwrap();
 
         file.lock_exclusive().expect("Could not lock on cache file");
 
-        let cache_file_content =
-            fs::read_to_string(&self.cache_file).expect("Should have been able to read the cache");
+        let mut cache_file_content = String::new();
+        file.read_to_string(&mut cache_file_content)
+            .expect("Should have been able to read the cache");
 
         let output = if cache_file_content.is_empty() {
             self.fork_cache_content.to_string()
@@ -152,6 +166,7 @@ impl ForkCache {
             fs_fork_cache_content.to_string()
         };
 
+        file.clear().expect("Failed to clear file");
         file.write_all(output.as_bytes())
             .expect("Could not write cache to file");
 

crates/forge/tests/e2e/mod.rs

@@ -17,9 +17,7 @@ mod debugging;
 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/integration/mod.rs

@@ -5,7 +5,6 @@ mod cheat_block_number;
 mod cheat_block_timestamp;
 mod cheat_caller_address;
 mod cheat_execution_info;
-#[cfg(not(target_os = "windows"))]
 mod cheat_fork;
 mod cheat_sequencer_address;
 mod declare;
@@ -26,7 +25,6 @@ mod pure_cairo;
 mod replace_bytecode;
 mod resources;
 mod runtime;
-#[cfg(not(target_os = "windows"))]
 mod setup_fork;
 mod should_panic;
 mod signing;

crates/forge/tests/integration/spy_events.rs

@@ -632,7 +632,6 @@ fn assert_not_emitted_fails() {
     assert_case_output_contains(&result, "assert_not_emitted_fails", "keys was emitted");
 }
 
-#[cfg(not(target_os = "windows"))]
 #[test]
 fn capture_cairo0_event() {
     let test = test_case!(

crates/forge/tests/integration/store_load.rs

@@ -561,7 +561,6 @@ fn store_load_felt_to_felt() {
     assert_passed(&result);
 }
 
-#[cfg(not(target_os = "windows"))]
 #[test]
 fn fork_store_load() {
     let test = test_utils::test_case!(formatdoc!(