Skip to content

Commit

Permalink
Remove flaky part of config management test (digital-asset#12726)
Browse files Browse the repository at this point in the history
* Remove flaky part of config management test

CHANGELOG_BEGIN
CHANGELOG_END

* fix wrong assertion text
  • Loading branch information
mziolekda authored Feb 2, 2022
1 parent 2ca639f commit 0f0183a
Showing 1 changed file with 1 addition and 19 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -63,16 +63,6 @@ final class ConfigManagementServiceIT extends LedgerTestSuite {
// Verify that we've restored the original time model
response3 <- ledger.getTimeModel()

// Try to set a time model with an expired MRT.
t3 <- ledger.time()
expiredMRTFailure <- ledger
.setTimeModel(
mrt = t3.minusSeconds(10),
generation = response3.configurationGeneration,
newTimeModel = oldTimeModel,
)
.mustFail("setting a time model with an expired MRT")

// Above operation finished on a timeout, but may still succeed asynchronously.
// Stabilize the ledger state before leaving the test case.
_ <- stabilize(ledger)
Expand All @@ -88,15 +78,7 @@ final class ConfigManagementServiceIT extends LedgerTestSuite {
assert(response2.timeModel.contains(newTimeModel), "Setting the new time model failed")
assert(
response3.timeModel.equals(response1.timeModel),
"Restoring the original time model failed",
)

assertGrpcError(
ledger,
expiredMRTFailure,
Status.Code.ABORTED,
LedgerApiErrors.RequestTimeOut,
exceptionMessageSubstring = None,
"Restoring the original time model succeeded",
)
}
})
Expand Down

0 comments on commit 0f0183a

Please sign in to comment.