Merge pull request #2265 from jonjove/fix-issue-2264

Update name of LiabilitiesMatchOffers invariant Reviewed-by: MonsieurNicolas
stellar · Sep 6, 2019 · 2757617 · 2757617
2 parents 859fae6 + a24f18f
commit 2757617
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/src/invariant/LiabilitiesMatchOffers.cpp b/src/invariant/LiabilitiesMatchOffers.cpp
@@ -251,10 +251,7 @@ LiabilitiesMatchOffers::LiabilitiesMatchOffers() : Invariant(false)
 std::string
 LiabilitiesMatchOffers::getName() const
 {
-    // NOTE: In order for the acceptance tests to run correctly, this will
-    // currently need to read "MinimumAccountBalance". We will update this to
-    // "LiabilitiesMatchOffers" after.
-    return "MinimumAccountBalance";
+    return "LiabilitiesMatchOffers";
 }
 
 std::string

diff --git a/src/invariant/test/LiabilitiesMatchOffersTests.cpp b/src/invariant/test/LiabilitiesMatchOffersTests.cpp
@@ -58,7 +58,7 @@ TEST_CASE("Create account above minimum balance",
           "[invariant][liabilitiesmatchoffers]")
 {
     Config cfg = getTestConfig(0);
-    cfg.INVARIANT_CHECKS = {"MinimumAccountBalance"};
+    cfg.INVARIANT_CHECKS = {"LiabilitiesMatchOffers"};
 
     for (uint32_t i = 0; i < 10; ++i)
     {
@@ -75,7 +75,7 @@ TEST_CASE("Create account below minimum balance",
           "[invariant][liabilitiesmatchoffers]")
 {
     Config cfg = getTestConfig(0);
-    cfg.INVARIANT_CHECKS = {"MinimumAccountBalance"};
+    cfg.INVARIANT_CHECKS = {"LiabilitiesMatchOffers"};
 
     for (uint32_t i = 0; i < 10; ++i)
     {
@@ -92,7 +92,7 @@ TEST_CASE("Create account then decrease balance below minimum",
           "[invariant][liabilitiesmatchoffers]")
 {
     Config cfg = getTestConfig(0);
-    cfg.INVARIANT_CHECKS = {"MinimumAccountBalance"};
+    cfg.INVARIANT_CHECKS = {"LiabilitiesMatchOffers"};
 
     for (uint32_t i = 0; i < 10; ++i)
     {
@@ -111,7 +111,7 @@ TEST_CASE("Account below minimum balance increases but stays below minimum",
           "[invariant][liabilitiesmatchoffers]")
 {
     Config cfg = getTestConfig(0);
-    cfg.INVARIANT_CHECKS = {"MinimumAccountBalance"};
+    cfg.INVARIANT_CHECKS = {"LiabilitiesMatchOffers"};
 
     for (uint32_t i = 0; i < 10; ++i)
     {
@@ -130,7 +130,7 @@ TEST_CASE("Account below minimum balance decreases",
           "[invariant][liabilitiesmatchoffers]")
 {
     Config cfg = getTestConfig(0);
-    cfg.INVARIANT_CHECKS = {"MinimumAccountBalance"};
+    cfg.INVARIANT_CHECKS = {"LiabilitiesMatchOffers"};
 
     for (uint32_t i = 0; i < 10; ++i)
     {
@@ -275,7 +275,7 @@ generateBuyingLiabilities(LedgerEntry offer, bool excess, bool authorized)
 TEST_CASE("Invariant for liabilities", "[invariant][liabilitiesmatchoffers]")
 {
     Config cfg = getTestConfig(0);
-    cfg.INVARIANT_CHECKS = {"MinimumAccountBalance"};
+    cfg.INVARIANT_CHECKS = {"LiabilitiesMatchOffers"};
 
     VirtualClock clock;
     Application::pointer app = createTestApplication(clock, cfg);