From 8e9fab60cae7991052b4e41f1ef2029d8beba0e7 Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Tue, 29 Mar 2022 17:12:39 +0200
Subject: [PATCH] Bug 30194: (follow-up) Fix xt/api.t

The query object can have an arbitrary structure, and thus it doesn't
make sense to check for the additionalProperties presence. also, it is
not allowed by OpenAPIv2 :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
---
 xt/api.t | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/xt/api.t b/xt/api.t
index 3d33a669bf..1dd206f498 100755
--- a/xt/api.t
+++ b/xt/api.t
@@ -39,7 +39,8 @@ foreach my $route ( keys %{$paths} ) {
                 && $parameter->{schema}->{type} eq 'object' ) {
 
                 # it is an object type definition
-                if ( not exists $parameter->{schema}->{additionalProperties} ) {
+                if ( $parameter->{name} ne 'query' # our query parameter is under-specified
+                    and not exists $parameter->{schema}->{additionalProperties} ) {
                     push @missing_additionalProperties,
                       { type  => 'parameter',
                         route => $route,
-- 
2.39.5