chore(hipchat): remove Hipchat integration (#533)

spinnaker · May 15, 2019 · d175913 · d175913
1 parent d71c9d1
commit d175913
Show file tree

Hide file tree

Showing 8 changed files with 2 additions and 327 deletions.
diff --git a/README.md b/README.md
@@ -8,7 +8,8 @@ It provides integrations for outgoing notifications in the echo-notifications pa
 
 *  email
 *  Slack
-*  Hipchat
+*  Bearychat
+*  Google Chat
 *  sms ( via Twilio )
 
 Echo is also able to send events within Spinnaker to a predefined url, which is configurable under the echo-rest module.

diff --git a/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/api/Notification.groovy b/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/api/Notification.groovy
@@ -38,7 +38,6 @@ class Notification {
     EMAIL,
     GITHUB_STATUS,
     GOOGLECHAT,
-    HIPCHAT,
     JIRA,
     PAGER_DUTY,
     PUBSUB,

diff --git a/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/config/HipchatConfig.groovy b/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/config/HipchatConfig.groovy
diff --git a/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/hipchat/HipchatMessage.groovy b/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/hipchat/HipchatMessage.groovy
diff --git a/...ions/src/main/groovy/com/netflix/spinnaker/echo/hipchat/HipchatNotificationService.groovy b/...ions/src/main/groovy/com/netflix/spinnaker/echo/hipchat/HipchatNotificationService.groovy
diff --git a/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/hipchat/HipchatService.groovy b/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/hipchat/HipchatService.groovy
diff --git a/...s/src/main/groovy/com/netflix/spinnaker/echo/notification/HipchatNotificationAgent.groovy b/...s/src/main/groovy/com/netflix/spinnaker/echo/notification/HipchatNotificationAgent.groovy
diff --git a/...ns/src/test/groovy/com/netflix/spinnaker/echo/notification/NotificationServiceSpec.groovy b/...ns/src/test/groovy/com/netflix/spinnaker/echo/notification/NotificationServiceSpec.groovy
@@ -17,9 +17,6 @@
 package com.netflix.spinnaker.echo.notification
 
 import com.netflix.spinnaker.echo.api.Notification
-import com.netflix.spinnaker.echo.hipchat.HipchatMessage
-import com.netflix.spinnaker.echo.hipchat.HipchatNotificationService
-import com.netflix.spinnaker.echo.hipchat.HipchatService
 import com.netflix.spinnaker.echo.twilio.TwilioNotificationService
 import com.netflix.spinnaker.echo.twilio.TwilioService
 import org.springframework.boot.autoconfigure.freemarker.FreeMarkerNonWebConfiguration
@@ -42,30 +39,6 @@ class NotificationServiceSpec extends Specification {
     )
   }
 
-  void "should send specific hipchat message"() {
-    given:
-    def hipchatService = Mock(HipchatService)
-    def hipchatNotificationService = new HipchatNotificationService(
-        token: "token",
-        notificationTemplateEngine: notificationTemplateEngine,
-        hipchat: hipchatService
-    )
-    def notification = new Notification(
-        notificationType: "HIPCHAT",
-        templateGroup: "example",
-        to: ["room"],
-        source: new Notification.Source(application: "application")
-    )
-
-    when:
-    hipchatNotificationService.handle(notification)
-
-    then:
-    1 * hipchatService.sendMessage("token", "room", { HipchatMessage message ->
-      message.message == "specific SPINNAKER_URL application"
-    } as HipchatMessage)
-    }
-
   void "should send generic twilio message"() {
     given:
     def twilioService = Mock(TwilioService)