Skip to content

test: add MySQL client compatibility test (PHP 8.4 + MySQL 8.0)#82

Merged
AaronFeledy merged 11 commits intomainfrom
test/mysql-client-compat
Feb 19, 2026
Merged

test: add MySQL client compatibility test (PHP 8.4 + MySQL 8.0)#82
AaronFeledy merged 11 commits intomainfrom
test/mysql-client-compat

Conversation

@AaronFeledy
Copy link
Copy Markdown
Member

@AaronFeledy AaronFeledy commented Feb 19, 2026

Adds an integration test for the MySQL client auto-detection with the backdrop recipe, covering the issues reported in lando/lando#3833.

Tests:

  • MySQL client installed (not MariaDB) via auto-detection
  • No SSL/TLS certificate errors on connect
  • bee db-export works without SSL errors
  • Exported SQL uses MySQL dump format (not MariaDB dump)

Expected to fail until @lando/php ships the recipe-prefixed db_client detection fix (lando/php#223).


Note

Medium Risk
Adds a new integration test matrix entry and updates @lando/php, which can change runtime PHP/service behavior in CI and for consumers relying on the plugin’s bundled dependencies.

Overview
Adds a new MySQL-client compatibility integration test by introducing examples/backdrop-mysql-client (PHP 8.4 + MySQL 8.0) and wiring it into the GitHub Actions Leia matrix to validate client auto-detection, SSL-free connectivity, and MySQL-format exports.

Updates dependencies by bumping @lando/php to ^1.11.0 (with corresponding lockfile update) and recording the change in CHANGELOG.md.

Written by Cursor Bugbot for commit ca2ad91. This will update automatically on new commits. Configure here.

Tests that the backdrop recipe with MySQL 8.0 and PHP 8.4 correctly
auto-detects and installs the native MySQL client, verifying:
- MySQL client installed (not MariaDB)
- No SSL/TLS errors on connect
- bee db-export works without SSL errors
- Export uses MySQL dump format (not MariaDB dump)

This test is expected to fail until @lando/php includes the
recipe-prefixed db_client detection fix (lando/php#223).

Refs: lando/lando#3833
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 19, 2026

Deploy Preview for lando-backdrop ready!

Name Link
🔨 Latest commit ca2ad91
🔍 Latest deploy log https://app.netlify.com/projects/lando-backdrop/deploys/6997434cff8f5a000825547c
😎 Deploy Preview https://deploy-preview-82--lando-backdrop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 81 (🔴 down 13 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is ON. A Cloud Agent has been kicked off to fix the reported issues.

@cursor
Copy link
Copy Markdown

cursor bot commented Feb 19, 2026

Bugbot Autofix prepared fixes for 2 of the 2 bugs found in the latest run.

  • ✅ Fixed: Inverted grep on multi-line file always passes
    • Replaced grep -qiv with the correct negation pattern (grep -qi || echo $? | grep 1) that properly verifies MariaDB dump format is not present.
  • ✅ Fixed: Dump footer line defeats MariaDB negation check
    • Replaced grep -qiv with the correct negation pattern (grep -qi || echo $? | grep 1) that properly checks for MariaDB in Dump lines.

View PR

Or push these changes by commenting:

@cursor push ebfa1ff66f
Preview (ebfa1ff66f)
diff --git a/examples/backdrop-mysql-client/README.md b/examples/backdrop-mysql-client/README.md
--- a/examples/backdrop-mysql-client/README.md
+++ b/examples/backdrop-mysql-client/README.md
@@ -37,13 +37,13 @@
 
 ```bash
 # Exported SQL should use MySQL dump format (not MariaDB)
-lando exec appserver -- cat /tmp/bee-export.sql | grep -qiv "MariaDB dump"
+lando exec appserver -- cat /tmp/bee-export.sql | grep -qi "MariaDB dump" || echo $? | grep 1
# Should be able to export with db-export without SSL errors
lando db-export --stdout > /tmp/lando-export.sql
-cat /tmp/lando-export.sql | grep -i "Dump" | grep -qiv "MariaDB"
+cat /tmp/lando-export.sql | grep -i "Dump" | grep -qi "MariaDB" || echo $? | grep 1

Destroy tests


</details>

@AaronFeledy AaronFeledy merged commit 2748f4e into main Feb 19, 2026
32 checks passed
@AaronFeledy AaronFeledy deleted the test/mysql-client-compat branch February 19, 2026 17:09
@cursor cursor bot mentioned this pull request Feb 19, 2026
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant