git-market/curl-curl
1
0
Fork 0
mirror of https://github.com/curl/curl.git synced 2026-04-11 12:01:42 +08:00
Code Issues Actions 14 Packages Projects Releases Wiki Activity

test1560: skip some URLs if UTF-8 is not supported

Browse Source 
Debian CI found that `lib1560` implements tests that will fail when
UTF-8 isn't supported.  We can detect that with `nl_langinfo` and skip
the specific URLs that fail (i.e., those whose `getflags` are either
`CURLU_PUNYCODE` or `CURLU_PUNY2IDN`).

Co-authored-by: Viktor Szakats
Closes #17933
This commit is contained in:
Sergio Durigan Junior 2025-07-17 05:38:10 -04:00 committed by Viktor Szakats
parent 1fcf22585f
commit 7d1ca2e7e1
No known key found for this signature in database
GPG Key ID: B5ABD165E2AEF201
2 changed files with 23 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
tests/libtest/lib1560.c
View File

@ -1428,7 +1428,7 @@ static int set_url(void)
2. Set one or more parts
3. Extract and compare all parts - not the URL
*/
static int setget_parts(void)
static int setget_parts(int has_utf8)
{
int i;
int error = 0;
@ -1440,6 +1440,10 @@ static int setget_parts(void)
error++;
break;
}
if((setget_parts_list[i].getflags == CURLU_PUNYCODE ||
setget_parts_list[i].getflags == CURLU_PUNY2IDN) && !has_utf8) {
continue;
}
if(setget_parts_list[i].in)
rc = curl_url_set(urlp, CURLUPART_URL, setget_parts_list[i].in,
setget_parts_list[i].urlflags);
@ -1525,7 +1529,7 @@ static int set_parts(void)
return error;
}
static int get_url(void)
static int get_url(int has_utf8)
{
int i;
int error = 0;
@ -1536,6 +1540,10 @@ static int get_url(void)
error++;
break;
}
if((get_url_list[i].getflags == CURLU_PUNYCODE ||
get_url_list[i].getflags == CURLU_PUNY2IDN) && !has_utf8) {
continue;
}
rc = curl_url_set(urlp, CURLUPART_URL, get_url_list[i].in,
get_url_list[i].urlflags);
if(!rc) {
@ -1565,7 +1573,7 @@ static int get_url(void)
return error;
}
static int get_parts(void)
static int get_parts(int has_utf8)
{
int i;
int error = 0;
@ -1576,6 +1584,10 @@ static int get_parts(void)
error++;
break;
}
if((get_parts_list[i].getflags == CURLU_PUNYCODE ||
get_parts_list[i].getflags == CURLU_PUNY2IDN) && !has_utf8) {
continue;
}
rc = curl_url_set(urlp, CURLUPART_URL,
get_parts_list[i].in,
get_parts_list[i].urlflags);
@ -2018,15 +2030,17 @@ err:
static CURLcode test_lib1560(char *URL)
{
int has_utf8 = !!getenv("CURL_TEST_HAVE_CODESET_UTF8");
(void)URL; /* not used */
if(urldup())
return (CURLcode)11;
if(setget_parts())
if(setget_parts(has_utf8))
return (CURLcode)10;
if(get_url())
if(get_url(has_utf8))
return (CURLcode)3;
if(huge())
@ -2047,7 +2061,7 @@ static CURLcode test_lib1560(char *URL)
if(set_parts())
return (CURLcode)2;
if(get_parts())
if(get_parts(has_utf8))
return (CURLcode)4;
if(clear_url())

3
tests/runtests.pl
View File

@ -809,6 +809,9 @@ sub checksystemfeatures {
$feature{"crypto"} = $feature{"NTLM"} || $feature{"Kerberos"} || $feature{"SPNEGO"};
$feature{"local-http"} = servers::localhttp();
$feature{"codeset-utf8"} = lc(langinfo(CODESET())) eq "utf-8";
if($feature{"codeset-utf8"}) {
$ENV{'CURL_TEST_HAVE_CODESET_UTF8'} = 1;
}
# make each protocol an enabled "feature"
for my $p (@protocols) {