Browse Source

Minor improvements for internal system notifications

pull/9104/head
ViacheslavKlimov 3 years ago
parent
commit
2a87f2ee7e
  1. 2
      application/src/main/java/org/thingsboard/server/service/notification/DefaultNotificationCenter.java
  2. 1
      application/src/test/java/org/thingsboard/server/service/notification/NotificationApiTest.java

2
application/src/main/java/org/thingsboard/server/service/notification/DefaultNotificationCenter.java

@ -207,7 +207,7 @@ public class DefaultNotificationCenter extends AbstractSubscriptionService imple
try { try {
processForTarget(target, ctx); processForTarget(target, ctx);
} catch (Exception e) { } catch (Exception e) {
log.error("[{}] Failed to process notification request for target {}", requestId, target.getId()); log.error("[{}] Failed to process notification request for target {}", requestId, target.getId(), e);
} }
} }
log.debug("[{}] Notification request processing is finished", requestId); log.debug("[{}] Notification request processing is finished", requestId);

1
application/src/test/java/org/thingsboard/server/service/notification/NotificationApiTest.java

@ -619,7 +619,6 @@ public class NotificationApiTest extends AbstractNotificationApiTest {
Notification notification = getAnotherWsClient().getLastDataUpdate().getUpdate(); Notification notification = getAnotherWsClient().getLastDataUpdate().getUpdate();
assertThat(notification.getSubject()).isEqualTo(expectedNotification.getSubject()); assertThat(notification.getSubject()).isEqualTo(expectedNotification.getSubject());
assertThat(notification.getText()).isEqualTo(expectedNotification.getText()); assertThat(notification.getText()).isEqualTo(expectedNotification.getText());
System.err.println(notification);
} }
@Test @Test

Loading…
Cancel
Save