lift restrictions on search with frequent terms slightly

[nominatim.git] / test / php / Nominatim / ParameterParserTest.php

diff --git a/test/php/Nominatim/ParameterParserTest.php b/test/php/Nominatim/ParameterParserTest.php
index 0e88d318bb0fcc0858dfeba85bd2597f86226cc1..82716d4de98d68ed29826621759c9d379d1701c4 100644 (file)
--- a/test/php/Nominatim/ParameterParserTest.php
+++ b/test/php/Nominatim/ParameterParserTest.php
@@ -1,15 +1,21 @@
 <?php
 <?php

+/**
+ * SPDX-License-Identifier: GPL-2.0-only
+ *
+ * This file is part of Nominatim. (https://nominatim.org)
+ *
+ * Copyright (C) 2022 by the Nominatim developer community.
+ * For a full list of authors see the git log.
+ */

 
 namespace Nominatim;
 
 
 namespace Nominatim;
 

-use Exception;
-
-require_once('../../lib/ParameterParser.php');
+require_once(CONST_LibDir.'/ParameterParser.php');

 
 
 function userError($sError)
 {
 
 
 function userError($sError)
 {

-    throw new Exception($sError);
+    throw new \Exception($sError);

 }
 
 class ParameterParserTest extends \PHPUnit\Framework\TestCase
 }
 
 class ParameterParserTest extends \PHPUnit\Framework\TestCase


@@ -55,14 +61,18 @@ class ParameterParserTest extends \PHPUnit\Framework\TestCase
 
     public function testGetIntWithNonNumber()
     {
 
     public function testGetIntWithNonNumber()
     {

-        $this->setExpectedException(Exception::class, "Integer number expected for parameter 'int4'");
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Integer number expected for parameter 'int4'");
+

         (new ParameterParser(array('int4' => 'a')))->getInt('int4');
     }
 
 
     public function testGetIntWithEmpytString()
     {
         (new ParameterParser(array('int4' => 'a')))->getInt('int4');
     }
 
 
     public function testGetIntWithEmpytString()
     {

-        $this->setExpectedException(Exception::class, "Integer number expected for parameter 'int5'");
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Integer number expected for parameter 'int5'");
+

         (new ParameterParser(array('int5' => '')))->getInt('int5');
     }
 
         (new ParameterParser(array('int5' => '')))->getInt('int5');
     }
 


@@ -85,20 +95,26 @@ class ParameterParserTest extends \PHPUnit\Framework\TestCase
 
     public function testGetFloatWithEmptyString()
     {
 
     public function testGetFloatWithEmptyString()
     {

-        $this->setExpectedException(Exception::class, "Floating-point number expected for parameter 'float4'");
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Floating-point number expected for parameter 'float4'");
+

         (new ParameterParser(array('float4' => '')))->getFloat('float4');
     }
 
     public function testGetFloatWithTextString()
     {
         (new ParameterParser(array('float4' => '')))->getFloat('float4');
     }
 
     public function testGetFloatWithTextString()
     {

-        $this->setExpectedException(Exception::class, "Floating-point number expected for parameter 'float5'");
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Floating-point number expected for parameter 'float5'");
+

         (new ParameterParser(array('float5' => 'a')))->getFloat('float5');
     }
 
 
     public function testGetFloatWithInvalidNumber()
     {
         (new ParameterParser(array('float5' => 'a')))->getFloat('float5');
     }
 
 
     public function testGetFloatWithInvalidNumber()
     {

-        $this->setExpectedException(Exception::class, "Floating-point number expected for parameter 'float6'");
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Floating-point number expected for parameter 'float6'");
+

         (new ParameterParser(array('float6' => '-55.')))->getFloat('float6');
     }
 
         (new ParameterParser(array('float6' => '-55.')))->getFloat('float6');
     }
 


@@ -132,13 +148,15 @@ class ParameterParserTest extends \PHPUnit\Framework\TestCase
         $this->assertSame('foo', $oParams->getSet('val1', array('foo', 'bar')));
 
         $this->assertSame(false, $oParams->getSet('val2', array('foo', 'bar')));
         $this->assertSame('foo', $oParams->getSet('val1', array('foo', 'bar')));
 
         $this->assertSame(false, $oParams->getSet('val2', array('foo', 'bar')));

-        $this->assertSame(0, $oParams->getSet('val3', array('foo', 'bar')));
+        $this->assertSame(false, $oParams->getSet('val3', array('foo', 'bar')));

     }
 
 
     public function testGetSetWithValueNotInSet()
     {
     }
 
 
     public function testGetSetWithValueNotInSet()
     {

-        $this->setExpectedException(Exception::class, "Parameter 'val4' must be one of: foo, bar");
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Parameter 'val4' must be one of: foo, bar");
+

         (new ParameterParser(array('val4' => 'faz')))->getSet('val4', array('foo', 'bar'));
     }
 
         (new ParameterParser(array('val4' => 'faz')))->getSet('val4', array('foo', 'bar'));
     }
 


