Skip to content

Commit 8b6b39f

Browse files
sgolemonTyrael
authored andcommitted
Add a couple more test cases to parse_url() tests
http://::# (valid but silly) http://# (invalid)
1 parent 290becb commit 8b6b39f

10 files changed

+37
-10
lines changed

ext/standard/tests/url/parse_url_basic_001.phpt

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -743,6 +743,13 @@ echo "Done";
743743
string(1) ":"
744744
}
745745

746+
--> http://::#: array(2) {
747+
["scheme"]=>
748+
string(4) "http"
749+
["host"]=>
750+
string(1) ":"
751+
}
752+
746753
--> x://::6.5: array(3) {
747754
["scheme"]=>
748755
string(1) "x"
@@ -856,11 +863,13 @@ echo "Done";
856863

857864
--> http://?: bool(false)
858865

866+
--> http://#: bool(false)
867+
859868
--> http://?:: bool(false)
860869

861870
--> http://:?: bool(false)
862871

863872
--> http://blah.com:123456: bool(false)
864873

865874
--> http://blah.com:abcdef: bool(false)
866-
Done
875+
Done

ext/standard/tests/url/parse_url_basic_002.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ echo "Done";
9696
--> x:/blah.com : string(1) "x"
9797
--> x://::abc/? : bool(false)
9898
--> http://::? : string(4) "http"
99+
--> http://::# : string(4) "http"
99100
--> x://::6.5 : string(1) "x"
100101
--> http://?:/ : string(4) "http"
101102
--> http://@?:/ : string(4) "http"
@@ -118,8 +119,9 @@ echo "Done";
118119
--> http://@:/ : bool(false)
119120
--> http://:/ : bool(false)
120121
--> http://? : bool(false)
122+
--> http://# : bool(false)
121123
--> http://?: : bool(false)
122124
--> http://:? : bool(false)
123125
--> http://blah.com:123456 : bool(false)
124126
--> http://blah.com:abcdef : bool(false)
125-
Done
127+
Done

ext/standard/tests/url/parse_url_basic_003.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ echo "Done";
9595
--> x:/blah.com : NULL
9696
--> x://::abc/? : bool(false)
9797
--> http://::? : string(1) ":"
98+
--> http://::# : string(1) ":"
9899
--> x://::6.5 : string(1) ":"
99100
--> http://?:/ : string(1) "?"
100101
--> http://@?:/ : string(1) "?"
@@ -117,8 +118,9 @@ echo "Done";
117118
--> http://@:/ : bool(false)
118119
--> http://:/ : bool(false)
119120
--> http://? : bool(false)
121+
--> http://# : bool(false)
120122
--> http://?: : bool(false)
121123
--> http://:? : bool(false)
122124
--> http://blah.com:123456 : bool(false)
123125
--> http://blah.com:abcdef : bool(false)
124-
Done
126+
Done

ext/standard/tests/url/parse_url_basic_004.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ echo "Done";
9595
--> x:/blah.com : NULL
9696
--> x://::abc/? : bool(false)
9797
--> http://::? : NULL
98+
--> http://::# : NULL
9899
--> x://::6.5 : int(6)
99100
--> http://?:/ : NULL
100101
--> http://@?:/ : NULL
@@ -117,8 +118,9 @@ echo "Done";
117118
--> http://@:/ : bool(false)
118119
--> http://:/ : bool(false)
119120
--> http://? : bool(false)
121+
--> http://# : bool(false)
120122
--> http://?: : bool(false)
121123
--> http://:? : bool(false)
122124
--> http://blah.com:123456 : bool(false)
123125
--> http://blah.com:abcdef : bool(false)
124-
Done
126+
Done

ext/standard/tests/url/parse_url_basic_005.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ echo "Done";
9595
--> x:/blah.com : NULL
9696
--> x://::abc/? : bool(false)
9797
--> http://::? : NULL
98+
--> http://::# : NULL
9899
--> x://::6.5 : NULL
99100
--> http://?:/ : NULL
100101
--> http://@?:/ : string(0) ""
@@ -117,8 +118,9 @@ echo "Done";
117118
--> http://@:/ : bool(false)
118119
--> http://:/ : bool(false)
119120
--> http://? : bool(false)
121+
--> http://# : bool(false)
120122
--> http://?: : bool(false)
121123
--> http://:? : bool(false)
122124
--> http://blah.com:123456 : bool(false)
123125
--> http://blah.com:abcdef : bool(false)
124-
Done
126+
Done

