Issue #155: Run tests on pull requests.

.github/misc/default.conf

@@ -0,0 +1,13 @@
+<VirtualHost *:80>
+  # Variables in this file will be substituted in step "Setup Apache".
+  DocumentRoot _PWD
+
+  <Directory _PWD>
+    AllowOverride All
+    Require all granted
+  </Directory>
+
+  <FilesMatch ".+\.php$">
+    SetHandler "proxy:unix:/run/php/php_PHP_V-fpm.sock|fcgi://localhost"
+  </FilesMatch>
+</VirtualHost>

.github/misc/settings.local.php

@@ -0,0 +1,8 @@
+<?php
+/**
+ * @file
+ * Custom settings for test runs.
+ */
+
+// Never send telemetry data.
+$settings['telemetry_enabled'] = FALSE;

.github/workflows/functional-tests.yml

@@ -0,0 +1,59 @@
+name: Functional Tests
+on: [pull_request]
+jobs:
+  simpletest:
+    name: Run Simpletest
+    runs-on: ubuntu-20.04
+    steps:
+      - name: Setup env
+        run: |
+          echo "REPO_NAME=${PWD##*/}" >> $GITHUB_ENV
+          echo 'USE_PHP_V=8.2' >> $GITHUB_ENV
+
+      - name: Install MariaDB
+        uses: shogo82148/actions-setup-mysql@v1
+        with:
+          mysql-version: 'mariadb-10.5'
+          root-password: 'root'
+          auto-start: true
+
+      - name: Verify setup and create database
+        run: |
+          echo -e '[client]\nuser = root\npassword = root\nhost = 127.0.0.1' > ~/.my.cnf
+          mysql -e 'SELECT version()\G'
+          mysql -e 'CREATE DATABASE backdrop;'
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ env.USE_PHP_V }}
+          coverage: none
+
+      - name: Checkout Backdrop core
+        uses: actions/checkout@v3
+        with:
+          repository: backdrop/backdrop
+
+      - name: Checkout module
+        uses: actions/checkout@v3
+        with:
+          path: modules/${{ env.REPO_NAME }}
+
+      - name: Setup Apache
+        run: |
+          sudo cp -f modules/${{ env.REPO_NAME }}/.github/misc/default.conf /etc/apache2/sites-available/000-default.conf
+          sudo sed -i -e "s?_PWD?$PWD?g" -e "s/_PHP_V/${{ env.USE_PHP_V }}/g" /etc/apache2/sites-available/000-default.conf
+          sudo apt-get install libapache2-mod-fcgid
+          sudo a2enmod rewrite proxy fcgid proxy_fcgi
+          sudo systemctl start apache2.service
+          sudo sed -i -e 's/user = www-data/user = runner/' /etc/php/${{ env.USE_PHP_V }}/fpm/pool.d/www.conf
+          sudo sed -i -e 's/listen.owner = www-data/listen.owner = runner/' /etc/php/${{ env.USE_PHP_V }}/fpm/pool.d/www.conf
+          sudo systemctl restart php${{ env.USE_PHP_V }}-fpm.service
+
+      - name: Install Backdrop
+        run: |
+          cp modules/${{ env.REPO_NAME }}/.github/misc/settings.local.php .
+          core/scripts/install.sh --db-url=mysql://root:[email protected]/backdrop
+
+      - name: Run functional tests
+        run: core/scripts/run-tests.sh --force --directory=modules/${{ env.REPO_NAME }} --verbose --color --concurrency=3 --cache --group Metatag 2>&1

metatag_app_links/tests/metatag_app_links.tags.test

@@ -4,6 +4,8 @@
  * Tests that each of the Metatag App Links tags work correctly.
  */
 
