ci: improving testing workflow (#46)

* test: add ci workflow for testing the bundler

* ci: enhance testing workflow

* ci: add `typos` action

* fix: rustfmt

* fix: typos

Author: hulxv

.github/workflows/bundler-tests.yml

@@ -0,0 +1,45 @@
+name: Bundler Tests
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  bundler-tests:
+    name: Bundler Integration Tests
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Build Docker image
+        run: docker build -f Dockerfile.dev -t metassr-test .
+
+      - name: Run bundler tests in Docker
+        run: |
+          docker run --rm -v $(pwd)/tests/web-app/dist:/root/tests/web-app/dist metassr-test bash -c "
+            # Run the build
+            cd /root/tests/web-app
+            npm run build
+            
+            # Copy the test script into the container and run it
+            cp /root/tests/test-bundle.sh ./
+            chmod +x test-bundle.sh
+            ./test-bundle.sh
+          "
+
+      - name: Upload dist artifacts
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: bundler-dist-output
+          path: tests/web-app/dist/
+          retention-days: 7

.github/workflows/clippy.yml

@@ -0,0 +1,40 @@
+name: Rust Clippy
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  clippy:
+    name: Run Clippy
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install Rust
+        uses: dtolnay/rust-toolchain@stable
+        with:
+          components: clippy
+
+      - name: Cache Rust dependencies
+        uses: actions/cache@v4
+        with:
+          path: |
+            ~/.cargo/bin/
+            ~/.cargo/registry/index/
+            ~/.cargo/registry/cache/
+            ~/.cargo/git/db/
+            target/
+          key: ${{ runner.os }}-cargo-clippy-${{ hashFiles('**/Cargo.lock') }}
+
+      - name: Run clippy
+        run: cargo clippy --all-targets --all-features -- -D warnings

.github/workflows/fmt.yml

@@ -0,0 +1,29 @@
+name: Rust Formatting
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  fmt:
+    name: Check Formatting
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Install Rust
+        uses: dtolnay/rust-toolchain@stable
+        with:
+          components: rustfmt
+
+      - name: Check formatting
+        run: cargo fmt --check

.github/workflows/test.yml

@@ -0,0 +1,15 @@
+name: Spell Checker action
+on: [pull_request, push]
+
+jobs:
+  run:
+    name: Start Spell Checking with Typos
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout Actions Repository
+        uses: actions/checkout@v2
+
+      - name: Check spelling of * Files
+        uses: crate-ci/typos@master
+        with:
+          files: ./*

.github/workflows/typos.yml

@@ -0,0 +1,31 @@
+name: Rust Tests
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+  workflow_dispatch:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  test:
+    name: Run Rust Tests
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Build Docker image
+        run: docker build -f Dockerfile.dev -t metassr-test .
+
+      - name: Run Rust tests in Docker
+        run: |
+          docker run --rm metassr-test bash -c "
+            cd /root
+            cargo test --verbose
+          "

.github/workflows/unit-tests.yml

@@ -1,7 +1,7 @@
 ## TODO
 
 #### Main features
-- [x] Serving staic files are located in ``./static/**``
+- [x] Serving static files are located in ``./static/**``
 
 - [x] the HTML builder
 

TODO.md

@@ -119,7 +119,7 @@ impl ManifestGenerator {
         }
     }
     pub fn generate<H: AsRef<OsStr> + ?Sized>(&self, head: &H) -> Result<Manifest> {
-        let cache_path  = self.cache.path();
+        let cache_path = self.cache.path();
         let global = GlobalEntry::new(head, cache_path)?;
         let mut manifest = Manifest::new(global);
 

crates/metassr-build/src/server/manifest.rs

@@ -79,10 +79,7 @@ impl Build for ServerSideBuilder {
         };
 
         let bundling_targets = targets.ready_for_bundling(&self.dist_path);
-        let bundler = WebBundler::new(
-            &bundling_targets,
-            &self.dist_path,
-        )?;
+        let bundler = WebBundler::new(&bundling_targets, &self.dist_path)?;
 
         if let Err(e) = bundler.exec() {
             return Err(anyhow!("Bundling failed: {e}"));

crates/metassr-build/src/server/mod.rs

@@ -51,8 +51,7 @@ impl HeadRenderer {
         let bundling_targets = self.bundling_target()?;
         let bundler = WebBundler::new(&bundling_targets, self.cache_dir.path())?;
 
-        if let Err(e) = bundler.exec()
-        {
+        if let Err(e) = bundler.exec() {
             return Err(anyhow!("Cannot bundling head: {e}"));
         }
         Ok(())