git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

Add more test coverage for jsonpath "$.*" with arrays

author Michael Paquier <michael@paquier.xyz>

2024年7月19日 05:17:56 +0000 (14:17 +0900)

committer Michael Paquier <michael@paquier.xyz>

2024年7月19日 05:17:56 +0000 (14:17 +0900)

There was no coverage for the code path to unwrap an array before
applying ".*" to it, so add tests to provide more coverage for both
objects and arrays.

This shows, for example, that no results are returned for an array of
scalars, and what results are returned when the array contains an
object. A few more scenarios are covered with the strict/lax modes and
the operator "@?".

Author: David Wheeler
Reported-by: David G. Johnston, Stepan Neretin
Discussion: https://postgr.es/m/A95346F9-6147-46E0-809E-532A485D71D6@justatheory.com

src/test/regress/expected/jsonb_jsonpath.out patch | blob | blame | history

src/test/regress/sql/jsonb_jsonpath.sql patch | blob | blame | history

diff --git a/src/test/regress/expected/jsonb_jsonpath.out b/src/test/regress/expected/jsonb_jsonpath.out

index a6112e86fa7ae17f725516f2e89f5ec24b8ec2e2..7bb4eb1bc276ef0bff04836e115ba359bc5dbd60 100644 (file)

--- a/src/test/regress/expected/jsonb_jsonpath.out

+++ b/src/test/regress/expected/jsonb_jsonpath.out

@@ -1135,6 +1135,56 @@ select jsonb_path_query('{"a": [1, 2]}', 'lax $.a * 3', silent => true);

------------------

(0 rows)

+-- any key on arrays with and without unwrapping.

+select jsonb_path_query('{"a": [1,2,3], "b": [3,4,5]}', '$.*');

+ jsonb_path_query

+------------------

+ [1, 2, 3]

+ [3, 4, 5]

+(2 rows)

+select jsonb_path_query('[1,2,3]', '$.*');

+ jsonb_path_query

+------------------

+(0 rows)

+select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'lax $.*');

+ jsonb_path_query

+------------------

+ [3, 4, 5]

+(1 row)

+select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'strict $.*');

+ERROR: jsonpath wildcard member accessor can only be applied to an object

+select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'strict $.*', NULL, true);

+ jsonb_path_query

+------------------

+(0 rows)

+select jsonb '{"a": [1,2,3], "b": [3,4,5]}' @? '$.*';

+ ?column?

+----------

+ t

+(1 row)

+select jsonb '[1,2,3]' @? '$.*';

+ ?column?

+----------

+ f

+(1 row)

+select jsonb '[1,2,3,{"b": [3,4,5]}]' @? 'lax $.*';

+ ?column?

+----------

+ t

+(1 row)

+select jsonb '[1,2,3,{"b": [3,4,5]}]' @? 'strict $.*';

+ ?column?

+----------

+(1 row)

-- extension: boolean expressions

select jsonb_path_query('2', '$ > 1');

jsonb_path_query

diff --git a/src/test/regress/sql/jsonb_jsonpath.sql b/src/test/regress/sql/jsonb_jsonpath.sql

index 5e14f7759bb64cc51d3acbf7798ed3be968f60d5..17f9d038c0aa91986880682205a981f9405e7b10 100644 (file)

--- a/src/test/regress/sql/jsonb_jsonpath.sql

+++ b/src/test/regress/sql/jsonb_jsonpath.sql

@@ -241,6 +241,17 @@ select jsonb_path_query('{"a": [2, 3, 4]}', 'lax -$.a');

select jsonb_path_query('{"a": [1, 2]}', 'lax $.a * 3');

select jsonb_path_query('{"a": [1, 2]}', 'lax $.a * 3', silent => true);

+-- any key on arrays with and without unwrapping.

+select jsonb_path_query('{"a": [1,2,3], "b": [3,4,5]}', '$.*');

+select jsonb_path_query('[1,2,3]', '$.*');

+select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'lax $.*');

+select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'strict $.*');

+select jsonb_path_query('[1,2,3,{"b": [3,4,5]}]', 'strict $.*', NULL, true);

+select jsonb '{"a": [1,2,3], "b": [3,4,5]}' @? '$.*';

+select jsonb '[1,2,3]' @? '$.*';

+select jsonb '[1,2,3,{"b": [3,4,5]}]' @? 'lax $.*';

+select jsonb '[1,2,3,{"b": [3,4,5]}]' @? 'strict $.*';

-- extension: boolean expressions

select jsonb_path_query('2', '$ > 1');

select jsonb_path_query('2', '$ <= 1');

This is the main PostgreSQL git repository.

RSS Atom