suzanne.soy/travis-web
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't setup pusher if we don't have key

Browse Source 
That way we don't need to even fire anything pusher related when running
tests
This commit is contained in:
Piotr Sarnacki 2013-03-27 20:06:07 +01:00
parent 745b3a0167
commit d5bfa092b0
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
assets/scripts/app/app.coffee
View File

@ -10,7 +10,7 @@ unless window.TravisApplication
@store.loadMany(Travis.Sponsor, Travis.SPONSORS) @store.loadMany(Travis.Sponsor, Travis.SPONSORS)
@slider = new Travis.Slider() @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() @tailing = new Travis.Tailing()
@set('auth', Travis.Auth.create(app: this, endpoint: Travis.config.api_endpoint)) @set('auth', Travis.Auth.create(app: this, endpoint: Travis.config.api_endpoint))

6
assets/scripts/app/models/job.coffee
View File

@ -87,12 +87,14 @@ require 'travis/model'
subscribe: -> subscribe: ->
return if @get('subscribed') return if @get('subscribed')
@set('subscribed', true) @set('subscribed', true)
Travis.pusher.subscribe "job-#{@get('id')}" if Travis.pusher
Travis.pusher.subscribe "job-#{@get('id')}"
unsubscribe: -> unsubscribe: ->
return unless @get('subscribed') return unless @get('subscribed')
@set('subscribed', false) @set('subscribed', false)
Travis.pusher.unsubscribe "job-#{@get('id')}" if Travis.pusher
Travis.pusher.unsubscribe "job-#{@get('id')}"
onStateChange: (-> onStateChange: (->
if @get('state') == 'finished' && Travis.pusher if @get('state') == 'finished' && Travis.pusher