PHP 8 behaves slightly different with in_array and usort

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

diff --git a/test/php/Nominatim/ParameterParserTest.php b/test/php/Nominatim/ParameterParserTest.php
index ddc0685134dd2a1364559db90e20508cfbd40f1e..7381bdf84a9cca05f13a0aff3ce6f0804076d62b 100644 (file)
--- a/test/php/Nominatim/ParameterParserTest.php
+++ b/test/php/Nominatim/ParameterParserTest.php
@@ -1,30 +1,36 @@
 <?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

 {
 
 
     public function testGetBool()
     {
 {
 
 
     public function testGetBool()
     {

-        $oParams = new ParameterParser([
+        $oParams = new ParameterParser(array(

                                         'bool1' => '1',
                                         'bool2' => '0',
                                         'bool3' => 'true',
                                         'bool4' => 'false',
                                         'bool5' => ''
                                         'bool1' => '1',
                                         'bool2' => '0',
                                         'bool3' => 'true',
                                         'bool4' => 'false',
                                         'bool5' => ''

-                                       ]);
+                                       ));

 
         $this->assertSame(false, $oParams->getBool('non-exists'));
         $this->assertSame(true, $oParams->getBool('non-exists', true));
 
         $this->assertSame(false, $oParams->getBool('non-exists'));
         $this->assertSame(true, $oParams->getBool('non-exists', true));


@@ -38,11 +44,11 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase
 
     public function testGetInt()
     {
 
     public function testGetInt()
     {

-        $oParams = new ParameterParser([
+        $oParams = new ParameterParser(array(

                                         'int1' => '5',
                                         'int2' => '-1',
                                         'int3' => 0
                                         'int1' => '5',
                                         'int2' => '-1',
                                         'int3' => 0

-                                       ]);
+                                       ));

 
         $this->assertSame(false, $oParams->getInt('non-exists'));
         $this->assertSame(999, $oParams->getInt('non-exists', 999));
 
         $this->assertSame(false, $oParams->getInt('non-exists'));
         $this->assertSame(999, $oParams->getInt('non-exists', 999));


@@ -55,106 +61,122 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase
 
     public function testGetIntWithNonNumber()
     {
 
     public function testGetIntWithNonNumber()
     {

-        $this->setExpectedException(Exception::class, "Integer number expected for parameter 'int4'");
-        (new ParameterParser(['int4' => 'a']))->getInt('int4');
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Integer number expected for parameter 'int4'");
+
+        (new ParameterParser(array('int4' => 'a')))->getInt('int4');

     }
 
 
     public function testGetIntWithEmpytString()
     {
     }
 
 
     public function testGetIntWithEmpytString()
     {

-        $this->setExpectedException(Exception::class, "Integer number expected for parameter 'int5'");
-        (new ParameterParser(['int5' => '']))->getInt('int5');
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Integer number expected for parameter 'int5'");
+
+        (new ParameterParser(array('int5' => '')))->getInt('int5');

     }
 
 
     public function testGetFloat()
     {
 
     }
 
 
     public function testGetFloat()
     {
 

-        $oParams = new ParameterParser([
+        $oParams = new ParameterParser(array(

                                         'float1' => '1.0',
                                         'float2' => '-5',
                                         'float1' => '1.0',
                                         'float2' => '-5',

-                                        'float3' => '',
-                                        'float4' => 0
-                                       ]);
+                                        'float3' => 0
+                                       ));

 
         $this->assertSame(false, $oParams->getFloat('non-exists'));
         $this->assertSame(999, $oParams->getFloat('non-exists', 999));
         $this->assertSame(1.0, $oParams->getFloat('float1'));
         $this->assertSame(-5.0, $oParams->getFloat('float2'));
 
         $this->assertSame(false, $oParams->getFloat('non-exists'));
         $this->assertSame(999, $oParams->getFloat('non-exists', 999));
         $this->assertSame(1.0, $oParams->getFloat('float1'));
         $this->assertSame(-5.0, $oParams->getFloat('float2'));

-        $this->assertSame(false, $oParams->getFloat('float3')); // FIXME: should be 0 instead?
-        $this->assertSame(0.0, $oParams->getFloat('float4'));
+        $this->assertSame(0.0, $oParams->getFloat('float3'));

     }
 
     }
 

+    public function testGetFloatWithEmptyString()
+    {
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Floating-point number expected for parameter 'float4'");
+
+        (new ParameterParser(array('float4' => '')))->getFloat('float4');
+    }

 
 

-    public function testGetFloatWithString()
+    public function testGetFloatWithTextString()

     {
     {

-        $this->setExpectedException(Exception::class, "Floating-point number expected for parameter 'float5'");
-        (new ParameterParser(['float5' => 'a']))->getFloat('float5');
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Floating-point number expected for parameter 'float5'");
+
+        (new ParameterParser(array('float5' => 'a')))->getFloat('float5');

     }
 
 
     public function testGetFloatWithInvalidNumber()
     {
     }
 
 
     public function testGetFloatWithInvalidNumber()
     {

-        $this->setExpectedException(Exception::class, "Floating-point number expected for parameter 'float6'");
-        (new ParameterParser(['float6' => '-55.']))->getFloat('float6');
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Floating-point number expected for parameter 'float6'");
+
+        (new ParameterParser(array('float6' => '-55.')))->getFloat('float6');

     }
 
 
     public function testGetString()
     {
     }
 
 
     public function testGetString()
     {

-        $oParams = new ParameterParser([
+        $oParams = new ParameterParser(array(

                                         'str1' => 'abc',
                                         'str2' => '',
                                         'str3' => '0'
                                         'str1' => 'abc',
                                         'str2' => '',
                                         'str3' => '0'

-                                       ]);
+                                       ));

 
         $this->assertSame(false, $oParams->getString('non-exists'));
         $this->assertSame('default', $oParams->getString('non-exists', 'default'));
         $this->assertSame('abc', $oParams->getString('str1'));
         $this->assertSame(false, $oParams->getStringList('str2'));
 
         $this->assertSame(false, $oParams->getString('non-exists'));
         $this->assertSame('default', $oParams->getString('non-exists', 'default'));
         $this->assertSame('abc', $oParams->getString('str1'));
         $this->assertSame(false, $oParams->getStringList('str2'));

-        $this->assertSame(false, $oParams->getStringList('str3')); // FIXME: should be 0 instead?
+        $this->assertSame(false, $oParams->getStringList('str3')); // sadly PHP magic treats 0 as false when returned

     }
 
 
     public function testGetSet()
     {
     }
 
 
     public function testGetSet()
     {

-        $oParams = new ParameterParser([
+        $this->expectException(\Exception::class);
+        $this->expectExceptionMessage("Parameter 'val3' must be one of: foo, bar");
+
+        $oParams = new ParameterParser(array(

                                         'val1' => 'foo',
                                         'val2' => '',
                                         'val3' => 0
                                         'val1' => 'foo',
                                         'val2' => '',
                                         'val3' => 0

-                                       ]);
+                                       ));

 
 

-        $this->assertSame(false, $oParams->getSet('non-exists', ['foo', 'bar']));
-        // FIXME: unclear if the default value has to be part of the set
-        $this->assertSame('default', $oParams->getSet('non-exists', ['foo', 'bar'], 'default'));
-        $this->assertSame('foo', $oParams->getSet('val1', ['foo', 'bar']));
+        $this->assertSame(false, $oParams->getSet('non-exists', array('foo', 'bar')));
+        $this->assertSame('default', $oParams->getSet('non-exists', array('foo', 'bar'), 'default'));
+        $this->assertSame('foo', $oParams->getSet('val1', array('foo', 'bar')));

 
 

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

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

-        $this->setExpectedException(Exception::class, "Parameter 'val4' must be one of: foo, bar");
-        (new ParameterParser(['val4' => 'faz']))->getSet('val4', ['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'));

     }
 
 
     public function testGetStringList()
     {
     }
 
 
     public function testGetStringList()
     {

-        $oParams = new ParameterParser([
+        $oParams = new ParameterParser(array(

                                         'list1' => ',a,b,c,,c,d',
                                         'list2' => 'a',
                                         'list3' => '',
                                         'list4' => '0'
                                         'list1' => ',a,b,c,,c,d',
                                         'list2' => 'a',
                                         'list3' => '',
                                         'list4' => '0'

-                                       ]);
+                                       ));

 
         $this->assertSame(false, $oParams->getStringList('non-exists'));
 
         $this->assertSame(false, $oParams->getStringList('non-exists'));

-        $this->assertSame(['a', 'b'], $oParams->getStringList('non-exists', ['a', 'b']));
-        // FIXME: unclear if empty string items should be removed
-        $this->assertSame(['', 'a', 'b', 'c', '', 'c', 'd'], $oParams->getStringList('list1'));
-        $this->assertSame(['a'], $oParams->getStringList('list2'));
+        $this->assertSame(array('a', 'b'), $oParams->getStringList('non-exists', array('a', 'b')));
+        $this->assertSame(array('a', 'b', 'c', 'c', 'd'), $oParams->getStringList('list1'));
+        $this->assertSame(array('a'), $oParams->getStringList('list2'));

         $this->assertSame(false, $oParams->getStringList('list3'));
         $this->assertSame(false, $oParams->getStringList('list4'));
     }
         $this->assertSame(false, $oParams->getStringList('list3'));
         $this->assertSame(false, $oParams->getStringList('list4'));
     }


@@ -162,55 +184,68 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase
 
     public function testGetPreferredLanguages()
     {
 
     public function testGetPreferredLanguages()
     {

-        $oParams = new ParameterParser(['accept-language' => '']);
-        $this->assertSame([
-                           'short_name:default' => 'short_name:default',
+        $oParams = new ParameterParser(array('accept-language' => ''));
+        $this->assertSame(array(

                            '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'));
-
-        $oParams = new ParameterParser(['accept-language' => 'de,en']);
-        $this->assertSame([
-                           'short_name:de' => 'short_name:de',
+                           '_place_name' => '_place_name'
+                          ), array_slice($oParams->getPreferredLanguages('default'), 0, 4));
+
+        $oParams = new ParameterParser(array('accept-language' => 'de,en'));
+        $this->assertSame(array(

                            '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'));
-
-        $oParams = new ParameterParser(['accept-language' => 'fr-ca,fr;q=0.8,en-ca;q=0.5,en;q=0.3']);
-        $this->assertSame([
-                           'short_name:fr-ca' => 'short_name:fr-ca',
+                           '_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(

                            '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')));

     }
 }
     }
 }