+include_once __DIR__ . '/metatag.tags.test';
+
 class MetatagAppLinksTagsTest extends MetatagTagsTestBase {
 
   /**

metatag_devel/tests/metatag_devel.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag Devel module.
  */
 
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagDevelest extends MetatagTestHelper {
 
   /**

metatag_facebook/tests/metatag_facebook.tags.test

@@ -3,6 +3,9 @@
  * @file
  * Tests that each of the Metatag Facebook tags work correctly.
  */
+
+include_once __DIR__ . '/../../tests/metatag.tags.test';
+
 class MetatagFacebookTagsTest extends MetatagTagsTestBase {
 
   /**

metatag_facebook/tests/metatag_facebook.test

@@ -3,6 +3,9 @@
  * @file
  * Tests for the Metatag Facebook module.
  */
+
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagFacebookTest extends MetatagTestHelper {
 
   /**

metatag_favicons/tests/metatag_favicons.tags.test

@@ -4,6 +4,8 @@
  * Tests that each of the Metatag Favicons tags work correctly.
  */
 
+include_once __DIR__ . '/../../tests/metatag.tags.test';
+
 class MetatagFaviconsTagsTest extends MetatagTagsTestBase {
 
   /**

metatag_favicons/tests/metatag_favicons.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag Favicons module.
  */
 
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagFaviconsTest extends MetatagTestHelper {
 
   /**

metatag_opengraph/tests/metatag_opengraph.tags.test

@@ -3,6 +3,9 @@
  * @file
  * Tests that each of the Metatag OpenGraph tags work correctly.
  */
+
+include_once __DIR__ . '/../../tests/metatag.tags.test';
+
 class MetatagOpenGraphTagsTest extends MetatagTagsTestBase {
 
   /**

metatag_opengraph/tests/metatag_opengraph.test

@@ -3,6 +3,9 @@
  * @file
  * Tests for the Metatag OpenGraph module.
  */
+
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagOpenGraphTest extends MetatagTestHelper {
 
   /**

metatag_twitter_cards/tests/metatag_twitter_cards.tags.test

@@ -3,6 +3,9 @@
  * @file
  * Tests that each of the Metatag Twitter Cards tags work correctly.
  */
+
+include_once __DIR__ . '/../../tests/metatag.tags.test';
+
 class MetatagTwitterCardsTagsTest extends MetatagTagsTestBase {
 
   /**

metatag_twitter_cards/tests/metatag_twitter_cards.test

@@ -3,6 +3,9 @@
  * @file
  * Tests for the Metatag Twitter Cards module.
  */
+
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagTwitterCardsTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

metatag_verification/tests/metatag_verification.tags.test

@@ -4,6 +4,8 @@
  * Tests that each of the Metatag Verification tags work correctly.
  */
 
+include_once __DIR__ . '/../../tests/metatag.tags.test';
+
 class MetatagVerificationTagsTest extends MetatagTagsTestBase {
 
   /**

metatag_verification/tests/metatag_verification.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag Verification module.
  */
 
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagVerificationTest extends MetatagTestHelper {
 
   /**

metatag_views/tests/metatag_views.i18n.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module for the direct Views integration.
  */
 
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagViewsI18nTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

metatag_views/tests/metatag_views.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module for the direct Views integration.
  */
 
+include_once __DIR__ . '/../../tests/metatag.helper.test';
+
 class MetatagViewsTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.bulk_revert.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure the bulk revert functionality works
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagBulkRevertTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.core_tag_removal.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure removal of core tags works correctly.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreTagRemovalTest extends MetatagTestHelper {
 
   /**

tests/metatag.image.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure image handling doesn't break.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreImageTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.locale.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure Locale integration doesn't break.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreLocaleTest extends MetatagTestHelper {
 
   /**

tests/metatag.node.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module and node entities.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreNodeTest extends MetatagTestHelper {
 
   /**

tests/metatag.node.with_i18n.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure i18n integration doesn't break.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreNodeWithI18nTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.output_caching.test

@@ -3,6 +3,8 @@
  * @file
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 /**
  * Tests for the Metatag module's output caching
  */

tests/metatag.string_handling.test

@@ -4,6 +4,8 @@
  * Contains the tests for Metatag's string handling.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 /**
  * Tests the Metatag string handling.
  */

tests/metatag.string_handling_with_i18n.test

@@ -4,6 +4,8 @@
  * Contains the tests for Metatag's string handling.
  */
 
+include_once __DIR__ . '/metatag.string_handling.test';
+
 /**
  * Tests the Metatag string handling.
  */

tests/metatag.tags.test

@@ -4,6 +4,8 @@
  * Tests that each of the Metatag base tags work correctly.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagTagsTestBase extends MetatagTestHelper {
 
   /**
@@ -103,9 +105,9 @@ class MetatagTagsTestBase extends MetatagTestHelper {
 }
 
 /**
- *
+ * Tests the built-in tags provided directly by metatag.module.
  */
-class MetatagTagsTest extends MetatagTagsTestBase {
+class MetatagBuiltInTagsTest extends MetatagTagsTestBase {
 
   /**
    * {@inheritdoc}

tests/metatag.term.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module and taxonomy term entities.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreTermTest extends MetatagTestHelper {
 
   /**

tests/metatag.term.with_i18n.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure i18n integration doesn't break.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreTermWithI18nTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.tests.info

@@ -4,8 +4,8 @@ description = Unit tests for meta tags (sort-of)
 group = Metatag
 file = metatag.unit.test
 
-[MetatagTagsTestBase]
-name = Metatag base tag tests
+[MetatagBuiltInTagsTest]
+name = Metatag built-in tags tests
 description = Tests that each of the Metatag base tags work correctly.
 group = Metatag
 file = metatag.tags.test

tests/metatag.unit.test

@@ -4,6 +4,8 @@
  * Unit tests for the Metatag module.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 /**
  * @todo These aren't really unit tests, we might need to fix that.
  */

tests/metatag.user.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module and user entities.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreUserTest extends MetatagTestHelper {
 
   /**

tests/metatag.with_i18n_config.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure i18n integration doesn't break.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreWithI18nConfigTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.with_i18n_disabled.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure i18n integration doesn't break.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreWithI18nDisabledTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.with_i18n_output.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module to ensure i18n integration doesn't break.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreWithI18nOutputTest extends MetatagTestHelper {
   /**
    * {@inheritdoc}

tests/metatag.with_me.test

@@ -4,6 +4,8 @@
  * Tests for the Metatag module for Me module integration.
  */
 
+include_once __DIR__ . '/metatag.helper.test';
+
 class MetatagCoreWithMeTest extends MetatagTestHelper {
 
   /**