From 666bdb5fda36ff89c9832d2202aadaca8d2e5d48 Mon Sep 17 00:00:00 2001
From: nick black <dankamongmen@gmail.com>
Date: Sun, 16 Aug 2020 06:52:45 -0400
Subject: [PATCH] ncreel unit tests: call ncreel_destroy()

---
 tests/reel.cpp | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/tests/reel.cpp b/tests/reel.cpp
index c6850956a..15ab02366 100644
--- a/tests/reel.cpp
+++ b/tests/reel.cpp
@@ -79,6 +79,8 @@ TEST_CASE("Reels") {
     ncreel_options r = { };
     struct ncreel* nr = ncreel_create(n_, &r);
     REQUIRE(nr);
+    CHECK(0 == notcurses_render(nc_));
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("InitLinearInfinite") {
@@ -86,6 +88,8 @@ TEST_CASE("Reels") {
     r.flags = NCREEL_OPTION_INFINITESCROLL;
     struct ncreel* nr = ncreel_create(n_, &r);
     REQUIRE(nr);
+    CHECK(0 == notcurses_render(nc_));
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("InitCircular") {
@@ -94,6 +98,7 @@ TEST_CASE("Reels") {
     struct ncreel* nr = ncreel_create(n_, &r);
     REQUIRE(nr);
     CHECK(ncreel_validate(nr));
+    CHECK(0 == notcurses_render(nc_));
     REQUIRE(0 == ncreel_destroy(nr));
   }
 
@@ -103,6 +108,8 @@ TEST_CASE("Reels") {
     r.flags = NCREEL_OPTION_CIRCULAR;
     struct ncreel* nr = ncreel_create(n_, &r);
     REQUIRE(!nr);
+    CHECK(0 == notcurses_render(nc_));
+    REQUIRE(0 == ncreel_destroy(nr));
   }
 
   // We ought be able to invoke ncreel_next() and ncreel_prev() safely,
@@ -119,6 +126,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    REQUIRE(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("OneTablet") {
@@ -134,6 +142,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    REQUIRE(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("MovementWithOneTablet") {
@@ -157,6 +166,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    REQUIRE(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("DeleteActiveTablet") {
@@ -169,6 +179,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    REQUIRE(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("NoBorder") {
@@ -180,6 +191,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("BadBorderBitsRejected") {
@@ -198,6 +210,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("NoTopBottomBorder") {
@@ -208,6 +221,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("NoSideBorders") {
@@ -218,6 +232,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   SUBCASE("BadTabletBorderBitsRejected") {
@@ -235,6 +250,7 @@ TEST_CASE("Reels") {
     CHECK_EQ(0, ncreel_redraw(nr));
     CHECK_EQ(0, notcurses_render(nc_));
     CHECK(ncreel_validate(nr));
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   // Layout tests. Add some tablets, move around, and verify that they all
@@ -318,6 +334,7 @@ TEST_CASE("Reels") {
       CHECK(y == expectedy);
       expectedy += 7;
     }
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   // Layout tests. Add some tablets, move around, and verify that they all
@@ -402,6 +419,7 @@ TEST_CASE("Reels") {
       CHECK(y == expectedy);
       expectedy += 5;
     }
+    CHECK(0 == ncreel_destroy(nr));
   }
 
   CHECK(0 == notcurses_stop(nc_));