Merge pull request brianmario#1193 from junaruga/wip/fix-tests-mariadb

Fix test failures on MariaDB

Author: sodabrew

.github/workflows/build.yml

@@ -40,9 +40,8 @@ jobs:
           # db: A DB's brew package name in macOS case.
           #     Set a name "db: 'name@X.Y'" when using an old version.
           # MariaDB lastet version
-          # Allow failure due to the following test failures.
-          # https://github.com/brianmario/mysql2/issues/965
-          # https://github.com/brianmario/mysql2/issues/1152
+          # Allow failure due to the following test failures that rarely happens.
+          # https://github.com/brianmario/mysql2/issues/1194
           - {os: macos-latest, ruby: 2.4, db: mariadb, allow-failure: true}
           # MySQL latest version
           # Allow failure due to the issue #1165.

.github/workflows/container.yml

@@ -14,13 +14,9 @@ jobs:
           # CentOS 7 system Ruby is the fixed version 2.0.0.
           - {distro: centos, image: 'centos:7', name_extra: 'ruby 2.0.0'}
           # Fedora latest stable version
-          # Allow failure due to the following test failures.
-          # https://github.com/brianmario/mysql2/issues/965
-          - {distro: fedora, image: 'fedora:latest', allow-failure: true}
+          - {distro: fedora, image: 'fedora:latest'}
           # Fedora development version
-          # Allow failure due to the following test failures.
-          # https://github.com/brianmario/mysql2/issues/1152
-          - {distro: fedora, image: 'fedora:rawhide', allow-failure: true}
+          - {distro: fedora, image: 'fedora:rawhide'}
       # On the fail-fast: true, it cancels all in-progress jobs
       # if any matrix job fails unlike Travis fast_finish.
       fail-fast: false

.travis.yml

@@ -41,3 +41,6 @@ matrix:
           - mysql2gem.example.com
   fast_finish: true
   allow_failures:
+    # Allow failure due to a package repository not found for now.
+    # https://travis-ci.org/github/brianmario/mysql2/jobs/767276558#L1255
+    - env: DB=mysql55

ci/Dockerfile_fedora

@@ -20,6 +20,7 @@ RUN dnf -yq install \
   redhat-rpm-config \
   ruby-devel \
   rubygem-bigdecimal \
-  rubygem-bundler
+  rubygem-bundler \
+  rubygem-json
 
 CMD bash ci/container.sh

spec/mysql2/client_spec.rb

@@ -1,6 +1,11 @@
 require 'spec_helper'
 
 RSpec.describe Mysql2::Client do # rubocop:disable Metrics/BlockLength
+  let(:performance_schema_enabled) do
+    performance_schema = @client.query "SHOW VARIABLES LIKE 'performance_schema'"
+    performance_schema.any? { |x| x['Value'] == 'ON' }
+  end
+
   context "using defaults file" do
     let(:cnf_file) { File.expand_path('../../my.cnf', __FILE__) }
 
@@ -479,6 +484,7 @@ def run_gc
   end
 
   it "should set default program_name in connect_attrs" do
+    skip("DON'T WORRY, THIS TEST PASSES - but PERFORMANCE SCHEMA is not enabled in your MySQL daemon.") unless performance_schema_enabled
     client = new_client
     if Mysql2::Client::CONNECT_ATTRS.zero? || client.server_info[:version].match(/10.[01].\d+-MariaDB/)
       pending('Both client and server versions must be MySQL 5.6 or MariaDB 10.2 or later.')
@@ -488,6 +494,7 @@ def run_gc
   end
 
   it "should set custom connect_attrs" do
+    skip("DON'T WORRY, THIS TEST PASSES - but PERFORMANCE SCHEMA is not enabled in your MySQL daemon.") unless performance_schema_enabled
     client = new_client(connect_attrs: { program_name: 'my_program_name', foo: 'fooval', bar: 'barval' })
     if Mysql2::Client::CONNECT_ATTRS.zero? || client.server_info[:version].match(/10.[01].\d+-MariaDB/)
       pending('Both client and server versions must be MySQL 5.6 or MariaDB 10.2 or later.')

spec/mysql2/error_spec.rb

@@ -39,6 +39,10 @@
       end
     end
 
+    let(:server_info) do
+      @client.server_info
+    end
+
     before do
       # sanity check
       expect(valid_utf8.encoding).to eql(Encoding::UTF_8)
@@ -56,7 +60,15 @@
         expect(bad_err.message.encoding).to eql(Encoding::UTF_8)
         expect(bad_err.message).to be_valid_encoding
 
-        expect(bad_err.message).to include("??}\u001F")
+        # MariaDB 10.5 returns a little different error message unlike MySQL
+        # and other old MariaDBs.
+        # https://jira.mariadb.org/browse/MDEV-25400
+        err_str = if server_info[:version].match(/MariaDB/) && server_info[:id] >= 100500
+          "??}\\001F"
+        else
+          "??}\u001F"
+        end
+        expect(bad_err.message).to include(err_str)
       end
     end
 

spec/mysql2/statement_spec.rb

@@ -5,12 +5,19 @@
     @client = new_client(encoding: "utf8")
   end
 
+  let(:performance_schema_enabled) do
+    performance_schema = @client.query "SHOW VARIABLES LIKE 'performance_schema'"
+    performance_schema.any? { |x| x['Value'] == 'ON' }
+  end
+
   def stmt_count
     # Use the performance schema in MySQL 5.7 and above
-    @client.query("SELECT COUNT(1) AS count FROM performance_schema.prepared_statements_instances").first['count'].to_i
-  rescue Mysql2::Error
-    # Fall back to the global prepapred statement counter
-    @client.query("SHOW STATUS LIKE 'Prepared_stmt_count'").first['Value'].to_i
+    if performance_schema_enabled
+      @client.query("SELECT COUNT(1) AS count FROM performance_schema.prepared_statements_instances").first['count'].to_i
+    else
+      # Fall back to the global prepapred statement counter
+      @client.query("SHOW STATUS LIKE 'Prepared_stmt_count'").first['Value'].to_i
+    end
   end
 
   it "should create a statement" do