From ff87cf3162750d50b74ae04b2298b896dad165e0 Mon Sep 17 00:00:00 2001
From: sfan5 <sfan5@live.de>
Date: Fri, 16 Aug 2024 17:10:25 +0200
Subject: [PATCH] Add smoke test with engine using Docker

---
 .../{check-release.yml => check.yml}          |  8 ++---
 .github/workflows/test.yml                    | 26 ++++++++++++++++
 .test/minetest.conf                           |  3 ++
 .test/run.sh                                  | 31 +++++++++++++++++++
 mesecons/init.lua                             |  8 +++++
 5 files changed, 72 insertions(+), 4 deletions(-)
 rename .github/workflows/{check-release.yml => check.yml} (94%)
 create mode 100644 .github/workflows/test.yml
 create mode 100644 .test/minetest.conf
 create mode 100755 .test/run.sh

diff --git a/.github/workflows/check-release.yml b/.github/workflows/check.yml
similarity index 94%
rename from .github/workflows/check-release.yml
rename to .github/workflows/check.yml
index 616adb3..3353325 100644
--- a/.github/workflows/check-release.yml
+++ b/.github/workflows/check.yml
@@ -1,10 +1,10 @@
 on: [push, pull_request]
-name: Check & Release
+name: "Check"
 
 jobs:
   lint:
     runs-on: ubuntu-latest
-
+    name: "Luacheck"
     steps:
     - uses: actions/checkout@main
     - name: apt
@@ -14,9 +14,9 @@ jobs:
     - name: luacheck run
       run: $HOME/.luarocks/bin/luacheck ./
 
-  test:
+  mineunit:
     runs-on: ubuntu-latest
-
+    name: "Mineunit tests"
     steps:
     - uses: actions/checkout@main
     - name: apt
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
new file mode 100644
index 0000000..300025e
--- /dev/null
+++ b/.github/workflows/test.yml
@@ -0,0 +1,26 @@
+on: [push, pull_request]
+name: "Test"
+
+jobs:
+  test:
+    name: "Smoke Test ${{ matrix.cfg.image }}"
+    runs-on: ubuntu-latest
+    timeout-minutes: 5
+    strategy:
+      matrix:
+        cfg:
+          - { image: 'registry.gitlab.com/minetest/minetest/server:5.0.1', mtg: false }
+          - { image: 'ghcr.io/minetest/minetest:5.9.0', mtg: true }
+    steps:
+    - uses: actions/checkout@main
+
+    - uses: actions/checkout@main
+      with:
+        repository: 'minetest/minetest_game'
+        path: ./.test/minetest_game
+      if: ${{ matrix.cfg.mtg }}
+
+    - name: Run tests
+      run: ./.test/run.sh
+      env:
+        DOCKER_IMAGE: "${{ matrix.cfg.image }}"
diff --git a/.test/minetest.conf b/.test/minetest.conf
new file mode 100644
index 0000000..7e7b767
--- /dev/null
+++ b/.test/minetest.conf
@@ -0,0 +1,3 @@
+mg_name = singlenode
+mesecon.internal_test = true
+random_mod_load_order = true
diff --git a/.test/run.sh b/.test/run.sh
new file mode 100755
index 0000000..eb8c198
--- /dev/null
+++ b/.test/run.sh
@@ -0,0 +1,31 @@
+#!/bin/bash
+tempdir=$(mktemp -d)
+confpath=$tempdir/minetest.conf
+worldpath=$tempdir/world
+trap 'rm -rf "$tempdir" || :' EXIT
+
+[ -f mesecons/mod.conf ] || { echo "Must be run in modpack root folder." >&2; exit 1; }
+
+command -v docker >/dev/null || { echo "Docker is not installed." >&2; exit 1; }
+mtg=.test/minetest_game
+[ -d $mtg ] || echo "A source checkout of minetest_game was not found. This can fail if your docker image does not ship a game." >&2
+
+mkdir "$worldpath"
+cp -v .test/minetest.conf "$confpath"
+chmod -R 777 "$tempdir"
+
+args=(
+	-v "$confpath":/etc/minetest/minetest.conf
+	-v "$tempdir":/var/lib/minetest/.minetest
+	-v "$PWD":/var/lib/minetest/.minetest/world/worldmods/mesecons
+)
+[ -d $mtg ] && args+=(
+	-v "$(realpath $mtg)":/var/lib/minetest/.minetest/games/minetest_game
+)
+args+=("$DOCKER_IMAGE")
+[ -d $mtg ] && args+=(--gameid minetest)
+docker run --rm -i "${args[@]}"
+
+ls -la "$worldpath"
+test -f "$worldpath/mesecon_actionqueue" || exit 1
+exit 0
diff --git a/mesecons/init.lua b/mesecons/init.lua
index 3446004..9a45a40 100644
--- a/mesecons/init.lua
+++ b/mesecons/init.lua
@@ -123,3 +123,11 @@ dofile(minetest.get_modpath("mesecons").."/legacy.lua");
 
 --Services like turnoff receptor on dignode and so on
 dofile(minetest.get_modpath("mesecons").."/services.lua");
+
+-- Automated test run
+if mesecon.setting("internal_test", false) then
+	-- currently does nothing, we only fail if some error happens right on startup
+	minetest.after(5, function()
+		minetest.request_shutdown()
+	end)
+end