CI: outsource Godot test to its own composite action

Author: Bromeon

.github/composite/godot/action.yml

@@ -0,0 +1,82 @@
+name: godot
+description: "Run Godot integration tests"
+
+inputs:
+  godot_ver:
+    required: true
+    description: "Godot version (e.g. '3.2')"
+
+  rust_toolchain:
+    required: false
+    default: 'stable'
+    description: "Rust toolchain specifier (e.g. 'nightly')"
+
+  rust_extra_args:
+    required: false
+    default: ''
+    description: "Extra command line arguments for 'cargo build', e.g. features"
+
+
+runs:
+  using: "composite"
+  steps:
+    - uses: actions/checkout@v2
+    - name: Install Rust
+      uses: ./.github/composite/rust
+      with:
+        rust: ${{ inputs.rust_toolchain }}
+    - name: "Check cache for installed Godot version"
+      id: "cache-godot"
+      uses: actions/cache@v2
+      with:
+        path: ${{ runner.temp }}/godot_bin
+        key: godot-${{ runner.os }}-v${{ inputs.godot_ver }}
+    - name: "Install Godot"
+      if: steps.cache-godot.outputs.cache-hit != 'true'
+      run: |
+        wget --no-verbose "https://downloads.tuxfamily.org/godotengine/${{ inputs.godot_ver }}/Godot_v${{ inputs.godot_ver }}-stable_linux_headless.64.zip" -O /tmp/godot.zip
+        unzip -o /tmp/godot.zip -d ${{ runner.temp }}/godot_bin
+      shell: bash
+    - name: "Set environment variable"
+      run: |
+        echo "GODOT_BIN=${{ runner.temp }}/godot_bin/Godot_v${{ inputs.godot_ver }}-stable_linux_headless.64" >> $GITHUB_ENV
+      shell: bash
+    - name: "Build godot-rust"
+      run: |
+        echo "File size of api.json -- before:"
+        stat -c %s gdnative-bindings/api.json
+        
+        #echo "CRC32 of api.json: "
+        #crc32 gdnative-bindings/api.json
+        cd test
+        cargo build ${{ inputs.rust_extra_args }}
+        
+        cd ..
+        echo "File size of api.json -- after:"
+        stat -c %s gdnative-bindings/api.json
+      shell: bash
+    - name: "Run Godot integration tests"
+      run: |
+        cd test;
+        mkdir -p ./project/lib;
+        cp ../target/debug/libgdnative_test.so ./project/lib/;
+        ${GODOT_BIN} --path ./project/ > >(tee "${{ runner.temp }}/stdout.log");
+        if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
+          exit 1;
+        fi;
+        ${GODOT_BIN} -e --path ./project/ --run-editor-tests > >(tee "${{ runner.temp }}/stdout.log");
+        if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
+          exit 1;
+        fi;
+        cargo build --features=type_tag_fallback;
+        mkdir -p ./project/lib;
+        cp ../target/debug/libgdnative_test.so ./project/lib/;
+        ${GODOT_BIN} --path ./project/ > >(tee "${{ runner.temp }}/stdout.log");
+        if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
+          exit 1;
+        fi;
+        ${GODOT_BIN} -e --path ./project/ --run-editor-tests > >(tee "${{ runner.temp }}/stdout.log");
+        if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
+          exit 1;
+        fi;
+      shell: bash

.github/workflows/full-ci.yml

@@ -32,8 +32,6 @@ env:
 
   # Local variables
   # Note: using variables is limited at the moment, see https://github.com/actions/runner/issues/480
-  GODOT_VER: "3.4"
-  GODOT_REL: stable
   GDRUST_FEATURES: "gdnative/async,gdnative/serde"
 
 on:
@@ -51,11 +49,11 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         with:
           components: rustfmt
-      - name: Check rustfmt
+      - name: "Check rustfmt"
         run: cargo fmt --all -- --check;
 
   clippy:
@@ -72,12 +70,12 @@ jobs:
             postfix: ' (nightly)'
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         with:
           rust: ${{ matrix.rust.toolchain }}
           components: clippy
-      - name: Check clippy
+      - name: "Check clippy"
         run: cargo clippy --workspace --features ${GDRUST_FEATURES} -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
 
   test:
@@ -112,16 +110,16 @@ jobs:
     runs-on: ${{ matrix.os.id }}
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         with:
           rust: ${{ matrix.rust.toolchain }}
