From d1687c6926819f023c78b353458950a303796aba Mon Sep 17 00:00:00 2001
From: Andrew Dunstan <andrew@dunslane.net>
Date: Tue, 19 Sep 2017 15:31:37 -0400
Subject: [PATCH] Disable multi-byte citext tests

This reverts commit 890faaf1 which attempted unsuccessfully to deal with
the problem, and instead just comments out these tests like other similar
tests elsewhere in the script.
---
 contrib/citext/expected/citext.out   | 32 ++++++----------------------
 contrib/citext/expected/citext_1.out | 32 ++++++----------------------
 contrib/citext/sql/citext.sql        | 14 ++++++------
 3 files changed, 19 insertions(+), 59 deletions(-)

diff --git a/contrib/citext/expected/citext.out b/contrib/citext/expected/citext.out
index ff0a6ed588..95373182af 100644
--- a/contrib/citext/expected/citext.out
+++ b/contrib/citext/expected/citext.out
@@ -1,8 +1,6 @@
 --
 --  Test citext datatype
 --
---- script setup
-set client_encoding = 'utf8';
 CREATE EXTENSION citext;
 -- Check whether any of our opclasses fail amvalidate
 SELECT amname, opcname
@@ -2599,6 +2597,8 @@ SELECT citext_pattern_ge('b'::citext, 'A'::citext) AS true;
  t
 (1 row)
 
+-- Multi-byte tests below are diabled like the sanity tests above.
+-- Uncomment to run them.
 -- Test ~<~ and ~<=~
 SELECT 'a'::citext ~<~  'B'::citext AS t;
  t 
@@ -2612,12 +2612,7 @@ SELECT 'b'::citext ~<~  'A'::citext AS f;
  f
 (1 row)
 
-SELECT 'à'::citext ~<~  'À'::citext AS f;
- f 
----
- f
-(1 row)
-
+-- SELECT 'à'::citext ~<~  'À'::citext AS f;
 SELECT 'a'::citext ~<=~ 'B'::citext AS t;
  t 
 ---
@@ -2630,12 +2625,7 @@ SELECT 'a'::citext ~<=~ 'A'::citext AS t;
  t
 (1 row)
 
-SELECT 'à'::citext ~<=~ 'À'::citext AS t;
- t 
----
- f
-(1 row)
-
+-- SELECT 'à'::citext ~<=~ 'À'::citext AS t;
 -- Test ~>~ and ~>=~
 SELECT 'B'::citext ~>~  'a'::citext AS t;
  t 
@@ -2649,12 +2639,7 @@ SELECT 'b'::citext ~>~  'A'::citext AS t;
  t
 (1 row)
 
-SELECT 'à'::citext ~>~  'À'::citext AS f;
- f 
----
- t
-(1 row)
-
+-- SELECT 'à'::citext ~>~  'À'::citext AS f;
 SELECT 'B'::citext ~>~  'b'::citext AS f;
  f 
 ---
@@ -2667,12 +2652,7 @@ SELECT 'B'::citext ~>=~ 'b'::citext AS t;
  t
 (1 row)
 
-SELECT 'à'::citext ~>=~ 'À'::citext AS t;
- t 
----
- t
-(1 row)
-
+-- SELECT 'à'::citext ~>=~ 'À'::citext AS t;
 -- Test implicit casting. citext casts to text, but not vice-versa.
 SELECT 'B'::citext ~<~  'a'::text AS t;  -- text wins.
  t 
diff --git a/contrib/citext/expected/citext_1.out b/contrib/citext/expected/citext_1.out
index 43a609b066..855ec3f10b 100644
--- a/contrib/citext/expected/citext_1.out
+++ b/contrib/citext/expected/citext_1.out
@@ -1,8 +1,6 @@
 --
 --  Test citext datatype
 --
---- script setup
-set client_encoding = 'utf8';
 CREATE EXTENSION citext;
 -- Check whether any of our opclasses fail amvalidate
 SELECT amname, opcname
@@ -2599,6 +2597,8 @@ SELECT citext_pattern_ge('b'::citext, 'A'::citext) AS true;
  t
 (1 row)
 
+-- Multi-byte tests below are diabled like the sanity tests above.
+-- Uncomment to run them.
 -- Test ~<~ and ~<=~
 SELECT 'a'::citext ~<~  'B'::citext AS t;
  t 
