Refactor integration tests to allow configuration of Faye port for te…

…sting.
kandanapp · Mar 13, 2013 · 9ea95bc · 9ea95bc
1 parent 6187c23
commit 9ea95bc
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/app/assets/javascripts/backbone/broadcasters/faye.js.coffee.erb b/app/assets/javascripts/backbone/broadcasters/faye.js.coffee.erb
@@ -1,7 +1,7 @@
 class Kandan.Broadcasters.FayeBroadcaster
 
   constructor: ()->
-    @fayeClient = new Faye.Client("<%= ENV['FULL_HOST'] %>/remote/faye")
+    @fayeClient = new Faye.Client("<%= ENV['KANDAN_FAYE_URL'] %>/remote/faye")
 
     @fayeClient.disable('websocket')
     authExtension = {

diff --git a/config/environments/test.rb b/config/environments/test.rb
@@ -36,6 +36,7 @@
   config.active_support.deprecation = :stderr
 
   # Variable set to be able to get faye client for test environments
-  ENV['FULL_HOST'] = "http://localhost:9292"
+  ENV['KANDAN_FAYE_PORT'] = "9292" unless ENV['KANDAN_FAYE_PORT']
+  ENV['KANDAN_FAYE_URL'] = "http://localhost:#{ENV['KANDAN_FAYE_PORT']}" unless ENV['KANDAN_FAYE_URL']
 
 end
diff --git a/lib/broadcasters/faye.rb b/lib/broadcasters/faye.rb
@@ -13,7 +13,7 @@ def broadcast(channel, message)
       end
 
       def assets
-        ["#{ENV['FULL_HOST']}/remote/faye.js"]
+        ["#{ENV['KANDAN_FAYE_URL']}/remote/faye.js"]
       end
     end
   end

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -34,7 +34,7 @@
   ActiveUsers.remove_by_client_id(client_id)
 end
 
-Thread.new {faye_server.listen(9292)}
+Thread.new {faye_server.listen(ENV['KANDAN_FAYE_PORT'].to_i)}
 
 Capybara.app = Rack::URLMap.new({
   "/"        => Kandan::Application