ci: fix MySQL version detection for ORDER BY placeholder parameter count on 2052 unit test (#3928)

* fix(tests): improve hasIncorrectPrepareParameter method

* fix(test): >= 9 instead of just > 9

* fix(test): fix hasIncorrectPrepareParameter method to target only known versions intead. uncomment ci pipeline to test against 8.4 of MySQL

Author: HappyZombies

.github/workflows/ci-mysql.yml

@@ -24,7 +24,7 @@ jobs:
             'mysql:8.1',
             'mysql:8.2',
             # 'mysql:8.3', # Already tested in "ci-linux"
-            # 'mysql:8.4', # TODO: Tests never end
+            'mysql:8.4',
             # 'mysql:9.0', # Already tested in "ci-coverage"
             'mysql:9.1',
             'mysql:9.2',

test/esm/regressions/2052.test.mjs

@@ -100,25 +100,10 @@ test(async () => {
   const mySqlVersion = await common.getMysqlVersion(connection);
 
   const hasIncorrectPrepareParameter = (() => {
+    const incorrectVersions = ['8.1.0', '8.2.0', '8.3.0', '8.4.0'];
     const { major, minor, patch } = mySqlVersion;
-
-    if (major === 9) return false;
-    if (major === 8 && minor === 4 && patch === 1) return false;
-    if (major === 8 && minor === 0 && patch >= 38) return false;
-
-    if (major > 8) {
-      return true;
-    }
-
-    if (major === 8 && minor > 0) {
-      return true;
-    }
-
-    if (major === 8 && minor === 0 && patch >= 22) {
-      return true;
-    }
-
-    return false;
+    const verString = `${major}.${minor}.${patch}`;
+    return incorrectVersions.includes(verString);
   })();
 
   await test(