Move some tests to a separate directory. (#4406)

## Motivation

Several tests have emerged that are unit test in nature but require a
full application to be built.
Currently, the source code of those tests is in `examples` which makes
them sit alongside more
functional applications that are of more general interest. 

## Proposal

The tests are moved to the directory `linera-sdk/tests/fixtures`. They
are not part of the strusture of the
cargo crate.

The advantage is that those smart contracts are no longer compiled again
and again (ScyllaDb, DynamoDb,
StorageService, RemoteNet), and so we gain on CI here. The cost of
compiling for tests in `--release` is about
2-3 minutes per such instance.

The cost of running `linera-sdk/tests/fixtures` is 7 minutes, so
overall, we have a speed gain from this work.

## Test Plan

The CI.
The same tests are being tested. The same tests are being applied, just
differently ordered.

## Release Plan

- Nothing to do / These changes follow the usual release cycle.

## Links

None.

Author: MathieuDutSik

.github/workflows/rust.yml

@@ -139,6 +139,29 @@ jobs:
         cd examples
         cargo test --locked
 
+  linera-sdk-tests-fixtures:
+    needs: changed-files
+    if: needs.changed-files.outputs.should-run == 'true'
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+
+    steps:
+    - uses: actions/checkout@v4
+    - uses: actions-rust-lang/setup-rust-toolchain@v1
+    - name: Install Protoc
+      uses: arduino/setup-protoc@v3
+      with:
+        repo-token: ${{ secrets.GITHUB_TOKEN }}
+    - name: Run linera-sdk fixtures
+      run: |
+        cd linera-sdk/tests/fixtures
+        cargo test --locked
+    - name: Run linera-sdk tests application lints
+      run: |
+        cd linera-sdk/tests/fixtures
+        cargo fmt -- --check
+        cargo clippy --all-targets --all-features --target wasm32-unknown-unknown --locked
+
   default-features-and-witty-integration-test:
     needs: changed-files
     if: needs.changed-files.outputs.should-run == 'true'

Cargo.lock

@@ -176,7 +176,7 @@ proptest = { version = "1.6.0", default-features = false, features = ["alloc"] }
 prost = "0.14"
 quick_cache = "0.6.13"
 quote = "1.0"
-rand = { version = "0.8.5", default-features = false }
+rand = { version = "0.8.5", default-features = false, features = ["std_rng"] }
 rand_chacha = { version = "0.3.1", default-features = false }
 rand_distr = { version = "0.4.3", default-features = false }
 rcgen = "0.12.1"
@@ -323,10 +323,10 @@ ethereum-tracker = { path = "./examples/ethereum-tracker" }
 fungible = { path = "./examples/fungible" }
 hex-game = { path = "./examples/hex-game" }
 matching-engine = { path = "./examples/matching-engine" }
-meta-counter = { path = "./examples/meta-counter" }
+meta-counter = { path = "./linera-sdk/tests/fixtures/meta-counter" }
 native-fungible = { path = "./examples/native-fungible" }
 non-fungible = { path = "./examples/non-fungible" }
-publish-read-data-blob = { path = "./examples/publish-read-data-blob" }
+publish-read-data-blob = { path = "./linera-sdk/tests/fixtures/publish-read-data-blob" }
 social = { path = "./examples/social" }
 track-instantiation = { path = "./examples/track-instantiation" }
 

Cargo.toml

@@ -3,10 +3,8 @@ resolver = "2"
 members = [
     "amm",
     "call-evm-counter",
-    "contract-call",
     "counter",
     "counter-no-graphql",
-    "create-and-call",
     "crowd-funding",
     "ethereum-tracker",
     "fungible",
@@ -15,13 +13,10 @@ members = [
     "how-to/perform-http-requests",
     "llm",
     "matching-engine",
-    "meta-counter",
     "native-fungible",
     "non-fungible",
-    "publish-read-data-blob",
     "rfq",
     "social",
-    "track-instantiation",
 ]
 
 [workspace.dependencies]

examples/Cargo.lock

@@ -19,6 +19,9 @@ mod wasmer;
 #[cfg(with_wasmtime)]
 mod wasmtime;
 
+#[cfg(with_fs)]
+use std::path::Path;
+
 use linera_base::data_types::Bytecode;
 #[cfg(with_metrics)]
 use linera_base::prometheus_util::MeasureLatency as _;
@@ -94,7 +97,7 @@ impl WasmContractModule {
     /// Creates a new [`WasmContractModule`] using the WebAssembly module in `contract_bytecode_file`.
     #[cfg(with_fs)]
     pub async fn from_file(
-        contract_bytecode_file: impl AsRef<std::path::Path>,
+        contract_bytecode_file: impl AsRef<Path>,
         runtime: WasmRuntime,
     ) -> Result<Self, WasmExecutionError> {
         Self::new(
@@ -156,7 +159,7 @@ impl WasmServiceModule {
     /// Creates a new [`WasmServiceModule`] using the WebAssembly module in `service_bytecode_file`.
     #[cfg(with_fs)]
     pub async fn from_file(
-        service_bytecode_file: impl AsRef<std::path::Path>,
+        service_bytecode_file: impl AsRef<Path>,
         runtime: WasmRuntime,
     ) -> Result<Self, WasmExecutionError> {
         Self::new(
@@ -339,36 +342,48 @@ impl From<::wasmer::InstantiationError> for WasmExecutionError {
 /// This assumes that the current directory is one of the crates.
 #[cfg(with_testing)]
 pub mod test {
-    use std::sync::LazyLock;
+    use std::{path::Path, sync::LazyLock};
 
     #[cfg(with_fs)]
     use super::{WasmContractModule, WasmRuntime, WasmServiceModule};
 
-    fn build_applications() -> Result<(), std::io::Error> {
-        tracing::info!("Building example applications with cargo");
+    fn build_applications_in_directory(dir: &str) -> Result<(), std::io::Error> {
         let output = std::process::Command::new("cargo")
-            .current_dir("../examples")
+            .current_dir(dir)
             .args(["build", "--release", "--target", "wasm32-unknown-unknown"])
             .output()?;
-        assert!(
-            output.status.success(),
-            "Failed to build example applications.\n\n\
-                stdout:\n-------\n{}\n\n\
-                stderr:\n-------\n{}",
-            String::from_utf8_lossy(&output.stdout),
-            String::from_utf8_lossy(&output.stderr),
-        );
+        if !output.status.success() {
+            panic!(
+                "Failed to build applications in directory {dir}.\n\n\
+                 stdout:\n-------\n{}\n\n\
+                 stderr:\n-------\n{}",
+                String::from_utf8_lossy(&output.stdout),
+                String::from_utf8_lossy(&output.stderr),
+            );
+        }
+        Ok(())
+    }
+
+    fn build_applications() -> Result<(), std::io::Error> {
+        for dir in ["../examples", "../linera-sdk/tests/fixtures"] {
+            build_applications_in_directory(dir)?;
+        }
         Ok(())
     }
 
     pub fn get_example_bytecode_paths(name: &str) -> Result<(String, String), std::io::Error> {
         let name = name.replace('-', "_");
         static INSTANCE: LazyLock<()> = LazyLock::new(|| build_applications().unwrap());
         LazyLock::force(&INSTANCE);
-        Ok((
-            format!("../examples/target/wasm32-unknown-unknown/release/{name}_contract.wasm"),
-            format!("../examples/target/wasm32-unknown-unknown/release/{name}_service.wasm"),
-        ))
+        for dir in ["../examples", "../linera-sdk/tests/fixtures"] {
+            let prefix = format!("{dir}/target/wasm32-unknown-unknown/release");
+            let file_contract = format!("{prefix}/{name}_contract.wasm");
+            let file_service = format!("{prefix}/{name}_service.wasm");
+            if Path::new(&file_contract).exists() && Path::new(&file_service).exists() {
+                return Ok((file_contract, file_service));
+            }
+        }
+        Err(std::io::Error::last_os_error())
     }
 
     #[cfg(with_fs)]

examples/Cargo.toml

@@ -0,0 +1,2 @@
+/target/
+.DS_Store