From b1050462020f60888e604c423a081138d7efc2d0 Mon Sep 17 00:00:00 2001
From: gabime <gmelman1@gmail.com>
Date: Sat, 21 Dec 2019 20:15:18 +0200
Subject: [PATCH] Fixed loaders tests

---
 tests/test_loaders.cpp | 29 +++++++++++++++++++++++------
 1 file changed, 23 insertions(+), 6 deletions(-)

diff --git a/tests/test_loaders.cpp b/tests/test_loaders.cpp
index bf8f9416..e3b0203f 100644
--- a/tests/test_loaders.cpp
+++ b/tests/test_loaders.cpp
@@ -9,59 +9,76 @@ using spdlog::loaders::load_env;
 
 TEST_CASE("env", "[loaders]")
 {
+    spdlog::drop("l1");
     auto l1 = spdlog::create<spdlog::sinks::test_sink_st>("l1");
     setenv("SPDLOG_LEVEL", "l1=warn", 1);
     load_env();
     REQUIRE(l1->level() == spdlog::level::warn);
     REQUIRE(spdlog::default_logger()->level() == spdlog::level::info);
-    spdlog::drop(l1->name());
 }
 
 TEST_CASE("argv1", "[loaders]")
 {
+    spdlog::drop("l1");
     const char *argv[] = {"ignore", "SPDLOG_LEVEL=l1=warn"};
     load_argv(2, const_cast<char**>(argv));
     auto l1 = spdlog::create<spdlog::sinks::test_sink_st>("l1");
     REQUIRE(l1->level() == spdlog::level::warn);
     REQUIRE(spdlog::default_logger()->level() == spdlog::level::info);
-    spdlog::drop(l1->name());
 }
 
 TEST_CASE("argv2", "[loaders]")
 {
+    spdlog::drop("l1");
     const char *argv[] = {"ignore", "SPDLOG_LEVEL=l1=warn,trace"};
     load_argv(2, const_cast<char**>(argv));
     auto l1 = spdlog::create<spdlog::sinks::test_sink_st>("l1");
     REQUIRE(l1->level() == spdlog::level::warn);
     REQUIRE(spdlog::default_logger()->level() == spdlog::level::trace);
-    spdlog::drop(l1->name());
 }
 
 TEST_CASE("argv3", "[loaders]")
 {
+    spdlog::drop("l1");
     const char *argv[] = {"ignore", "SPDLOG_LEVEL="};
     load_argv(2, const_cast<char**>(argv));
     auto l1 = spdlog::create<spdlog::sinks::test_sink_st>("l1");
     REQUIRE(l1->level() == spdlog::level::info);
     REQUIRE(spdlog::default_logger()->level() == spdlog::level::info);
-    spdlog::drop(l1->name());
 }
 
 TEST_CASE("argv4", "[loaders]")
 {
+    spdlog::drop("l1");
     const char *argv[] = {"ignore", "SPDLOG_LEVEL=junk"};
     load_argv(2, const_cast<char**>(argv));
     auto l1 = spdlog::create<spdlog::sinks::test_sink_st>("l1");
     REQUIRE(l1->level() == spdlog::level::info);
-    spdlog::drop(l1->name());
 }
 
 TEST_CASE("argv5", "[loaders]")
 {
+    spdlog::drop("l1");
     const char *argv[] = {"ignore", "ignore", "SPDLOG_LEVEL=l1=warn,trace"};
     load_argv(3, const_cast<char**>(argv));
     auto l1 = spdlog::create<spdlog::sinks::test_sink_st>("l1");
     REQUIRE(l1->level() == spdlog::level::warn);
     REQUIRE(spdlog::default_logger()->level() == spdlog::level::trace);
-    spdlog::drop(l1->name());
+}
+
+TEST_CASE("argv6", "[loaders]")
+{
+    spdlog::set_level(spdlog::level::err);
+    const char *argv[] = {""};
+    load_argv(1, const_cast<char**>(argv));
+    REQUIRE(spdlog::default_logger()->level() == spdlog::level::err);
+
+}
+
+TEST_CASE("argv7", "[loaders]")
+{
+    spdlog::set_level(spdlog::level::err);
+    const char *argv[] = {""};
+    load_argv(0, const_cast<char**>(argv));
+    REQUIRE(spdlog::default_logger()->level() == spdlog::level::err);
 }