Commit 22476f6c authored by Marc Burnie [AWS]'s avatar Marc Burnie [AWS]
Browse files

Merge branch 'enable-publishing-tag-status-events' into 'master'

re-enabling publishing of legal tag status change events

See merge request !255

(cherry picked from commit c9657407)

61e3f01c re-enabling publishing of legal tag status change events
e2fed383 re-enabling tests for publishing status events to message queue
parent 5a441682
Pipeline #118358 passed with stages
in 33 minutes and 2 seconds
......@@ -30,8 +30,7 @@ public class LegalTagStatusJob {
statusChangedTags = checkAndUpdateLegalTagStatus(true, tenantName, statusChangedTags);
statusChangedTags = checkAndUpdateLegalTagStatus(false, tenantName, statusChangedTags);
//TODO: Uncomment following event publishing method after this issue is resolved:
//publishLegalTagStatusUpdateEvents(!statusChangedTags.getStatusChangedTags().isEmpty(), projectId, headers, statusChangedTags);
publishLegalTagStatusUpdateEvents(!statusChangedTags.getStatusChangedTags().isEmpty(), projectId, headers, statusChangedTags);
return statusChangedTags;
......@@ -89,7 +89,6 @@ public class LegalTagStatusJobTests {
assertEquals(LegalTagCompliance.incompliant, result.getStatusChangedTags().get(0).getChangedTagStatus());
public void should_sendLegalTagNameAndNewStatusWithCorrelationIdToPubSub_when_legalTagBecomesIncompliant() throws
Exception {
......@@ -105,7 +104,6 @@ public class LegalTagStatusJobTests {
verify(messagePublisherMock, times(1)).publish(any(), any(), any());
public void should_notSendMessageToPubSub_when_noLegalTagStatusHasChanged() throws Exception {
Collection<LegalTag> validLegalTags = new ArrayList<>();
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment