diff options
author | jade <lix@jade.fyi> | 2024-05-30 14:58:23 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@lix-systems> | 2024-05-30 14:58:23 +0000 |
commit | c161687b5fa6e7604e99ee5df2e73388952baafb (patch) | |
tree | 701aef58694ba5a05874e338c314f258ca04b5d8 | |
parent | 18aa3e1d570b4ecbb9962376e5fba5757dad8da9 (diff) | |
parent | 285bc67318e2ee4b69b13eb0b8e7b202fc287c51 (diff) |
Merge "tests/filetransfer: reënable on Darwin" into main
-rw-r--r-- | tests/unit/libstore/filetransfer.cc | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/tests/unit/libstore/filetransfer.cc b/tests/unit/libstore/filetransfer.cc index 0e5c0965e..b74d01516 100644 --- a/tests/unit/libstore/filetransfer.cc +++ b/tests/unit/libstore/filetransfer.cc @@ -13,14 +13,6 @@ #include <thread> #include <unistd.h> -// local server tests don't work on darwin without some incantations -// the horrors do not want to look up. contributions welcome though! -#if __APPLE__ -#define NOT_ON_DARWIN(n) DISABLED_##n -#else -#define NOT_ON_DARWIN(n) n -#endif - using namespace std::chrono_literals; namespace { @@ -150,7 +142,7 @@ TEST(FileTransfer, exceptionAbortsDownload) } } -TEST(FileTransfer, NOT_ON_DARWIN(reportsSetupErrors)) +TEST(FileTransfer, reportsSetupErrors) { auto [port, srv] = serveHTTP("404 not found", "", [] { return ""; }); auto ft = makeFileTransfer(); @@ -159,7 +151,7 @@ TEST(FileTransfer, NOT_ON_DARWIN(reportsSetupErrors)) FileTransferError); } -TEST(FileTransfer, NOT_ON_DARWIN(reportsTransferError)) +TEST(FileTransfer, reportsTransferError) { auto [port, srv] = serveHTTP("200 ok", "content-length: 100\r\n", [] { std::this_thread::sleep_for(10ms); @@ -171,7 +163,7 @@ TEST(FileTransfer, NOT_ON_DARWIN(reportsTransferError)) ASSERT_THROW(ft->download(req), FileTransferError); } -TEST(FileTransfer, NOT_ON_DARWIN(handlesContentEncoding)) +TEST(FileTransfer, handlesContentEncoding) { std::string original = "Test data string"; std::string compressed = compress("gzip", original); |