Merge bitcoin/bitcoin#29967: test: Add two more urlDecode tests

fanquake · fanquake · commit 7973a6709156 · 2024-04-26T16:40:04.000+08:00
fa55972 test: Add two more urlDecode tests (MarcoFalke) Pull request description: Trivial follow-up after bitcoin/bitcoin#29904 (comment) ACKs for top commit: laanwj: Code review ACK bitcoin/bitcoin@fa55972 fjahr: ACK fa55972 stickies-v: ACK fa55972 Sjors: utACK fa55972 Tree-SHA512: 99916feebb35b5670a365120f962fd6c28cb124635c99ac3ee3520dfc130bd1672f43b06b05b7b0b9e563d223bd009f8d6622817a2d2b4ee24596af40e2cdfaf
diff --git a/src/test/common_url_tests.cpp b/src/test/common_url_tests.cpp
@@ -54,6 +54,9 @@ BOOST_AUTO_TEST_CASE(decode_malformed_test) {
     BOOST_CHECK_EQUAL(UrlDecode(" %Z "), " %Z ");
     BOOST_CHECK_EQUAL(UrlDecode(" % X"), " % X");
 
+    BOOST_CHECK_EQUAL(UrlDecode("%%ffg"), "%\xffg");
+    BOOST_CHECK_EQUAL(UrlDecode("%fg"), "%fg");
+
     BOOST_CHECK_EQUAL(UrlDecode("%-1"), "%-1");
     BOOST_CHECK_EQUAL(UrlDecode("%1-"), "%1-");
 }

Original file line number	Diff line number	Diff line change
`@@ -54,6 +54,9 @@ BOOST_AUTO_TEST_CASE(decode_malformed_test) {`
`54`	`54`	`BOOST_CHECK_EQUAL(UrlDecode(" %Z "), " %Z ");`
`55`	`55`	`BOOST_CHECK_EQUAL(UrlDecode(" % X"), " % X");`
`56`	`56`
	`57`	`+ BOOST_CHECK_EQUAL(UrlDecode("%%ffg"), "%\xffg");`
	`58`	`+ BOOST_CHECK_EQUAL(UrlDecode("%fg"), "%fg");`
	`59`	`+`
`57`	`60`	`BOOST_CHECK_EQUAL(UrlDecode("%-1"), "%-1");`
`58`	`61`	`BOOST_CHECK_EQUAL(UrlDecode("%1-"), "%1-");`
`59`	`62`	`}`