-      - name: Install LLVM
+      - name: "Install LLVM"
         uses: ./.github/composite/llvm
         if: ${{ matrix.os.id == 'windows-latest' }}
-      - name: Compile tests
+      - name: "Compile tests"
         run: cargo test --workspace --features ${GDRUST_FEATURES} --no-run;
-      - name: Test
+      - name: "Test"
         run: cargo test --workspace --features ${GDRUST_FEATURES} ${{ matrix.testflags }};
 
   build-release:
@@ -140,14 +138,14 @@ jobs:
     runs-on: ${{ matrix.os.id }}
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         with:
           rust: stable
-      - name: Install LLVM
+      - name: "Install LLVM"
         uses: ./.github/composite/llvm
         if: ${{ matrix.os.id == 'windows-latest' }}
-      - name: Release build (check only)
+      - name: "Release build (check only)"
         run: cargo check --release;
 
   build-ios:
@@ -159,18 +157,18 @@ jobs:
     runs-on: macos-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         #with:
         #  rust: ${{ matrix.rust.toolchain }}
-      - name: Install cargo-dinghy
+      - name: "Install cargo-dinghy"
         run: |
           rustup target add x86_64-apple-ios;
           curl -L https://github.com/sonos/dinghy/releases/download/0.4.62/cargo-dinghy-macos-0.4.62.tgz -o cargo-dinghy-macos.tar.gz;
           tar -zxvf cargo-dinghy-macos.tar.gz;
           mkdir -p $HOME/.cargo/bin;
           cp cargo-dinghy-0.4.62/cargo-dinghy $HOME/.cargo/bin;
-      - name: Cross-compile to iOS
+      - name: "Cross-compile to iOS"
         run: |
           RUNTIME_ID=$(xcrun simctl list runtimes | grep iOS | cut -d ' ' -f 7 | tail -1);
           export SIM_ID=$(xcrun simctl create My-iphone11 com.apple.CoreSimulator.SimDeviceType.iPhone-11 $RUNTIME_ID);
@@ -191,11 +189,11 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         #with:
         #  rust: ${{ matrix.rust.toolchain }}
-      - name: Install Java + NDK
+      - name: "Install Java + NDK"
         run: |
           export JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64;
           export ANDROID_SDK_ROOT=/opt/ndk/android-ndk-r21d;
@@ -228,65 +226,38 @@ jobs:
           cargo build --target armv7-linux-androideabi --release;
 
   integration-test-godot:
-    name: itest-godot${{ matrix.rust.postfix }}
+    name: itest-godot-${{ matrix.godot }}${{ matrix.postfix }}
     needs: rustfmt
     continue-on-error: ${{ matrix.rust.toolchain == 'nightly' }}
     strategy:
+      fail-fast: false # cancel all jobs as soon as one fails?
       matrix:
-        rust:
-          - toolchain: stable
+        include:
+          - rust: stable
+            godot: "3.4.1"
             postfix: ''
-          - toolchain: nightly
+          - rust: nightly
+            godot: "3.4.1"
             postfix: ' (nightly)'
-          - toolchain: '1.48'
+          - rust: '1.48'
+            godot: "3.4.1"
             postfix: ' (msrv 1.48)'
+          - rust: stable
+            godot: "3.2"
+            postfix: ''
+            build_args: '--features custom-godot'
+
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
-        uses: ./.github/composite/rust
+      - name: "Run Godot integration test"
+        uses: ./.github/composite/godot
         with:
