obey the cops

Stephen Crosby
1 parent 70d21539
Showing 2 changed files with 19 additions and 13 deletions Show diff stats
app/models/notification_services/slack_service.rb
spec/models/notification_service/slack_service_spec.rb
@@ -19,8 +19,8 @@ class NotificationServices::SlackService &lt; NotificationService
   def post_payload(problem)
     {
-      username: "Errbit",
-      icon_emoji: ":collision:",
+      username:    "Errbit",
+      icon_emoji:  ":collision:",
       attachments: [
         {
           fallback:   message_for_slack(problem),
@@ -28,7 +28,7 @@ class NotificationServices::SlackService &lt; NotificationService
           title_link: problem.url,
           text:       problem.where,
           color:      "#D00000",
-          fields: [
+          fields:     [
             {
               title: "Application",
               value: problem.app.name,
 describe NotificationServices::SlackService, type: 'model' do
+  let(:notice) { Fabricate :notice }
+  let(:service_url) do
+    "https://hooks.slack.com/services/XXXXXXXXX/XXXXXXXXX/XXXXXXXXX"
+  end
+
+  let(:service) do
+    Fabricate :slack_notification_service, app:         notice.app,
+                                           service_url: service_url
+  end
+
   it "it should send a notification to Slack with hook url" do
     # setup
-    notice = Fabricate :notice
-    notification_service = Fabricate :slack_notification_service,
-      app: notice.app,
-      service_url: "https://hooks.slack.com/services/XXXXXXXXX/XXXXXXXXX/XXXXXXXXX"
     problem = notice.problem
     # faraday stubbing
     payload = {
-      username: "Errbit",
-      icon_emoji: ":collision:",
+      username:    "Errbit",
+      icon_emoji:  ":collision:",
       attachments: [
         {
-          fallback:   notification_service.message_for_slack(problem),
+          fallback:   service.message_for_slack(problem),
           title:      problem.message.to_s.truncate(100),
           title_link: problem.url,
           text:       problem.where,
           color:      "#D00000",
-          fields: [
+          fields:     [
             {
               title: "Application",
               value: problem.app.name,
@@ -43,8 +49,8 @@ describe NotificationServices::SlackService, type: &#39;model&#39; do
         }
       ]
     }.to_json
-    expect(HTTParty).to receive(:post).with(notification_service.service_url, body: payload, headers: { "Content-Type" => "application/json" }).and_return(true)
+    expect(HTTParty).to receive(:post).with(service.service_url, body: payload, headers: { "Content-Type" => "application/json" }).and_return(true)
-    notification_service.create_notification(problem)
+    service.create_notification(problem)
   end
 end