unit tests: don't try to close a NULL outfp_

5 years ago · 23f8a070b3
parent ca2f0471c3
commit 23f8a070b3
6 changed files with 18 additions and 6 deletions
--- a/tests/cell.cpp
+++ b/tests/cell.cpp
@ -23,7 +23,9 @@ class CellTest : public :: testing::Test {
    if(nc_){
      EXPECT_EQ(0, notcurses_stop(nc_));
    }
-    fclose(outfp_);
+    if(outfp_){
+      fclose(outfp_);
+    }
  }

  struct notcurses* nc_{};
--- a/tests/fade.cpp
+++ b/tests/fade.cpp
@ -41,7 +41,9 @@ class FadeTest : public :: testing::Test {
    if(nc_){
      EXPECT_EQ(0, notcurses_stop(nc_));
    }
-    fclose(outfp_);
+    if(outfp_){
+      fclose(outfp_);
+    }
  }

  struct notcurses* nc_{};
--- a/tests/libav.cpp
+++ b/tests/libav.cpp
@ -22,7 +22,9 @@ class LibavTest : public :: testing::Test {
    if(nc_){
      EXPECT_EQ(0, notcurses_stop(nc_));
    }
-    fclose(outfp_);
+    if(outfp_){
+      fclose(outfp_);
+    }
  }

  notcurses* nc_{};
--- a/tests/ncplane.cpp
+++ b/tests/ncplane.cpp
@ -24,7 +24,9 @@ class NcplaneTest : public :: testing::Test {
    if(nc_){
      EXPECT_EQ(0, notcurses_stop(nc_));
    }
-    fclose(outfp_);
+    if(outfp_){
+      fclose(outfp_);
+    }
  }

  struct notcurses* nc_{};
--- a/tests/notcurses.cpp
+++ b/tests/notcurses.cpp
@ -23,7 +23,9 @@ class NotcursesTest : public :: testing::Test {
    if(nc_){
      EXPECT_EQ(0, notcurses_stop(nc_));
    }
-    fclose(outfp_);
+    if(outfp_){
+      fclose(outfp_);
+    }
  }

  struct notcurses* nc_{};
--- a/tests/panelreel.cpp
+++ b/tests/panelreel.cpp
@ -23,7 +23,9 @@ class PanelReelTest : public :: testing::Test {
    if(nc_){
      EXPECT_EQ(0, notcurses_stop(nc_));
    }
-    fclose(outfp_);
+    if(outfp_){
+      fclose(outfp_);
+    }
  }

  struct notcurses* nc_{};