-          rust: ${{ matrix.rust.toolchain }}
-      - name: Check cache for installed Godot version
-        id: cache-godot
-        uses: actions/cache@v2
-        with:
-          path: ${{ runner.temp }}/godot_bin
-          key: godot-${{ runner.os }}-v${{ env.GODOT_VER }}-${{ env.GODOT_REL }}
-      - name: Install Godot
-        if: steps.cache-godot.outputs.cache-hit != 'true'
-        run: |
-          wget "https://downloads.tuxfamily.org/godotengine/$GODOT_VER/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64.zip" -O /tmp/godot.zip
-          unzip /tmp/godot.zip -d ${{ runner.temp }}/godot_bin
-      - name: Build godot-rust
-        run: |
-          cd test;
-          cargo build;
-      - name: Run Godot integration tests
-        run: |
-          cd test;
-          mkdir -p ./project/lib;
-          cp ../target/debug/libgdnative_test.so ./project/lib/;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" --path ./project/ > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" -e --path ./project/ --run-editor-tests > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          cargo build --features=type_tag_fallback;
-          mkdir -p ./project/lib;
-          cp ../target/debug/libgdnative_test.so ./project/lib/;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" --path ./project/ > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" -e --path ./project/ --run-editor-tests > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          
+          rust_toolchain: ${{ matrix.rust }}
+          rust_extra_args: ${{ matrix.build_args }}
+          godot_ver: ${{ matrix.godot }}
+
+
   # This job doesn't actually test anything, but is used to tell bors that the build completed,
   # as there is no practical way to detect when a workflow is successful, listening to webhooks only.
   # The ID (not name) of this job is the one referenced in bors.toml.
@@ -304,5 +275,5 @@ jobs:
       - build-android
     runs-on: ubuntu-latest
     steps:
-      - name: Mark the job as a success
+      - name: "Mark the job as a success"
         run: exit 0

.github/workflows/minimal-ci.yml

@@ -12,8 +12,7 @@ env:
 
   # Local variables
   # Note: using variables is limited at the moment, see https://github.com/actions/runner/issues/480
-  GODOT_VER: "3.4"
-  GODOT_REL: stable
+  GODOT_VER: "3.4.1"
   GDRUST_FEATURES: "gdnative/async,gdnative/serde"
 
 on:
@@ -35,84 +34,46 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         with:
           rust: stable
           components: rustfmt
-      - name: Check rustfmt
+      - name: "Check rustfmt"
         run: cargo fmt --all -- --check;
 
   clippy:
     runs-on: ubuntu-latest
     continue-on-error: ${{ matrix.rust == 'nightly' }}
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
         with:
           rust: stable
           components: clippy
-      - name: Check clippy
+      - name: "Check clippy"
         run: cargo clippy --workspace --features ${GDRUST_FEATURES} -- -D clippy::style -D clippy::complexity -D clippy::perf -D clippy::dbg_macro -D clippy::todo -D clippy::unimplemented;
 
   unit-test:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
+      - name: "Install Rust"
         uses: ./.github/composite/rust
-      - name: Compile tests
+      - name: "Compile tests"
         run: cargo test --workspace --features ${GDRUST_FEATURES} --no-run;
-      - name: Test
+      - name: "Test"
         run: cargo test --workspace --features ${GDRUST_FEATURES};
 
   integration-test-godot:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install Rust
-        uses: ./.github/composite/rust
-      - name: Check cache for installed Godot version
-        id: cache-godot
-        uses: actions/cache@v2
+      - name: "Run Godot integration test"
+        uses: ./.github/composite/godot
         with:
-          path: ${{ runner.temp }}/godot_bin
-          key: godot-${{ runner.os }}-v${{ env.GODOT_VER }}-${{ env.GODOT_REL }}
-      - name: Install Godot
-        if: steps.cache-godot.outputs.cache-hit != 'true'
-        run: |
-          wget "https://downloads.tuxfamily.org/godotengine/$GODOT_VER/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64.zip" -O /tmp/godot.zip
-          unzip /tmp/godot.zip -d ${{ runner.temp }}/godot_bin
-      - name: Build godot-rust
-        run: |
-          cd test;
-          cargo build;
-      - name: Run Godot integration tests
-        run: |
-          cd test;
-          mkdir -p ./project/lib;
-          cp ../target/debug/libgdnative_test.so ./project/lib/;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" --path ./project/ > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" -e --path ./project/ --run-editor-tests > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          cargo build --features=type_tag_fallback;
-          mkdir -p ./project/lib;
-          cp ../target/debug/libgdnative_test.so ./project/lib/;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" --path ./project/ > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          "${{ runner.temp }}/godot_bin/Godot_v${GODOT_VER}-${GODOT_REL}_linux_headless.64" -e --path ./project/ --run-editor-tests > >(tee "${{ runner.temp }}/stdout.log");
-          if grep -q "Leaked instance" "${{ runner.temp }}/stdout.log"; then
-            exit 1;
-          fi;
-          
+          godot_ver: ${{ env.GODOT_VER }}
 
 # Not really needed, since bors is not involved. Just needs an extra runner and makes the tests run longer.