ext/standard/tests/url/parse_url_basic_006.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ echo "Done";
9595
--> x:/blah.com : NULL
9696
--> x://::abc/? : bool(false)
9797
--> http://::? : NULL
98+
--> http://::# : NULL
9899
--> x://::6.5 : NULL
99100
--> http://?:/ : NULL
100101
--> http://@?:/ : NULL
@@ -117,8 +118,9 @@ echo "Done";
117118
--> http://@:/ : bool(false)
118119
--> http://:/ : bool(false)
119120
--> http://? : bool(false)
121+
--> http://# : bool(false)
120122
--> http://?: : bool(false)
121123
--> http://:? : bool(false)
122124
--> http://blah.com:123456 : bool(false)
123125
--> http://blah.com:abcdef : bool(false)
124-
Done
126+
Done

ext/standard/tests/url/parse_url_basic_007.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ echo "Done";
9595
--> x:/blah.com : string(9) "/blah.com"
9696
--> x://::abc/? : bool(false)
9797
--> http://::? : NULL
98+
--> http://::# : NULL
9899
--> x://::6.5 : NULL
99100
--> http://?:/ : string(1) "/"
100101
--> http://@?:/ : string(1) "/"
@@ -117,8 +118,9 @@ echo "Done";
117118
--> http://@:/ : bool(false)
118119
--> http://:/ : bool(false)
119120
--> http://? : bool(false)
121+
--> http://# : bool(false)
120122
--> http://?: : bool(false)
121123
--> http://:? : bool(false)
122124
--> http://blah.com:123456 : bool(false)
123125
--> http://blah.com:abcdef : bool(false)
124-
Done
126+
Done

ext/standard/tests/url/parse_url_basic_008.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ echo "Done";
9595
--> x:/blah.com : NULL
9696
--> x://::abc/? : bool(false)
9797
--> http://::? : NULL
98+
--> http://::# : NULL
9899
--> x://::6.5 : NULL
99100
--> http://?:/ : NULL
100101
--> http://@?:/ : NULL
@@ -117,8 +118,9 @@ echo "Done";
117118
--> http://@:/ : bool(false)
118119
--> http://:/ : bool(false)
119120
--> http://? : bool(false)
121+
--> http://# : bool(false)
120122
--> http://?: : bool(false)
121123
--> http://:? : bool(false)
122124
--> http://blah.com:123456 : bool(false)
123125
--> http://blah.com:abcdef : bool(false)
124-
Done
126+
Done

ext/standard/tests/url/parse_url_basic_009.phpt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@ echo "Done";
9595
--> x:/blah.com : NULL
9696
--> x://::abc/? : bool(false)
9797
--> http://::? : NULL
98+
--> http://::# : NULL
9899
--> x://::6.5 : NULL
99100
--> http://?:/ : NULL
100101
--> http://@?:/ : NULL
@@ -117,8 +118,9 @@ echo "Done";
117118
--> http://@:/ : bool(false)
118119
--> http://:/ : bool(false)
119120
--> http://? : bool(false)
121+
--> http://# : bool(false)
120122
--> http://?: : bool(false)
121123
--> http://:? : bool(false)
122124
--> http://blah.com:123456 : bool(false)
123125
--> http://blah.com:abcdef : bool(false)
124-
Done
126+
Done

ext/standard/tests/url/urls.inc

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ $urls = array(
7575
'x:/blah.com',
7676
'x://::abc/?',
7777
'http://::?',
78+
'http://::#',
7879
'x://::6.5',
7980
'http://?:/',
8081
'http://@?:/',
@@ -99,11 +100,12 @@ $urls = array(
99100
'http://@:/',
100101
'http://:/',
101102
'http://?',
103+
'http://#',
102104
'http://?:',
103105
'http://:?',
104106
'http://blah.com:123456',
105107
'http://blah.com:abcdef',
106108
);
107109

108110

109-
?>
111+
?>

0 commit comments

Comments
 (0)