#371: disable direct unpack tests in Travis CI

because it misses unrar 5 required for the tests
nzbget · May 5, 2017 · 39089b6 · 39089b6
1 parent 61af2b3
commit 39089b6
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -33,4 +33,4 @@ matrix:
 script:
   - $COMPILER --version
   - CXX=$COMPILER ./configure --enable-tests && make
-  - ./nzbget --tests exclude:[DupeMatcher]
+  - ./nzbget --tests exclude:[Unrar]
diff --git a/tests/postprocess/DirectUnpackTest.cpp b/tests/postprocess/DirectUnpackTest.cpp
@@ -38,7 +38,7 @@ class DirectUnpackDownloadQueueMock : public DownloadQueue
 	virtual void Save() {};
 };
 
-TEST_CASE("Direct-unpack simple", "[Rar][DirectUnpack][Slow][TestData]")
+TEST_CASE("Direct-unpack simple", "[Rar][DirectUnpack][Unrar][Slow][TestData]")
 {
 	Options::CmdOptList cmdOpts;
 	cmdOpts.push_back("WriteLog=none");
@@ -49,6 +49,8 @@ TEST_CASE("Direct-unpack simple", "[Rar][DirectUnpack][Slow][TestData]")
 
 	TestUtil::PrepareWorkingDir("empty");
 
+	INFO("This test requires working unrar 5 in search path");
+
 	REQUIRE(FileSystem::CopyFile((TestUtil::TestDataDir() + "/rarrenamer/testfile3.part01.rar").c_str(),
 		(TestUtil::WorkingDir() + "/testfile3.part01.rar").c_str()));
 	REQUIRE(FileSystem::CopyFile((TestUtil::TestDataDir() + "/rarrenamer/testfile3.part02.rar").c_str(),
@@ -84,7 +86,7 @@ TEST_CASE("Direct-unpack simple", "[Rar][DirectUnpack][Slow][TestData]")
 	REQUIRE(FileSystem::FileExists((TestUtil::WorkingDir() + "/_unpack/testfile3.dat").c_str()));
 }
 
-TEST_CASE("Direct-unpack two archives", "[Rar][DirectUnpack][Slow][TestData]")
+TEST_CASE("Direct-unpack two archives", "[Rar][DirectUnpack][Unrar][Slow][TestData]")
 {
 	Options::CmdOptList cmdOpts;
 	cmdOpts.push_back("WriteLog=none");
@@ -95,6 +97,8 @@ TEST_CASE("Direct-unpack two archives", "[Rar][DirectUnpack][Slow][TestData]")
 
 	TestUtil::PrepareWorkingDir("empty");
 
+	INFO("This test requires working unrar 5 in search path");
+
 	REQUIRE(FileSystem::CopyFile((TestUtil::TestDataDir() + "/rarrenamer/testfile3.part01.rar").c_str(),
 		(TestUtil::WorkingDir() + "/testfile3.part01.rar").c_str()));
 	REQUIRE(FileSystem::CopyFile((TestUtil::TestDataDir() + "/rarrenamer/testfile3.part02.rar").c_str(),

diff --git a/tests/postprocess/DupeMatcherTest.cpp b/tests/postprocess/DupeMatcherTest.cpp
@@ -27,7 +27,7 @@
 #include "FileSystem.h"
 #include "TestUtil.h"
 
-TEST_CASE("Dupe matcher", "[Par][DupeMatcher][Slow][TestData]")
+TEST_CASE("Dupe matcher", "[Par][DupeMatcher][Unrar][Slow][TestData]")
 {
 	Options options(nullptr, nullptr);