refactor: push docker images only on main, and test docker images for… (

#469)

* refactor: push docker images only on main, and test docker images for avatar testing on every PR

* fix

* test

* try more

* fix

* test

* test

* w

* better desc

* remove .godot folder that was not removed correctly

* remove docker tests file

.github/workflows/docker_builds.yml → .github/workflows/docker_build_test.yml

@@ -1,4 +1,4 @@
-name: 🐳 Docker builds
+name: 🐳 Docker builds and test
 on:
   workflow_call:
     secrets:
@@ -13,7 +13,7 @@ concurrency:
 
 jobs:
   build-images:
-    name: Build images
+    name: Build images and test
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout sources
@@ -28,14 +28,14 @@ jobs:
         uses: redhat-actions/buildah-build@v2
         with:
           image: godot-explorer
-          tags: ${{ github.sha }}
+          tags: latest ${{ github.sha }}
           dockerfiles: |
             ./Dockerfile
           build-args: |
-            ${{ inputs.build-args }}
             COMMIT_HASH=${{ github.sha }}
 
       - name: Push To quay.io
+        if: github.ref == 'refs/heads/main' # TODO: Push only on tags / releases?
         id: push-to-quay
         uses: redhat-actions/push-to-registry@v2
         with:
@@ -45,5 +45,23 @@ jobs:
           username: ${{ secrets.QUAY_USERNAME }}
           password: ${{ secrets.QUAY_TOKEN }}
 
-      - name: Print image url
-        run: echo "Image pushed to ${{ steps.push-to-quay.outputs.registry-paths }}"
+      - name: Run avatar test
+        run: |
+          mkdir -p output
+          podman run --rm -v \
+            $(pwd)/tests/avatars-test-input.json:/app/avatars.json \
+            -v $(pwd)/output:/app/output localhost/godot-explorer:latest
+
+      - name: Compare images with snapshots
+        run: |
+          cargo run -- compare-image-folders \
+            --snapshots tests/snapshots/avatar-image-generation/ \
+            --result ${{ github.workspace }}/output/
+
+      - name: Upload artifacts
+        if: success() || failure()
+        uses: actions/upload-artifact@v4
+        with:
+          name: avatar-snapshots
+          path: |
+            ${{ github.workspace }}/output/**/*

.github/workflows/linux_builds.yml

@@ -28,11 +28,10 @@ jobs:
       - name: Install dependencies
         uses: ./.github/actions/install-deps
 
-      - name: Install
+      # Build section
+      - name: Cargo install
         run: cargo run -- install
 
-
-      # Build section
       - name: Build
         working-directory: lib
         run: cargo build --release

.github/workflows/macos_builds.yml

@@ -26,10 +26,10 @@ jobs:
       - name: Install dependencies
         uses: ./.github/actions/install-deps
 
-      - name: Install
+      # Build section
+      - name: Cargo install
         run: cargo run -- install
 
-      # Build section
       - name: Build
         working-directory: lib
         run: cargo build --release

.github/workflows/runner.yml

@@ -41,17 +41,11 @@ jobs:
     needs: static-checks
     uses: ./.github/workflows/windows_builds.yml
 
-  # Third stage: Extra ci
-  docker-build:
-    name: 🐳 Docker builds
+  # Third stage: Extras
+  docker-build-test:
+    name: 🐳 Docker builds and test
     needs: linux-build
-    uses: ./.github/workflows/docker_builds.yml
-    # if: github.ref == 'refs/heads/main'
+    uses: ./.github/workflows/docker_build_test.yml
     secrets:
       QUAY_USERNAME: ${{ secrets.QUAY_USERNAME }}
       QUAY_TOKEN: ${{ secrets.QUAY_TOKEN }}
-
-  docker-tests:
-    name: 🧪 Docker tests
-    needs: docker-build
-    uses: ./.github/workflows/docker_tests.yml

.github/workflows/windows_builds.yml

@@ -29,11 +29,10 @@ jobs:
       - name: Install dependencies
         uses: ./.github/actions/install-deps
 
-      - name: Install
+      # Build section
+      - name: Cargo install
         run: cargo run -- install
 
-
-      # Build section
       - name: Build
         working-directory: lib
         run: cargo build --release

godot/.gitignore

@@ -1,7 +1,7 @@
 # Godot 4+ specific ignores
 /.godot/
 
-android/
+/android/
 
 # Godot-specific ignores
 .import/