fix: allow env values to be empty/null (#10)

Author: woodruffw

src/common.rs

@@ -55,7 +55,7 @@ pub enum Permission {
 }
 
 /// An environment mapping.
-pub type Env = HashMap<String, EnvValue>;
+pub type Env = HashMap<String, Option<EnvValue>>;
 
 /// Environment variable values are always strings, but GitHub Actions
 /// allows users to configure them as various native YAML types before

src/workflow/job.rs

@@ -176,7 +176,10 @@ mod tests {
         let Secrets::Env(secrets) = serde_yaml::from_str::<Secrets>(secrets).unwrap() else {
             panic!("unexpected secrets variant");
         };
-        assert_eq!(secrets["foo-secret"], EnvValue::String("bar".into()));
+        assert_eq!(
+            secrets["foo-secret"].as_ref().unwrap(),
+            &EnvValue::String("bar".into())
+        );
     }
 
     #[test]

tests/sample-workflows/adafruit-circuitpython-run-tests.yml

@@ -0,0 +1,69 @@
+# https://github.com/adafruit/circuitpython/blob/24a8927d0ce1d77addd584bf809ab73cb0386e75/.github/workflows/run-tests.yml
+name: Run tests
+
+on:
+  workflow_call:
+    inputs:
+      cp-version:
+        required: true
+        type: string
+
+jobs:
+  run:
+    runs-on: ubuntu-24.04
+    strategy:
+      fail-fast: false
+      matrix:
+        test: [all, mpy, native, native_mpy]
+    env:
+      CP_VERSION: ${{ inputs.cp-version }}
+      MICROPY_CPYTHON3: python3.12
+      MICROPY_MICROPYTHON: ../ports/unix/build-coverage/micropython
+      TEST_all:
+      TEST_mpy: --via-mpy -d basics float micropython
+      TEST_native: --emit native
+      TEST_native_mpy: --via-mpy --emit native -d basics float micropython
+    steps:
+      - name: Set up repository
+        uses: actions/checkout@v4
+        with:
+          submodules: false
+          show-progress: false
+          fetch-depth: 1
+      - name: Set up python
+        uses: actions/setup-python@v5
+        with:
+          python-version: 3.12
+      - name: Set up submodules
+        uses: ./.github/actions/deps/submodules
+        with:
+          target: tests
+      - name: Set up external
+        if: matrix.test == 'all'
+        uses: ./.github/actions/deps/external
+      - name: Set up mpy-cross
+        uses: ./.github/actions/mpy_cross
+        with:
+          cp-version: ${{ inputs.cp-version }}
+      - name: Build unix port
+        run: make -C ports/unix VARIANT=coverage -j4
+      - name: Run tests
+        run: ./run-tests.py -j4 ${{ env[format('TEST_{0}', matrix.test)] }}
+        working-directory: tests
+      - name: Print failure info
+        run: ./run-tests.py -j4 --print-failures
+        if: failure()
+        working-directory: tests
+    # Not working after MicroPython v1.23 merge.
+    # - name: Build native modules
+    #   if: matrix.test == 'all'
+    #   run: |
+    #     make -C examples/natmod/features1
+    #     make -C examples/natmod/features2
+    #     make -C examples/natmod/heapq
+    #     make -C examples/natmod/random
+    #     make -C examples/natmod/re
+    # - name: Test native modules
+    #   if: matrix.test == 'all'
+    #   run: ./run-natmodtests.py extmod/{heapq*,random*,re*}.py
+    #   working-directory: tests

tests/test_workflow.rs

@@ -55,9 +55,15 @@ fn test_pip_audit_ci() {
         panic!("expected uses step");
     };
     assert_eq!(uses, "actions/setup-python@v5");
-    assert_eq!(with["python-version"].to_string(), "${{ matrix.python }}");
-    assert_eq!(with["cache"].to_string(), "pip");
-    assert_eq!(with["cache-dependency-path"].to_string(), "pyproject.toml");
+    assert_eq!(
+        with["python-version"].as_ref().unwrap().to_string(),
+        "${{ matrix.python }}"
+    );
+    assert_eq!(with["cache"].as_ref().unwrap().to_string(), "pip");
+    assert_eq!(
+        with["cache-dependency-path"].as_ref().unwrap().to_string(),
+        "pyproject.toml"
+    );
 
     let StepBody::Run {
         run,