]> granicus.if.org Git - php/commitdiff
Remove bogus skipifs in get_browser tests
authorNikita Popov <nikita.ppv@gmail.com>
Wed, 6 Nov 2019 16:48:16 +0000 (17:48 +0100)
committerNikita Popov <nikita.ppv@gmail.com>
Wed, 6 Nov 2019 16:48:52 +0000 (17:48 +0100)
We provide this file, it should be readable.

These checks no longer work, since --INI-- values are no longer
passed to --SKIPIF--.

ext/standard/tests/misc/bug65550.phpt
ext/standard/tests/misc/get_browser_basic.phpt
ext/standard/tests/misc/get_browser_error.phpt
ext/standard/tests/misc/get_browser_variation1.phpt
ext/standard/tests/misc/get_browser_variation2.phpt

index 41967426fa3037301b15626133bc40ad766497be..5ac0389d333c3aff8110734f248aad00aa000d97 100644 (file)
@@ -2,10 +2,6 @@
 Bug #65550 (get_browser() incorrectly parses entries with "+" sign)
 --INI--
 browscap={PWD}/browscap.ini
---SKIPIF--
-<?php
-if (!is_readable(ini_get('browscap'))) die('skip browscap.ini file ' . ini_get('browscap') . ' not readable');
-?>
 --FILE--
 <?php
 $user_agent = 'Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US) AppleWebKit/522+ (KHTML, like Gecko, Safari/522) OmniWeb/v613';
index fdc2a7868c11dfe64033d805d4a143b9b8654a6c..7daf668c50bbb7f282f5b71a0a0284332d3725d1 100644 (file)
@@ -2,16 +2,6 @@
 Test get_browser() function : basic functionality
 --INI--
 browscap={PWD}/browscap.ini
---SKIPIF--
-<?php
-       /**
-        * Basic test, it would be pretty much coincidence if there's
-        * a browscap.ini on another place that isn't valid.
-        */
-       if(! is_readable( ini_get( 'browscap' ) ) ) {
-               die( 'skip: browscap.ini file ' . ini_get('browscap') . " not readable" );
-       }
-?>
 --FILE--
 <?php
 /* Prototype  : mixed get_browser([string browser_name [, bool return_array]])
index c8689d8370483dec35c99480a452512b2a9b1a4b..16e07f9f7defa9ce7d4ff097f07dccaf6218f619 100644 (file)
@@ -2,16 +2,6 @@
 Test get_browser() function : error functionality
 --INI--
 browscap={PWD}/browscap.ini
---SKIPIF--
-<?php
-       /**
-        * Basic test, it would be pretty much coincidence if there's
-        * a browscap.ini on another place that isn't valid.
-        */
-       if(! is_readable( ini_get( 'browscap' ) ) ) {
-               die( 'skip: browscap.ini file ' . ini_get('browscap') . " not readable" );
-       }
-?>
 --FILE--
 <?php
 /* Prototype  : mixed get_browser([string browser_name [, bool return_array]])
index 2cd2c4e5a0155d3765b098e415232aa0056bc526..dacfdfc8d9fdf66f8763d98c8a6ca7884d6f5fac 100644 (file)
@@ -2,16 +2,6 @@
 Test get_browser() function : variation functionality: extra browser names
 --INI--
 browscap={PWD}/browscap.ini
---SKIPIF--
-<?php
-       /**
-        * Basic test, it would be pretty much coincidence if there's
-        * a browscap.ini on another place that isn't valid.
-        */
-       if(! is_readable( ini_get( 'browscap' ) ) ) {
-               die( 'skip: browscap.ini file ' . ini_get('browscap') . " not readable" );
-       }
-?>
 --FILE--
 <?php
 /* Prototype  : mixed get_browser([string browser_name [, bool return_array]])
index 5c4d2ecd6e803b3a0b1e60ac8d7ba152072e055b..be487d8701dcf7fb2774a3665189bf11e9ab5277 100644 (file)
@@ -2,16 +2,6 @@
 Test get_browser() function variation : Return data as object
 --INI--
 browscap={PWD}/browscap.ini
---SKIPIF--
-<?php
-       /**
-        * Basic test, it would be pretty much coincidence if there's
-        * a browscap.ini on another place that isn't valid.
-        */
-       if(! is_readable( ini_get( 'browscap' ) ) ) {
-               die( 'skip: browscap.ini file ' . ini_get('browscap') . " not readable" );
-       }
-?>
 --FILE--
 <?php