From 0de48617ec0913ea508854fb4bee3c99074e1ea9 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Fri, 30 Oct 2020 09:46:53 +0100
Subject: [PATCH] Only run phpunit when php, xml or test files are modified

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 .drone.yml                           | 21 +++++++++++++++++++++
 tests/drone-run-integration-tests.sh |  2 ++
 tests/drone-run-php-tests.sh         | 19 +++++++++++++++++++
 3 files changed, 42 insertions(+)
 create mode 100755 tests/drone-run-php-tests.sh

diff --git a/.drone.yml b/.drone.yml
index cadcc0d45a9..aa718998203 100644
--- a/.drone.yml
+++ b/.drone.yml
@@ -249,14 +249,17 @@ steps:
 - name: nodb-php7.3
   image: nextcloudci/php7.3:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=NODB ./autotest.sh sqlite
 - name: nodb-php7.4
   image: nextcloudci/php7.4:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=NODB ./autotest.sh sqlite
 #- name: nodb-php8.0
 #  image: nextcloudci/php8.0:latest
 #  commands:
+#    - bash tests/drone-run-php-tests.sh || exit 0
 #    - NOCOVERAGE=true TEST_SELECTION=NODB ./autotest.sh sqlite
 
 services:
@@ -283,14 +286,17 @@ steps:
 - name: sqlite-php7.3
   image: nextcloudci/php7.3:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh sqlite
 - name: sqlite-php7.4
   image: nextcloudci/php7.4:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh sqlite
 #- name: sqlite-php8.0
 #  image: nextcloudci/php8.0:latest
 #  commands:
+#    - bash tests/drone-run-php-tests.sh || exit 0
 #    - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh sqlite
 
 services:
@@ -317,6 +323,7 @@ steps:
 - name: mariadb10.1-php7.3
   image: nextcloudci/php7.3:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mariadb
 
 services:
@@ -352,6 +359,7 @@ steps:
 - name: mariadb10.2-php7.3
   image: nextcloudci/php7.3:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mariadb
 
 services:
@@ -386,6 +394,7 @@ steps:
 - name: mariadb10.3-php7.3
   image: nextcloudci/php7.3:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mariadb
 
 services:
@@ -421,10 +430,12 @@ steps:
 - name: mariadb10.4-php7.4
   image: nextcloudci/php7.4:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mariadb
 #- name: mariadb10.4-php8.0
 #  image: nextcloudci/php8.0:latest
 #  commands:
+#    - bash tests/drone-run-php-tests.sh || exit 0
 #    - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mariadb
 
 services:
@@ -460,6 +471,7 @@ steps:
 - name: mysql-php7.4
   image: nextcloudci/php7.4:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mysql
 
 services:
@@ -496,6 +508,7 @@ steps:
 - name: mysql-php7.4
   image: nextcloudci/php7.4:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mysql
 
 services:
@@ -528,6 +541,7 @@ steps:
 - name: mysql-php7.3
   image: nextcloudci/php7.3:php7.3-4
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mysql
 
 services:
@@ -560,6 +574,7 @@ steps:
 - name: mysql5.6-php7.3
   image: nextcloudci/php7.3:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mysql
 
 services:
@@ -594,6 +609,7 @@ steps:
 - name: postgres-php7.3
   image: nextcloudci/php7.3:php7.3-4
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - sleep 10 # gives the database enough time to initialize
     - POSTGRES=9 NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh pgsql
 
@@ -629,6 +645,7 @@ steps:
 - name: postgres-php7.3
   image: nextcloudci/php7.3:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - sleep 10 # gives the database enough time to initialize
     - POSTGRES=10 NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh pgsql
 
@@ -663,6 +680,7 @@ steps:
 - name: postgres-php7.4
   image: nextcloudci/php7.4:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - sleep 10 # gives the database enough time to initialize
     - POSTGRES=11 NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh pgsql
 
@@ -699,10 +717,12 @@ steps:
 - name: mysqlmb4-php7.4
   image: nextcloudci/php7.4:latest
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mysqlmb4
 #- name: mysqlmb4-php8.0
 #  image: nextcloudci/php8.0:latest
 #  commands:
+#    - bash tests/drone-run-php-tests.sh || exit 0
 #    - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mysqlmb4
 
 services:
@@ -738,6 +758,7 @@ steps:
 - name: mysqlmb4-php7.3
   image: nextcloudci/php7.3:php7.3-4
   commands:
+    - bash tests/drone-run-php-tests.sh || exit 0
     - NOCOVERAGE=true TEST_SELECTION=DB ./autotest.sh mysqlmb4
 
 services:
diff --git a/tests/drone-run-integration-tests.sh b/tests/drone-run-integration-tests.sh
index 9da224ccbad..605e1eb356b 100755
--- a/tests/drone-run-integration-tests.sh
+++ b/tests/drone-run-integration-tests.sh
@@ -8,6 +8,8 @@ echo "========================="
 
 [[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | wc -l) -eq 0 ]] && echo "No files are modified => merge commit" && exit 0
 
+[[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | grep -c ".xml") -gt 0 ]] && echo "info.xml files are modified" && exit 0
+
 [[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | grep -c ".php$") -gt 0 ]] && echo "PHP files are modified" && exit 0
 
 [[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | grep -c "^build/integration/") -gt 0 ]] && echo "Integration test files are modified" && exit 0
diff --git a/tests/drone-run-php-tests.sh b/tests/drone-run-php-tests.sh
new file mode 100755
index 00000000000..9552141adcb
--- /dev/null
+++ b/tests/drone-run-php-tests.sh
@@ -0,0 +1,19 @@
+#!/bin/bash
+
+echo "========================="
+echo "= List of changed files ="
+echo "========================="
+git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA
+echo "========================="
+
+[[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | wc -l) -eq 0 ]] && echo "No files are modified => merge commit" && exit 0
+
+[[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | grep -c ".xml") -gt 0 ]] && echo "info.xml files are modified" && exit 0
+
+[[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | grep -c ".php$") -gt 0 ]] && echo "PHP files are modified" && exit 0
+
+[[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | grep -c "^tests/") -gt 0 ]] && echo "PHP test files are modified" && exit 0
+
+[[ $(git diff --name-only origin/$DRONE_TARGET_BRANCH...$DRONE_COMMIT_SHA | grep -c "/tests/") -gt 0 ]] && echo "PHP test files of an app are modified" && exit 0
+
+exit 1
-- 
GitLab