Skip to content

Commit 41cb14d

Browse files
authored
fix(revocation): save database changes afert revocation (eclipse-tractusx#159)
Reviewed-By: Evelyn Gurschler <[email protected]>
1 parent 9a9b4f4 commit 41cb14d

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

src/issuer/SsiCredentialIssuer.Service/BusinessLogic/RevocationBusinessLogic.cs

+2
Original file line numberDiff line numberDiff line change
@@ -78,5 +78,7 @@ public async Task RevokeCredential(Guid credentialId, bool revokeForIssuer, Canc
7878
credentialRepository.AttachAndModifyCredential(credentialId,
7979
x => x.CompanySsiDetailStatusId = data.StatusId,
8080
x => x.CompanySsiDetailStatusId = CompanySsiDetailStatusId.REVOKED);
81+
82+
await _repositories.SaveAsync().ConfigureAwait(ConfigureAwaitOptions.None);
8183
}
8284
}

tests/issuer/SsiCredentialIssuer.Service.Tests/BusinessLogic/RevocationBusinessLogicTests.cs

+6-4
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ public class RevocationBusinessLogicTests
2424
private readonly IWalletService _walletService;
2525
private readonly IIdentityService _identityService;
2626
private readonly IIdentityData _identityData;
27+
private readonly IIssuerRepositories _issuerRepositories;
2728

2829
public RevocationBusinessLogicTests()
2930
{
@@ -32,7 +33,7 @@ public RevocationBusinessLogicTests()
3233
.ForEach(b => _fixture.Behaviors.Remove(b));
3334
_fixture.Behaviors.Add(new OmitOnRecursionBehavior());
3435

35-
var issuerRepositories = A.Fake<IIssuerRepositories>();
36+
_issuerRepositories = A.Fake<IIssuerRepositories>();
3637
_documentRepository = A.Fake<IDocumentRepository>();
3738
_credentialRepository = A.Fake<ICredentialRepository>();
3839
_walletService = A.Fake<IWalletService>();
@@ -41,10 +42,10 @@ public RevocationBusinessLogicTests()
4142
A.CallTo(() => _identityData.Bpnl).Returns(Bpnl);
4243
A.CallTo(() => _identityService.IdentityData).Returns(_identityData);
4344

44-
A.CallTo(() => issuerRepositories.GetInstance<IDocumentRepository>()).Returns(_documentRepository);
45-
A.CallTo(() => issuerRepositories.GetInstance<ICredentialRepository>()).Returns(_credentialRepository);
45+
A.CallTo(() => _issuerRepositories.GetInstance<IDocumentRepository>()).Returns(_documentRepository);
46+
A.CallTo(() => _issuerRepositories.GetInstance<ICredentialRepository>()).Returns(_credentialRepository);
4647

47-
_sut = new RevocationBusinessLogic(issuerRepositories, _walletService, _identityService);
48+
_sut = new RevocationBusinessLogic(_issuerRepositories, _walletService, _identityService);
4849
}
4950

5051
#region RevokeIssuerCredential
@@ -155,6 +156,7 @@ public async Task RevokeIssuerCredential_WithValid_CallsExpected()
155156
A.CallTo(() => _walletService.RevokeCredentialForIssuer(A<Guid>._, A<CancellationToken>._)).MustHaveHappenedOnceExactly();
156157
document.DocumentStatusId.Should().Be(DocumentStatusId.INACTIVE);
157158
credential.CompanySsiDetailStatusId.Should().Be(CompanySsiDetailStatusId.REVOKED);
159+
A.CallTo(() => _issuerRepositories.SaveAsync()).MustHaveHappenedOnceExactly();
158160
}
159161

160162
#endregion

0 commit comments

Comments
 (0)