Better message about optional test, warning suppression

Henry Fredrick Schreiner · Henry Schreiner
1 parent 48cb5008
Showing 5 changed files with 11 additions and 4 deletions
.appveyor.yml
.ci/make_and_test.sh
CMakeLists.txt
include/CLI/Option.hpp
tests/OptionalTest.cpp
@@ -18,7 +18,9 @@ build_script:
   - conan create . CLIUtils/CLI11
  
 test_script:
+  - cd build
   - ctest --output-on-failure -C Debug
+  - tests\OptionalTest
  
 notifications:
   - provider: Webhook
@@ -15,6 +15,7 @@ echo &quot;Testing...&quot;
 set -evx
  
 ctest --output-on-failure
+./tests/OptionalTest
  
 set +evx
 echo -en "travis_fold:end:script.test\\r"
@@ -27,7 +27,7 @@ if(CMAKE_PROJECT_NAME STREQUAL PROJECT_NAME)
     if(MSVC)
         add_definitions("/W4")
     else()
-        add_definitions(-Wall -Wextra -pedantic)
+        add_definitions(-Wall -Wextra -pedantic -Wno-deprecated-declarations)
     endif()
  
     if(CMAKE_VERSION VERSION_GREATER 3.6)
@@ -309,7 +309,7 @@ class Option : public OptionBase&lt;Option&gt; {
     template <typename T = App> Option *requires(std::string opt_name) {
         for(const Option_p &opt : dynamic_cast<T *>(parent_)->options_)
             if(opt.get() != this && opt->check_name(opt_name))
-                return requires(opt.get());
+                return needs(opt.get());
         throw IncorrectConstruction::MissingOption(opt_name);
     }
  
 #include <cstdlib>
 #include <iostream>
  
-#if CLI11_OPTIONAL
-
 #include "app_helper.hpp"
  
+#if CLI11_OPTIONAL
+
 TEST_F(TApp, OptionalTest) {
     optional<int> opt;
     app.add_option("-c,--count", opt);
@@ -24,4 +24,8 @@ TEST_F(TApp, OptionalTest) {
     EXPECT_EQ(*opt, 3);
 }
  
+#else
+
+TEST_F(TApp, DISABLED_OptionalTest) {}
+
 #endif