summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Stenberg <daniel@haxx.se>2020-09-05 20:30:26 +0200
committerDaniel Stenberg <daniel@haxx.se>2020-09-05 20:32:11 +0200
commit842f03df56d99b2438c77e69a0c463703d4727ab (patch)
tree7a08b51b6224044ede61a05c096f4fa58c8e0013
parent6ecb63e493fd64855961758ca1b448d03f59a16b (diff)
downloadcurl-bagder/rm-test1541.tar.gz
test1541: remove since a known bugbagder/rm-test1541
A shared connection cache is not thread-safe is a known issue. Stop testing that until we believe this is addressed, to reduce occasional test failures we don't care about. The test code in lib1541.c is left in git to allow us to restore this when we get to fix this.
-rw-r--r--docs/KNOWN_BUGS2
-rw-r--r--tests/data/Makefile.inc2
-rw-r--r--tests/data/test154134
-rw-r--r--tests/libtest/Makefile.inc2
-rw-r--r--tests/libtest/lib1541.c9
5 files changed, 11 insertions, 38 deletions
diff --git a/docs/KNOWN_BUGS b/docs/KNOWN_BUGS
index bc77913b6..8e129c34b 100644
--- a/docs/KNOWN_BUGS
+++ b/docs/KNOWN_BUGS
@@ -761,7 +761,7 @@ problems may have been fixed or changed somewhat since this was written!
handle share a connection cache, but due to how connections are used they are
still not thread-safe when used shared.
- See https://github.com/curl/curl/issues/4915
+ See https://github.com/curl/curl/issues/4915 and lib1541.c
11.12 'no_proxy' string-matches IPv6 numerical addreses
diff --git a/tests/data/Makefile.inc b/tests/data/Makefile.inc
index bc6dea9f8..c6f36bc1e 100644
--- a/tests/data/Makefile.inc
+++ b/tests/data/Makefile.inc
@@ -185,7 +185,7 @@ test1508 test1509 test1510 test1511 test1512 test1513 test1514 test1515 \
test1516 test1517 test1518 test1519 test1520 test1521 test1522 test1523 \
test1524 test1525 test1526 test1527 test1528 test1529 test1530 test1531 \
test1532 test1533 test1534 test1535 test1536 test1537 test1538 test1539 \
-test1540 test1541 \
+test1540 \
\
test1550 test1551 test1552 test1553 test1554 test1555 test1556 test1557 \
test1558 test1559 test1560 test1561 test1562 test1563 test1564 test1565 \
diff --git a/tests/data/test1541 b/tests/data/test1541
deleted file mode 100644
index e18bb47df..000000000
--- a/tests/data/test1541
+++ /dev/null
@@ -1,34 +0,0 @@
-<testcase>
-<info>
-<keywords>
-HTTP
-HTTP GET
-multi-threaded
-connection-sharing
-</keywords>
-</info>
-
-# Server-side
-<reply>
-</reply>
-
-# Client-side
-<client>
-<server>
-http
-</server>
-<tool>
-lib1541
-</tool>
- <name>
-connection sharing using 67 parallel threads for 7 seconds
- </name>
- <command>
-http://%HOSTIP:%HTTPPORT/1
-</command>
-</client>
-
-# Verify data after the test has been "shot"
-<verify>
-</verify>
-</testcase>
diff --git a/tests/libtest/Makefile.inc b/tests/libtest/Makefile.inc
index c568e1354..51b1ffaf7 100644
--- a/tests/libtest/Makefile.inc
+++ b/tests/libtest/Makefile.inc
@@ -54,7 +54,7 @@ noinst_PROGRAMS = chkhostname libauthretry libntlmconnect \
lib1518 lib1520 lib1521 lib1522 lib1523 \
lib1525 lib1526 lib1527 lib1528 lib1529 lib1530 lib1531 lib1532 lib1533 \
lib1534 lib1535 lib1536 lib1537 lib1538 lib1539 \
- lib1540 lib1541 \
+ lib1540 \
lib1550 lib1551 lib1552 lib1553 lib1554 lib1555 lib1556 lib1557 \
lib1558 lib1559 lib1560 lib1564 lib1565 lib1567 \
lib1591 lib1592 lib1593 lib1594 lib1596 \
diff --git a/tests/libtest/lib1541.c b/tests/libtest/lib1541.c
index f3b41f5be..c47c54e25 100644
--- a/tests/libtest/lib1541.c
+++ b/tests/libtest/lib1541.c
@@ -5,7 +5,7 @@
* | (__| |_| | _ <| |___
* \___|\___/|_| \_\_____|
*
- * Copyright (C) 2019, Daniel Stenberg, <daniel@haxx.se>, et al.
+ * Copyright (C) 2019 - 2020, Daniel Stenberg, <daniel@haxx.se>, et al.
*
* This software is licensed as described in the file COPYING, which
* you should have received as part of this distribution. The terms
@@ -19,6 +19,13 @@
* KIND, either express or implied.
*
***************************************************************************/
+/*
+ * KNOW_BUGS "A shared connection cache is not thread-safe"
+ *
+ * This source code was used to verify shared connection cache but since this
+ * is a known issue the test is no longer built or run. This code is here to
+ * allow for testing once someone gets to work on fixing this.
+ */
#include "test.h"
#include "testutil.h"