chore(PocketIC): PocketIC library build on Windows (#2094)

This PR adds a GitHub CI flow for PocketIC library build on Windows.

.github/workflows/pocket-ic-build-windows.yml

@@ -0,0 +1,27 @@
+name: 'PocketIC library build on Windows'
+on:
+  pull_request:
+    paths:
+      - packages/pocket-ic/**
+  push:
+    branches:
+      - master
+# runs for the same workflow are cancelled on PRs but not on master
+# explanation: on push to master head_ref is not set, so we want it to fall back to run_id so it is not cancelled
+concurrency:
+  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+  cancel-in-progress: true
+jobs:
+  pocket_ic_library_build_windows:
+    name: 'PocketIC library build on Windows'
+    runs-on: windows-2022
+    steps:
+      - name: 'Checkout'
+        uses: actions/checkout@v4
+      - name: 'cargo clippy'
+        run: |
+          cargo clippy --locked --all-features -p pocket-ic -p pocket-ic-test-canister --all-targets -- -D warnings -D clippy::all -D clippy::mem_forget -C debug-assertions=off
+      - name: 'cargo build'
+        run: |
+          cargo build --release --locked -p pocket-ic
+          cargo build --release --locked -p pocket-ic-test-canister --target wasm32-unknown-unknown

Cargo.toml

@@ -10,6 +10,7 @@ members = [
     "packages/ic-ledger-hash-of",
     "packages/ic-signature-verification",
     "packages/pocket-ic",
+    "packages/pocket-ic/test_canister",
     "packages/ic-vetkd-utils",
     "rs/artifact_pool",
     "rs/async_utils",

packages/pocket-ic/BUILD.bazel

@@ -1,5 +1,4 @@
-load("@rules_rust//rust:defs.bzl", "rust_library", "rust_test", "rust_test_suite")
-load("//bazel:canisters.bzl", "rust_canister")
+load("@rules_rust//rust:defs.bzl", "rust_library", "rust_test_suite")
 
 package(default_visibility = ["//visibility:public"])
 
@@ -46,38 +45,17 @@ rust_library(
     deps = DEPENDENCIES,
 )
 
-rust_canister(
-    name = "test_canister",
-    srcs = ["tests/test_canister.rs"],
-    service_file = ":tests/test_canister.did",
-    deps = [
-        # Keep sorted.
-        "@crate_index//:candid",
-        "@crate_index//:ic-cdk",
-        "@crate_index//:serde",
-        "@crate_index//:serde_bytes",
-    ],
-)
-
-rust_test(
-    name = "test_canister_unit_test",
-    crate = ":_wasm_test_canister",
-    data = ["tests/test_canister.did"],
-    env = {"CARGO_MANIFEST_DIR": "packages/pocket-ic"},
-    deps = [":pocket-ic"] + DEPENDENCIES + TEST_DEPENDENCIES,
-)
-
 rust_test_suite(
     name = "test",
     size = "medium",
     srcs = ["tests/tests.rs"],
     data = [
-        ":test_canister.wasm",
+        "//packages/pocket-ic/test_canister:test_canister.wasm",
         "//rs/pocket_ic_server:pocket-ic-server",
     ],
     env = {
         "POCKET_IC_BIN": "$(rootpath //rs/pocket_ic_server:pocket-ic-server)",
-        "TEST_WASM": "$(rootpath :test_canister.wasm)",
+        "TEST_WASM": "$(rootpath //packages/pocket-ic/test_canister:test_canister.wasm)",
     },
     flaky = False,
     proc_macro_deps = MACRO_DEPENDENCIES,

packages/pocket-ic/test_canister/BUILD.bazel

@@ -0,0 +1,14 @@
+load("//bazel:canisters.bzl", "rust_canister")
+
+rust_canister(
+    name = "test_canister",
+    srcs = ["src/canister.rs"],
+    service_file = ":canister.did",
+    deps = [
+        # Keep sorted.
+        "@crate_index//:candid",
+        "@crate_index//:ic-cdk",
+        "@crate_index//:serde",
+        "@crate_index//:serde_bytes",
+    ],
+)

packages/pocket-ic/test_canister/Cargo.toml

@@ -0,0 +1,17 @@
+[package]
+name = "pocket-ic-test-canister"
+version.workspace = true
+authors.workspace = true
+edition.workspace = true
+description.workspace = true
+documentation.workspace = true
+
+[[bin]]
+name = "pocket-ic-test-canister"
+path = "src/canister.rs"
+
+[dependencies]
+candid = { workspace = true }
+ic-cdk = { workspace = true }
+serde = { workspace = true }
+serde_bytes = { workspace = true }