From 00f1a0cfcb05e60cbdeb3f39cfb4d87f01c8d6de Mon Sep 17 00:00:00 2001
From: Cees-Jan Kiewiet <github@ceesjankiewiet.nl>
Date: Thu, 21 Mar 2024 21:39:19 +0100
Subject: [PATCH] Add rdbms matrix for wider compatibility testing

This change set introduces a `rdbms` matrix with the purpose of increasing insurance this package works with up to the latest MySQL/MariaDB versions.

Note that due to MySQL's versioning there are no `v6` and `v7`.

Refs: #194
---
 .github/workflows/ci.yml | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index c6c4eb8..821c3db 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -6,10 +6,12 @@ on:
 
 jobs:
   PHPUnit:
-    name: PHPUnit (PHP ${{ matrix.php }})
+    name: PHPUnit (PHP ${{ matrix.php }} + ${{ matrix.rdbms }})
     runs-on: ubuntu-24.04
     strategy:
       matrix:
+        rdbms:
+          - mysql:5
         php:
           - 8.4
           - 8.3
@@ -24,6 +26,9 @@ jobs:
           - 5.6
           - 5.5
           - 5.4
+        include:
+          - php: 8.4
+            rdbms: mariadb:10
     steps:
       - uses: actions/checkout@v4
       - uses: shivammathur/setup-php@v2
@@ -32,7 +37,7 @@ jobs:
           coverage: ${{ matrix.php < 8.0 && 'xdebug' || 'pcov' }}
           ini-file: development
       - run: composer install
-      - run: docker run -d --name mysql --net=host -e MYSQL_RANDOM_ROOT_PASSWORD=yes -e MYSQL_DATABASE=test -e MYSQL_USER=test -e MYSQL_PASSWORD=test mysql:5
+      - run: docker run -d --name mysql --net=host -e MYSQL_RANDOM_ROOT_PASSWORD=yes -e MYSQL_DATABASE=test -e MYSQL_USER=test -e MYSQL_PASSWORD=test ${{ matrix.rdbms }}
       - run: bash tests/wait-for-mysql.sh
       - run: vendor/bin/phpunit --coverage-text
         if: ${{ matrix.php >= 7.3 }}