@@ -2612,12 +2612,7 @@ SELECT 'b'::citext ~<~  'A'::citext AS f;
  f
 (1 row)
 
-SELECT 'à'::citext ~<~  'À'::citext AS f;
- f 
----
- f
-(1 row)
-
+-- SELECT 'à'::citext ~<~  'À'::citext AS f;
 SELECT 'a'::citext ~<=~ 'B'::citext AS t;
  t 
 ---
@@ -2630,12 +2625,7 @@ SELECT 'a'::citext ~<=~ 'A'::citext AS t;
  t
 (1 row)
 
-SELECT 'à'::citext ~<=~ 'À'::citext AS t;
- t 
----
- t
-(1 row)
-
+-- SELECT 'à'::citext ~<=~ 'À'::citext AS t;
 -- Test ~>~ and ~>=~
 SELECT 'B'::citext ~>~  'a'::citext AS t;
  t 
@@ -2649,12 +2639,7 @@ SELECT 'b'::citext ~>~  'A'::citext AS t;
  t
 (1 row)
 
-SELECT 'à'::citext ~>~  'À'::citext AS f;
- f 
----
- f
-(1 row)
-
+-- SELECT 'à'::citext ~>~  'À'::citext AS f;
 SELECT 'B'::citext ~>~  'b'::citext AS f;
  f 
 ---
@@ -2667,12 +2652,7 @@ SELECT 'B'::citext ~>=~ 'b'::citext AS t;
  t
 (1 row)
 
-SELECT 'à'::citext ~>=~ 'À'::citext AS t;
- t 
----
- t
-(1 row)
-
+-- SELECT 'à'::citext ~>=~ 'À'::citext AS t;
 -- Test implicit casting. citext casts to text, but not vice-versa.
 SELECT 'B'::citext ~<~  'a'::text AS t;  -- text wins.
  t 
diff --git a/contrib/citext/sql/citext.sql b/contrib/citext/sql/citext.sql
index 91dd7d03d0..2732be436d 100644
--- a/contrib/citext/sql/citext.sql
+++ b/contrib/citext/sql/citext.sql
@@ -2,9 +2,6 @@
 --  Test citext datatype
 --
 
---- script setup
-set client_encoding = 'utf8';
-
 CREATE EXTENSION citext;
 
 -- Check whether any of our opclasses fail amvalidate
@@ -804,21 +801,24 @@ SELECT citext_pattern_ge('b'::citext, 'a'::citext) AS true;
 SELECT citext_pattern_ge('B'::citext, 'a'::citext) AS true;
 SELECT citext_pattern_ge('b'::citext, 'A'::citext) AS true;
 
+-- Multi-byte tests below are diabled like the sanity tests above.
+-- Uncomment to run them.
+
 -- Test ~<~ and ~<=~
 SELECT 'a'::citext ~<~  'B'::citext AS t;
 SELECT 'b'::citext ~<~  'A'::citext AS f;
-SELECT 'à'::citext ~<~  'À'::citext AS f;
+-- SELECT 'à'::citext ~<~  'À'::citext AS f;
 SELECT 'a'::citext ~<=~ 'B'::citext AS t;
 SELECT 'a'::citext ~<=~ 'A'::citext AS t;
-SELECT 'à'::citext ~<=~ 'À'::citext AS t;
+-- SELECT 'à'::citext ~<=~ 'À'::citext AS t;
 
 -- Test ~>~ and ~>=~
 SELECT 'B'::citext ~>~  'a'::citext AS t;
 SELECT 'b'::citext ~>~  'A'::citext AS t;
-SELECT 'à'::citext ~>~  'À'::citext AS f;
+-- SELECT 'à'::citext ~>~  'À'::citext AS f;
 SELECT 'B'::citext ~>~  'b'::citext AS f;
 SELECT 'B'::citext ~>=~ 'b'::citext AS t;
-SELECT 'à'::citext ~>=~ 'À'::citext AS t;
+-- SELECT 'à'::citext ~>=~ 'À'::citext AS t;
 
 -- Test implicit casting. citext casts to text, but not vice-versa.
 SELECT 'B'::citext ~<~  'a'::text AS t;  -- text wins.