@@ -165,53 +183,66 @@ class ParameterParserTest extends \PHPUnit\Framework\TestCase
     {
         $oParams = new ParameterParser(array('accept-language' => ''));
         $this->assertSame(array(
     {
         $oParams = new ParameterParser(array('accept-language' => ''));
         $this->assertSame(array(

-                           'short_name:default' => 'short_name:default',

                            'name:default' => 'name:default',
                            'name:default' => 'name:default',

-                           'short_name' => 'short_name',
+                           '_place_name:default' => '_place_name:default',

                            'name' => 'name',
                            'name' => 'name',

-                           'brand' => 'brand',
-                           'official_name:default' => 'official_name:default',
-                           'official_name' => 'official_name',
-                           'ref' => 'ref',
-                           'type' => 'type'
-                          ), $oParams->getPreferredLanguages('default'));
+                           '_place_name' => '_place_name'
+                          ), array_slice($oParams->getPreferredLanguages('default'), 0, 4));

 
         $oParams = new ParameterParser(array('accept-language' => 'de,en'));
         $this->assertSame(array(
 
         $oParams = new ParameterParser(array('accept-language' => 'de,en'));
         $this->assertSame(array(

-                           'short_name:de' => 'short_name:de',

                            'name:de' => 'name:de',
                            'name:de' => 'name:de',

-                           'short_name:en' => 'short_name:en',
+                           '_place_name:de' => '_place_name:de',

                            'name:en' => 'name:en',
                            'name:en' => 'name:en',

-                           'short_name' => 'short_name',
+                           '_place_name:en' => '_place_name:en',

                            'name' => 'name',
                            'name' => 'name',

-                           'brand' => 'brand',
-                           'official_name:de' => 'official_name:de',
-                           'official_name:en' => 'official_name:en',
-                           'official_name' => 'official_name',
-                           'ref' => 'ref',
-                           'type' => 'type'
-                          ), $oParams->getPreferredLanguages('default'));
+                           '_place_name' => '_place_name'
+                          ), array_slice($oParams->getPreferredLanguages('default'), 0, 6));

 
         $oParams = new ParameterParser(array('accept-language' => 'fr-ca,fr;q=0.8,en-ca;q=0.5,en;q=0.3'));
         $this->assertSame(array(
 
         $oParams = new ParameterParser(array('accept-language' => 'fr-ca,fr;q=0.8,en-ca;q=0.5,en;q=0.3'));
         $this->assertSame(array(

-                           'short_name:fr-ca' => 'short_name:fr-ca',

                            'name:fr-ca' => 'name:fr-ca',
                            'name:fr-ca' => 'name:fr-ca',

-                           'short_name:fr' => 'short_name:fr',
+                           '_place_name:fr-ca' => '_place_name:fr-ca',

                            'name:fr' => 'name:fr',
                            'name:fr' => 'name:fr',

-                           'short_name:en-ca' => 'short_name:en-ca',
+                           '_place_name:fr' => '_place_name:fr',

                            'name:en-ca' => 'name:en-ca',
                            'name:en-ca' => 'name:en-ca',

-                           'short_name:en' => 'short_name:en',
+                           '_place_name:en-ca' => '_place_name:en-ca',

                            'name:en' => 'name:en',
                            'name:en' => 'name:en',

-                           'short_name' => 'short_name',
+                           '_place_name:en' => '_place_name:en',

                            'name' => 'name',
                            'name' => 'name',

-                           'brand' => 'brand',
-                           'official_name:fr-ca' => 'official_name:fr-ca',
-                           'official_name:fr' => 'official_name:fr',
-                           'official_name:en-ca' => 'official_name:en-ca',
-                           'official_name:en' => 'official_name:en',
-                           'official_name' => 'official_name',
-                           'ref' => 'ref',
-                           'type' => 'type',
-                          ), $oParams->getPreferredLanguages('default'));
+                           '_place_name' => '_place_name'
+                          ), array_slice($oParams->getPreferredLanguages('default'), 0, 10));
+
+        $oParams = new ParameterParser(array('accept-language' => 'ja_rm,zh_pinyin'));
+        $this->assertSame(array(
+                           'name:ja_rm' => 'name:ja_rm',
+                           '_place_name:ja_rm' => '_place_name:ja_rm',
+                           'name:zh_pinyin' => 'name:zh_pinyin',
+                           '_place_name:zh_pinyin' => '_place_name:zh_pinyin',
+                           'name:ja' => 'name:ja',
+                           '_place_name:ja' => '_place_name:ja',
+                           'name:zh' => 'name:zh',
+                           '_place_name:zh' => '_place_name:zh',
+                           'name' => 'name',
+                           '_place_name' => '_place_name'
+                          ), array_slice($oParams->getPreferredLanguages('default'), 0, 10));
+    }
+
+    public function testHasSetAny()
+    {
+        $oParams = new ParameterParser(array(
+                                        'one' => '',
+                                        'two' => 0,
+                                        'three' => '0',
+                                        'four' => '1',
+                                        'five' => 'anystring'
+        ));
+        $this->assertFalse($oParams->hasSetAny(array()));
+        $this->assertFalse($oParams->hasSetAny(array('')));
+        $this->assertFalse($oParams->hasSetAny(array('unknown')));
+        $this->assertFalse($oParams->hasSetAny(array('one', 'two', 'three')));
+        $this->assertTrue($oParams->hasSetAny(array('one', 'four')));
+        $this->assertTrue($oParams->hasSetAny(array('four')));
+        $this->assertTrue($oParams->hasSetAny(array('five')));

     }
 }
     }
 }