Merge pull request #400 from wpengine/chore/wordpress-6.9-compatibility

chore: add WordPress 6.9 to test matrix

Author: josephfusco

.changeset/wordpress-69-compatibility.md

@@ -0,0 +1,5 @@
+---
+"@wpengine/wp-graphql-content-blocks": patch
+---
+
+Add WordPress 6.9 compatibility and update test matrix to support WordPress 6.9, 6.8 with PHP 8.3, 8.1.

.docker/mysql/00-create-test-db.sql

@@ -0,0 +1,5 @@
+-- Create test database for PHPUnit tests
+CREATE DATABASE IF NOT EXISTS wordpress_unit_test;
+GRANT ALL PRIVILEGES ON wordpress_unit_test.* TO 'root'@'%';
+GRANT ALL PRIVILEGES ON wordpress_unit_test.* TO 'wordpress'@'%';
+FLUSH PRIVILEGES;

.env.dist

@@ -49,6 +49,6 @@ MYSQL_USER=${DB_USER}
 MYSQL_PASSWORD=${DB_PASSWORD}
 
 # Change these to test different versions of WP, WPGraphQL, ACF etc.
-WP_VERSION=6.2
-PHP_VERSION=8.0
+WP_VERSION=6.9
+PHP_VERSION=8.1
 WPGRAPHQL_VERSION=latest

.github/actions/setup-wordpress/action.yml

@@ -1,6 +1,12 @@
 name: Set up WordPress
 description: Sets up WordPress. Assumes mariadb is available as a service.
 
+inputs:
+  wp-version:
+    description: 'WordPress version to install'
+    required: false
+    default: '6.9'
+
 runs:
   using: 'composite'
   steps:
@@ -23,6 +29,8 @@ runs:
 
     - name: Setup WordPress
       shell: bash
+      env:
+        WP_VERSION: ${{ inputs.wp-version }}
       run: |
         cp .env.dist .env
         composer run install-test-env

.github/workflows/schema-linter.yml

@@ -38,6 +38,8 @@ jobs:
 
       - name: Setup WordPress
         uses: ./.github/actions/setup-wordpress
+        with:
+          wp-version: '6.9'
 
       - name: Setup GraphQL Schema Linter
         run: npm install -g graphql-schema-linter@^3.0 graphql@^16
@@ -77,7 +79,11 @@ jobs:
           name: schema.graphql
           path: /tmp/schema.graphql
 
+      # Schema changes between WordPress versions are expected and not regressions in this plugin.
+      # The inspector runs to document changes for release notes, but doesn't block CI.
+      # Update wp-version above when releasing a new version with updated "Tested up to:" header.
       - name: Run Schema Inspector
+        continue-on-error: true
         run: |
           # This schema and previous release schema
           node_modules/.bin/graphql-inspector diff /tmp/${{ steps.get-latest-tag.outputs.tag }}.graphql /tmp/schema.graphql

.github/workflows/test-plugin-nightly.yml

@@ -16,11 +16,11 @@ jobs:
       - name: Create Docker Containers
         env:
           PHP_VERSION: 8.2
-          WP_VERSION: 6.8
+          WP_VERSION: 6.9
         working-directory: ./
         run: |
           docker compose build \
-            --build-arg WP_VERSION=6.8 \
+            --build-arg WP_VERSION=6.9 \
             --build-arg PHP_VERSION=8.2
           docker compose up -d
 

.github/workflows/test-plugin.yml

@@ -11,25 +11,8 @@ jobs:
     runs-on: ubuntu-22.04
     strategy:
       matrix:
-        php: [ '8.2', '7.4' ]
-        wordpress: [ '6.8', '6.7', '6.6','6.5', '6.4', '6.3', '6.2', '6.1' ]
-        exclude:
-        - php: 8.2
-          wordpress: 6.1
-        - php: 7.4
-          wordpress: 6.2
-        - php: 7.4
-          wordpress: 6.3
-        - php: 7.4
-          wordpress: 6.4
-        - php: 7.4
-          wordpress: 6.5
-        - php: 7.4
-          wordpress: 6.6
-        - php: 7.4
-          wordpress: 6.7
-        - php: 7.4
-          wordpress: 6.8
+        php: [ '8.3', '8.1' ]
+        wordpress: [ '6.9', '6.8', '6.7', '6.6' ]
       fail-fast: false
     name: WordPress ${{ matrix.wordpress }}, PHP ${{ matrix.php }}
     steps:

bin/_lib.sh

@@ -92,9 +92,6 @@ install_db() {
 		fi
 	fi
 
-	# Always disable SSL for local test DB connections (fixes WP 6.8 DB creation issues)
-	EXTRA="$EXTRA --ssl=false"
-
 	# create database
 	if [ $(mysql --user="$DB_USER" --password="$DB_PASS"$EXTRA --execute='show databases;' | grep ^$DB_NAME$) ]
 	then