commit 80b0d44: [Test] Banish luacheck from functional tests

Andrew Lewis nerf at judo.za.org
Tue Oct 17 13:49:04 UTC 2023


Author: Andrew Lewis
Date: 2023-10-16 19:50:56 +0200
URL: https://github.com/rspamd/rspamd/commit/80b0d44da4addc96bed4eeecfc01b4e5cc36278f (refs/pull/4648/head)

[Test] Banish luacheck from functional tests
 - Use luacheck from GitLab

---
 .drone.jsonnet                           |  8 ++++++++
 .drone.yml                               | 10 +++++++++-
 test/functional/cases/200_luacheck.robot | 10 ----------
 3 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/.drone.jsonnet b/.drone.jsonnet
index 694a63ef2..ef0f56c61 100644
--- a/.drone.jsonnet
+++ b/.drone.jsonnet
@@ -287,6 +287,14 @@ local noarch_pipeline = {
         './node_modules/.bin/stylelint ./**/*.css ./**/*.html ./**/*.js',
       ],
     },
+    {
+      name: 'luacheck',
+      image: 'pipelinecomponents/luacheck',
+      pull: 'if-not-exists',
+      commands: [
+        'luacheck -q --no-color .',
+      ],
+    },
   ],
 } + default_trigger + docker_pipeline;
 
diff --git a/.drone.yml b/.drone.yml
index 74231368a..196dbf9ad 100644
--- a/.drone.yml
+++ b/.drone.yml
@@ -499,6 +499,14 @@
          "image": "node:18-alpine",
          "name": "eslint",
          "pull": "if-not-exists"
+      },
+      {
+         "commands": [
+            "luacheck -q --no-color ."
+         ],
+         "image": "pipelinecomponents/luacheck",
+         "name": "luacheck",
+         "pull": "if-not-exists"
       }
    ],
    "trigger": {
@@ -553,7 +561,7 @@
 }
 ---
 {
-   "hmac": "3f01ceb14c7dd3ba2f5343fc52216843f72212e1cb25f944c2366d57e78d2b79",
+   "hmac": "6c89d1423ccbddbfac9e2b67f24192018670209a293cfee308880a16a2f2b372",
    "kind": "signature"
 }
 ...
diff --git a/test/functional/cases/200_luacheck.robot b/test/functional/cases/200_luacheck.robot
deleted file mode 100644
index a38fe083e..000000000
--- a/test/functional/cases/200_luacheck.robot
+++ /dev/null
@@ -1,10 +0,0 @@
-*** Settings ***
-Library         Process
-
-*** Test Cases ***
-Lua Check
-  ${result} =  Run Process  luacheck  -q  --no-color  .  cwd=${TOPDIR}
-  IF  ${result.rc} != 0
-    Log  ${result.stdout}
-  END
-  Should Be Equal As Integers  ${result.rc}  0  msg=${result.stdout}  values=false


More information about the Commits mailing list