From d5bfa092b0afd16c87a637d26cadc1612fd92d78 Mon Sep 17 00:00:00 2001 From: Piotr Sarnacki Date: Wed, 27 Mar 2013 20:06:07 +0100 Subject: [PATCH] Don't setup pusher if we don't have key That way we don't need to even fire anything pusher related when running tests --- assets/scripts/app/app.coffee | 2 +- assets/scripts/app/models/job.coffee | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/assets/scripts/app/app.coffee b/assets/scripts/app/app.coffee index c296f5eb..fe42c9f2 100644 --- a/assets/scripts/app/app.coffee +++ b/assets/scripts/app/app.coffee @@ -10,7 +10,7 @@ unless window.TravisApplication @store.loadMany(Travis.Sponsor, Travis.SPONSORS) @slider = new Travis.Slider() - @pusher = new Travis.Pusher(Travis.config.pusher_key) + @pusher = new Travis.Pusher(Travis.config.pusher_key) if Travis.config.pusher_key @tailing = new Travis.Tailing() @set('auth', Travis.Auth.create(app: this, endpoint: Travis.config.api_endpoint)) diff --git a/assets/scripts/app/models/job.coffee b/assets/scripts/app/models/job.coffee index 98334b06..e54a717c 100644 --- a/assets/scripts/app/models/job.coffee +++ b/assets/scripts/app/models/job.coffee @@ -87,12 +87,14 @@ require 'travis/model' subscribe: -> return if @get('subscribed') @set('subscribed', true) - Travis.pusher.subscribe "job-#{@get('id')}" + if Travis.pusher + Travis.pusher.subscribe "job-#{@get('id')}" unsubscribe: -> return unless @get('subscribed') @set('subscribed', false) - Travis.pusher.unsubscribe "job-#{@get('id')}" + if Travis.pusher + Travis.pusher.unsubscribe "job-#{@get('id')}" onStateChange: (-> if @get('state') == 'finished' && Travis.pusher