ApprovalTests.Net icon indicating copy to clipboard operation
ApprovalTests.Net copied to clipboard

fix typo in IApprovalApprover description

Open askazakov opened this issue 2 years ago • 1 comments

Description

fix typo that misleads about call to Approve() and Fail()

Fail() actually invoked only if Approve() returns false

askazakov avatar Nov 17 '23 10:11 askazakov

probably the whole comment should be rewritten because actual logic a little more complicated then just

if (!Approve()) {
 Fail()
}

askazakov avatar Nov 17 '23 10:11 askazakov