X-Git-Url: https://git.openstreetmap.org./nominatim.git/blobdiff_plain/3ef4c4fbe7da4b8feb9566d3455badffec55e151..f45b3fa3f243de8894c03d8c9377157d01092c8c:/test/php/Nominatim/ParameterParserTest.php?ds=sidebyside diff --git a/test/php/Nominatim/ParameterParserTest.php b/test/php/Nominatim/ParameterParserTest.php index 105f9d34..9f51a629 100644 --- a/test/php/Nominatim/ParameterParserTest.php +++ b/test/php/Nominatim/ParameterParserTest.php @@ -2,29 +2,27 @@ namespace Nominatim; -use Exception; - -require_once('../../lib/ParameterParser.php'); +require_once(CONST_BasePath.'/lib/ParameterParser.php'); 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() { - $oParams = new ParameterParser([ + $oParams = new ParameterParser(array( 'bool1' => '1', 'bool2' => '0', 'bool3' => 'true', 'bool4' => 'false', 'bool5' => '' - ]); + )); $this->assertSame(false, $oParams->getBool('non-exists')); $this->assertSame(true, $oParams->getBool('non-exists', true)); @@ -38,11 +36,11 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase public function testGetInt() { - $oParams = new ParameterParser([ + $oParams = new ParameterParser(array( 'int1' => '5', 'int2' => '-1', 'int3' => 0 - ]); + )); $this->assertSame(false, $oParams->getInt('non-exists')); $this->assertSame(999, $oParams->getInt('non-exists', 999)); @@ -55,26 +53,30 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase 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() { - $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() { - $oParams = new ParameterParser([ + $oParams = new ParameterParser(array( 'float1' => '1.0', 'float2' => '-5', 'float3' => 0 - ]); + )); $this->assertSame(false, $oParams->getFloat('non-exists')); $this->assertSame(999, $oParams->getFloat('non-exists', 999)); @@ -85,31 +87,37 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase public function testGetFloatWithEmptyString() { - $this->setExpectedException(Exception::class, "Floating-point number expected for parameter 'float4'"); - (new ParameterParser(['float4' => '']))->getFloat('float4'); + $this->expectException(\Exception::class); + $this->expectExceptionMessage("Floating-point number expected for parameter 'float4'"); + + (new ParameterParser(array('float4' => '')))->getFloat('float4'); } 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() { - $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() { - $oParams = new ParameterParser([ + $oParams = new ParameterParser(array( 'str1' => 'abc', 'str2' => '', 'str3' => '0' - ]); + )); $this->assertSame(false, $oParams->getString('non-exists')); $this->assertSame('default', $oParams->getString('non-exists', 'default')); @@ -121,42 +129,43 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase public function testGetSet() { - $oParams = new ParameterParser([ + $oParams = new ParameterParser(array( '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'))); + $this->assertSame(0, $oParams->getSet('val3', array('foo', 'bar'))); } 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() { - $oParams = new ParameterParser([ + $oParams = new ParameterParser(array( 'list1' => ',a,b,c,,c,d', 'list2' => 'a', 'list3' => '', 'list4' => '0' - ]); + )); $this->assertSame(false, $oParams->getStringList('non-exists')); - $this->assertSame(['a', 'b'], $oParams->getStringList('non-exists', ['a', 'b'])); - $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')); } @@ -164,8 +173,8 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase public function testGetPreferredLanguages() { - $oParams = new ParameterParser(['accept-language' => '']); - $this->assertSame([ + $oParams = new ParameterParser(array('accept-language' => '')); + $this->assertSame(array( 'short_name:default' => 'short_name:default', 'name:default' => 'name:default', 'short_name' => 'short_name', @@ -175,10 +184,10 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase 'official_name' => 'official_name', 'ref' => 'ref', 'type' => 'type' - ], $oParams->getPreferredLanguages('default')); + ), $oParams->getPreferredLanguages('default')); - $oParams = new ParameterParser(['accept-language' => 'de,en']); - $this->assertSame([ + $oParams = new ParameterParser(array('accept-language' => 'de,en')); + $this->assertSame(array( 'short_name:de' => 'short_name:de', 'name:de' => 'name:de', 'short_name:en' => 'short_name:en', @@ -191,10 +200,10 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase 'official_name' => 'official_name', 'ref' => 'ref', 'type' => 'type' - ], $oParams->getPreferredLanguages('default')); + ), $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([ + $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', 'short_name:fr' => 'short_name:fr', @@ -213,6 +222,6 @@ class ParameterParserTest extends \PHPUnit_Framework_TestCase 'official_name' => 'official_name', 'ref' => 'ref', 'type' => 'type', - ], $oParams->getPreferredLanguages('default')); + ), $oParams->getPreferredLanguages('default')); } }