summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2018-11-04 23:30:48 +0100
committerDaniel Stenberg <daniel@haxx.se>2018-11-06 19:11:58 +0100
commit9df8dc101ba03807a3257ba0922fe4dd03c81ed3 (patch)
treed989d54168bbcd479768725e0555a1de79cf5a2d /tests
parent74f4782319d478953a29ed7de77d50901760ac3c (diff)
downloadcurl-9df8dc101ba03807a3257ba0922fe4dd03c81ed3.tar.gz
url: a short host name + port is not a scheme
The function identifying a leading "scheme" part of the URL considered a few letters ending with a colon to be a scheme, making something like "short:80" to become an unknown scheme instead of a short host name and a port number. Extended test 1560 to verify. Also fixed test203 to use file_pwd to make it get the correct path on windows. Removed test 2070 since it was a duplicate of 203. Assisted-by: Marcel Raad Reported-by: Hagai Auro Fixes #3220 Fixes #3233 Closes #3223 Closes #3235
Diffstat (limited to 'tests')
-rw-r--r--tests/FILEFORMAT2
-rw-r--r--tests/data/Makefile.inc2
-rw-r--r--tests/data/test2036
-rw-r--r--tests/data/test207041
-rw-r--r--tests/libtest/lib1560.c14
-rwxr-xr-xtests/runtests.pl7
6 files changed, 29 insertions, 43 deletions
diff --git a/tests/FILEFORMAT b/tests/FILEFORMAT
index d2bcc4e00..505c573cb 100644
--- a/tests/FILEFORMAT
+++ b/tests/FILEFORMAT
@@ -365,6 +365,8 @@ Available substitute variables include:
%POP3PORT - Port number of the POP3 server
%PROXYPORT - Port number of the HTTP proxy
%PWD - Current directory
+%POSIX_PWD - Current directory somewhat mingw friendly
+%FILE_PWD - Current directory, on windows prefixed with a slash
%RTSP6PORT - IPv6 port number of the RTSP server
%RTSPPORT - Port number of the RTSP server
%SMTP6PORT - IPv6 port number of the SMTP server
diff --git a/tests/data/Makefile.inc b/tests/data/Makefile.inc
index 4611d3cb8..a4fe1deff 100644
--- a/tests/data/Makefile.inc
+++ b/tests/data/Makefile.inc
@@ -201,7 +201,7 @@ test2040 test2041 test2042 test2043 test2044 test2045 test2046 test2047 \
test2048 test2049 test2050 test2051 test2052 test2053 test2054 test2055 \
test2056 test2057 test2058 test2059 test2060 test2061 test2062 test2063 \
test2064 test2065 test2066 test2067 test2068 test2069 \
-test2070 test2071 test2072 test2073 test2074 test2075 \
+ test2071 test2072 test2073 test2074 test2075 \
test2080 \
test2100 \
\
diff --git a/tests/data/test203 b/tests/data/test203
index 393842656..ee850cb1c 100644
--- a/tests/data/test203
+++ b/tests/data/test203
@@ -24,8 +24,12 @@ file
<name>
file:/path URL with a single slash
</name>
+<setenv>
+# Needed for MSYS2 to not convert
+MSYS2_ARG_CONV_EXCL=file:
+</setenv>
<command option="no-include">
-file:%PWD/log/test203.txt
+file:%FILE_PWD/log/test203.txt
</command>
<file name="log/test203.txt">
foo
diff --git a/tests/data/test2070 b/tests/data/test2070
deleted file mode 100644
index 655cd8a39..000000000
--- a/tests/data/test2070
+++ /dev/null
@@ -1,41 +0,0 @@
-<testcase>
-<info>
-<keywords>
-FILE
-</keywords>
-</info>
-
-<reply>
-<data>
-foo
- bar
-bar
- foo
-moo
-</data>
-</reply>
-
-# Client-side
-<client>
-<server>
-file
-</server>
- <name>
-basic file:// file with no authority
- </name>
-<command option="no-include">
-file:%PWD/log/test2070.txt
-</command>
-<file name="log/test2070.txt">
-foo
- bar
-bar
- foo
-moo
-</file>
-</client>
-
-# Verify data after the test has been "shot"
-<verify>
-</verify>
-</testcase>
diff --git a/tests/libtest/lib1560.c b/tests/libtest/lib1560.c
index 57469a906..5aa6f4bbc 100644
--- a/tests/libtest/lib1560.c
+++ b/tests/libtest/lib1560.c
@@ -128,6 +128,20 @@ struct querycase {
};
static struct testcase get_parts_list[] ={
+#ifdef WIN32
+ {"file:/C:\\programs\\foo",
+ "file | [11] | [12] | [13] | [14] | [15] | C:\\programs\\foo | [16] | [17]",
+ CURLU_DEFAULT_SCHEME, 0, CURLUE_OK},
+ {"file://C:\\programs\\foo",
+ "file | [11] | [12] | [13] | [14] | [15] | C:\\programs\\foo | [16] | [17]",
+ CURLU_DEFAULT_SCHEME, 0, CURLUE_OK},
+ {"file:///C:\\programs\\foo",
+ "file | [11] | [12] | [13] | [14] | [15] | C:\\programs\\foo | [16] | [17]",
+ CURLU_DEFAULT_SCHEME, 0, CURLUE_OK},
+#endif
+ {"boing:80",
+ "https | [11] | [12] | [13] | boing | 80 | / | [16] | [17]",
+ CURLU_DEFAULT_SCHEME, 0, CURLUE_OK},
{"http://[fd00:a41::50]:8080",
"http | [11] | [12] | [13] | [fd00:a41::50] | 8080 | / | [16] | [17]",
CURLU_DEFAULT_SCHEME, 0, CURLUE_OK},
diff --git a/tests/runtests.pl b/tests/runtests.pl
index 421cf2a6b..5b51d3e4f 100755
--- a/tests/runtests.pl
+++ b/tests/runtests.pl
@@ -3113,6 +3113,13 @@ sub subVariables {
$$thing =~ s/%CURL/$CURL/g;
$$thing =~ s/%PWD/$pwd/g;
$$thing =~ s/%POSIX_PWD/$posix_pwd/g;
+
+ my $file_pwd = $pwd;
+ if($file_pwd !~ /^\//) {
+ $file_pwd = "/$file_pwd";
+ }
+
+ $$thing =~ s/%FILE_PWD/$file_pwd/g;
$$thing =~ s/%SRCDIR/$srcdir/g;
$$thing =~ s/%USER/$USER/g;