js_spec 0.1.0 → 0.2.0

data/lib/js_spec/thin/backends/js_spec_server.rb

@@ -0,0 +1,9 @@
+module Thin
+  module Backends
+    class JsSpecServer < TcpServer
+      def connect
+        @signature = EventMachine.start_server(@host, @port, JsSpecConnection, &method(:initialize_connection))
+      end
+    end
+  end
+end

data/lib/js_spec/thin/js_spec_connection.rb

@@ -0,0 +1,43 @@
+module Thin
+  class JsSpecConnection < Connection
+    def process
+      # Add client info to the request env
+      @request.remote_address = remote_address
+
+      env = @request.env
+      env['js_spec.connection'] = self
+      status, headers, body = @app.call(env)
+      unless status.to_i == 0
+        send_response(status, headers, body)
+      end
+    rescue
+      handle_error
+    end
+
+    def send_response(status, headers, body)
+      @response.status, @response.headers, @response.body = status, headers, body
+      @response.persistent! if @request.persistent?
+      @response.each do |chunk|
+        trace { chunk }
+        send_data chunk
+      end
+      # If no more request on that same connection, we close it.
+      close_connection_after_writing unless persistent?
+    rescue
+      handle_error
+    ensure
+      @request.close  rescue nil
+      @response.close rescue nil
+
+      # Prepare the connection for another request if the client
+      # supports HTTP pipelining (persistent connection).
+      post_init if persistent?
+    end
+
+    def handle_error
+      log "!! Unexpected error while processing request: #{$!.message}"
+      log_error
+      close_connection rescue nil
+    end
+  end
+end

data/lib/js_spec.rb

@@ -1,15 +1,22 @@
 require "rubygems"
-require "rack"
-require "mongrel"
+require "thin"
 require "fileutils"
 require "tmpdir"
 require "timeout"
 require "uuid"
+require "cgi"
+require "net/http"
+require "selenium"
+require "optparse"
 
 dir = File.dirname(__FILE__)
+require "#{dir}/js_spec/thin/js_spec_connection"
+require "#{dir}/js_spec/thin/backends/js_spec_server"
+require "#{dir}/js_spec/rack/response"
 require "#{dir}/js_spec/resources/runners"
 require "#{dir}/js_spec/resources/runners/firefox_runner"
 
+require "#{dir}/js_spec/client"
 require "#{dir}/js_spec/server"
 require "#{dir}/js_spec/rails_server"
 require "#{dir}/js_spec/resources/spec_runner"
@@ -20,3 +27,8 @@ require "#{dir}/js_spec/resources/dir"
 require "#{dir}/js_spec/resources/web_root"
 require "#{dir}/js_spec/resources/suite"
 require "#{dir}/js_spec/resources/suite_finish"
+
+module JsSpec
+  DEFAULT_HOST = "127.0.0.1"
+  DEFAULT_PORT = 8080  
+end

data/spec/integration/integration_spec.rb

@@ -0,0 +1,14 @@
+require File.expand_path("#{File.dirname(__FILE__)}/integration_spec_helper")
+
+describe "JsSpec" do
+  it "runs a full passing Suite" do
+    mock(JsSpec::Client).puts("SUCCESS")
+    JsSpec::Client.run(:spec_url => "#{root_url}/specs/foo/passing_spec")
+  end
+
+  it "runs a full failing Suite" do
+    mock(JsSpec::Client).puts("FAILURE")
+    mock(JsSpec::Client).puts(/A failing spec in foo fails/)
+    JsSpec::Client.run(:spec_url => "#{root_url}/specs/foo/failing_spec")
+  end
+end

data/spec/integration/integration_spec_helper.rb

@@ -0,0 +1,52 @@
+require "rubygems"
+dir = File.dirname(__FILE__)
+$LOAD_PATH.unshift "#{dir}/../../../plugins/rspec/lib"
+require "spec"
+
+$LOAD_PATH.unshift "#{dir}/../../lib"
+require "js_spec"
+require "hpricot"
+require "lsof"
+
+Spec::Runner.configure do |config|
+  config.mock_with :rr
+end
+
+Thin::Logging.silent = false
+Thin::Logging.debug = true
+
+module WaitFor
+  extend self
+  def wait_for(time=5)
+    Timeout.timeout(time) do
+      loop do
+        value = yield
+        return value if value
+      end
+    end
+  end
+end
+
+module Spec::Example::ExampleMethods
+  include WaitFor
+  attr_reader :spec_root_path, :implementation_root_path, :public_path
+  before(:all) do
+    dir = File.dirname(__FILE__)
+    @spec_root_path = "#{dir}/../example_specs"
+    @public_path = "#{dir}/../example_public"
+    @implementation_root_path = "#{public_path}/javascripts"
+    unless $js_spec_server_started
+      Thread.start do
+        JsSpec::Server.run(spec_root_path, implementation_root_path, public_path)
+      end
+      $js_spec_server_started = true
+    end
+    wait_for do
+      Lsof.running?(8080)
+    end
+  end
+
+  def root_url
+    "http://#{JsSpec::DEFAULT_HOST}:#{JsSpec::DEFAULT_PORT}"
+  end
+end

data/spec/integration_suite.rb

@@ -0,0 +1,10 @@
+class IntegrationSuite
+  def run
+    dir = File.dirname(__FILE__)
+    Dir["#{dir}/integration/**/*_spec.rb"].each do |file|
+      require file
+    end
+  end
+end
+
+IntegrationSuite.new.run

data/spec/spec_suite.rb

@@ -1,3 +1,4 @@
 dir = File.dirname(__FILE__)
 raise "Failure" unless system(%Q|ruby #{dir}/unit_suite.rb|)
 raise "Failure" unless system(%Q|ruby #{dir}/functional_suite.rb|)
+raise "Failure" unless system(%Q|ruby #{dir}/integration_suite.rb|)

data/spec/unit/js_spec/client_spec.rb

@@ -0,0 +1,137 @@
+require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
+
+module JsSpec
+  describe Client do
+    describe '.run' do
+      describe 'when successful' do
+        before do
+          request = Object.new
+          mock(request).post("/runners/firefox", "selenium_host=localhost&selenium_port=4444")
+          response = Object.new
+          mock(response).body {""}
+          mock(Net::HTTP).start(DEFAULT_HOST, DEFAULT_PORT).yields(request) {response}
+          stub(Client).puts
+        end
+
+        it "returns true" do
+          Client.run.should be_true
+        end
+
+        it "prints 'SUCCESS'" do
+          mock(Client).puts("SUCCESS")
+          Client.run
+        end
+      end
+
+      describe 'when unsuccessful' do
+        before do
+          request = Object.new
+          mock(request).post("/runners/firefox", "selenium_host=localhost&selenium_port=4444")
+          response = Object.new
+          mock(response).body {"the failure message"}
+          mock(Net::HTTP).start(DEFAULT_HOST, DEFAULT_PORT).yields(request) {response}
+          stub(Client).puts
+        end
+
+        it "returns false" do
+          Client.run.should be_false
+        end
+
+        it "prints 'FAILURE' and the error message(s)" do
+          mock(Client).puts("FAILURE")
+          mock(Client).puts("the failure message")
+          Client.run
+        end
+      end
+
+      describe "arguments" do
+        attr_reader :request, :response
+        before do
+          @request = Object.new
+          @response = Object.new
+          mock(response).body {""}
+          mock(Net::HTTP).start(DEFAULT_HOST, DEFAULT_PORT).yields(request) {response}
+          stub(Client).puts
+        end
+
+        describe "when passed a custom spec_url" do
+          it "passes the spec_url as a post parameter" do
+            spec_url = 'http://foobar.com/foo'
+            mock(request).post(
+              "/runners/firefox",
+              "selenium_host=localhost&selenium_port=4444&spec_url=#{CGI.escape(spec_url)}"
+            )
+            Client.run(:spec_url => spec_url)
+          end
+        end
+
+        describe "when passed a custom selenium host" do
+          it "passes the selenium_host as a post parameter" do
+            selenium_host = 'test-runner'
+            mock(request).post(
+              "/runners/firefox",
+              "selenium_host=test-runner&selenium_port=4444"
+            )
+            Client.run(:selenium_host => selenium_host)
+          end
+        end
+
+        describe "when passed a custom selenium port" do
+          it "passes the selenium_port as a post parameter" do
+            selenium_port = 5000
+            mock(request).post(
+              "/runners/firefox",
+              "selenium_host=localhost&selenium_port=5000"
+            )
+            Client.run(:selenium_port => selenium_port)
+          end
+        end
+      end
+
+    end
+    
+    describe ".run_argv" do
+      attr_reader :request, :response
+      before do
+          @request = Object.new
+          @response = Object.new
+          mock(response).body {""}
+          mock(Net::HTTP).start(DEFAULT_HOST, DEFAULT_PORT).yields(request) {response}
+          stub(Client).puts
+        end
+
+      describe "when passed a custom spec_url" do
+        it "passes the spec_url as a post parameter" do
+          spec_url = 'http://foobar.com/foo'
+          mock(request).post(
+            "/runners/firefox",
+            "selenium_host=localhost&selenium_port=4444&spec_url=#{CGI.escape(spec_url)}"
+          )
+          Client.run_argv(['--spec_url', spec_url])
+        end
+      end
+
+      describe "when passed a custom selenium host" do
+        it "passes the selenium_host as a post parameter" do
+          selenium_host = 'test-runner'
+          mock(request).post(
+            "/runners/firefox",
+            "selenium_host=test-runner&selenium_port=4444"
+          )
+          Client.run_argv(['--selenium_host', selenium_host])
+        end
+      end
+
+      describe "when passed a custom selenium port" do
+        it "passes the selenium_port as a post parameter" do
+          selenium_port = 5000
+          mock(request).post(
+            "/runners/firefox",
+            "selenium_host=localhost&selenium_port=5000"
+          )
+          Client.run_argv(['--selenium_port', selenium_port.to_s])
+        end
+      end
+    end
+  end
+end

data/spec/unit/{rails_server_spec.rb → js_spec/rails_server_spec.rb}

@@ -1,4 +1,4 @@
-require File.expand_path("#{File.dirname(__FILE__)}/unit_spec_helper")
+require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
 
 module JsSpec
   describe RailsServer do
@@ -13,19 +13,20 @@ module JsSpec
         Server.instance = nil
       end
 
-      it "initializes the RailsServer and runs the Mongrel Handler and sets Server.instance to the RailsServer instance" do
-        host = Server::DEFAULT_HOST
-        port = Server::DEFAULT_PORT
-        server_instance = Object.new
+      it "initializes the RailsServer and runs the Thin Handler and sets Server.instance to the RailsServer instance" do
+        host = DEFAULT_HOST
+        port = DEFAULT_PORT
+        server_instance = nil
         mock.proxy(RailsServer).new(
           rails_root,
           host,
           port
         ) do |new_instance|
-          server_instance
+          server_instance = new_instance
         end
-        mock(Rack::Handler::Mongrel).run(server_instance, {:Host => host, :Port => port})
 
+        mock(EventMachine).run.yields
+        mock(EventMachine).start_server(host, port, ::Thin::JsSpecConnection)
         RailsServer.run(rails_root)
         Server.instance.should == server_instance
       end

data/spec/unit/{resources → js_spec/resources}/dir_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
 
 module JsSpec
   module Resources

data/spec/unit/{resources → js_spec/resources}/file_spec.rb

@@ -1,12 +1,15 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
 
 module JsSpec
   module Resources
     describe File do
-    attr_reader :file
+    attr_reader :request, :file
 
     before do
+      stub(EventMachine).send_data
+      stub(EventMachine).close_connection
       @file = Resources::File.new(absolute_path, relative_path)
+      @request = create_request('get', relative_path)
     end
 
     def absolute_path
@@ -33,24 +36,24 @@ module JsSpec
       attr_reader :response
       before do
         @response = Rack::Response.new
-        mock.proxy(Server).response.returns(@response)
       end
 
       it "returns the contents of the file" do
-        file.get.should == ::File.read(absolute_path)
+        file.get(request, response)
+        response.body.should == ::File.read(absolute_path)
       end
 
       describe "when File has an extension" do
         describe '.js' do
           it "sets Content-Type to text/javascript" do
-            file.get
+            file.get(request,response)
             response.content_type.should == "text/javascript"
           end
         end
 
         describe '.css' do
           it "sets Content-Type to text/css" do
-            file.get
+            file.get(request, response)
             response.content_type.should == "text/css"
           end
 

data/spec/unit/js_spec/resources/runner_spec.rb

@@ -0,0 +1,24 @@
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
+
+module JsSpec
+  module Resources
+    describe Runners do
+      attr_reader :runner
+      before do
+        @runner = Runners.new
+      end
+
+      describe "#locate" do
+        it "when passed 'firefox', returns a Firefox1Runner" do
+          runner.locate('firefox').is_a?(Runners::FirefoxRunner).should be_true
+        end
+
+        it "when not passed 'firefox', raises an error" do
+          lambda do
+            runner.locate('invalid')
+          end.should raise_error
+        end
+      end
+    end
+  end
+end

data/spec/unit/js_spec/resources/runners/firefox_runner_spec.rb

@@ -0,0 +1,152 @@
+require File.expand_path("#{File.dirname(__FILE__)}/../../../unit_spec_helper")
+
+module JsSpec
+  module Resources
+    describe Runners::FirefoxRunner do
+      attr_reader :runner, :request, :response, :driver
+
+      before do
+        Thread.current[:connection] = connection
+        @driver = "Selenium Driver"
+        stub(Selenium::SeleniumDriver).new('localhost', 4444, '*firefox', 'http://127.0.0.1:8080') do
+          driver
+        end
+      end
+
+      describe "#post" do
+        attr_reader :firefox_profile_path
+        before do
+          @request = Rack::Request.new( Rack::MockRequest.env_for('/runners/firefox') )
+          @response = Rack::Response.new
+          @runner = Runners::FirefoxRunner.new(request, response)
+          stub(Thread).start.yields
+        end
+
+        it "keeps the connection open" do
+          stub(driver).start
+          stub(driver).open
+          dont_allow(EventMachine).send_data
+          dont_allow(EventMachine).close_connection
+          runner.post(request, response)
+
+          response.should_not be_ready
+        end
+        
+        describe "when a selenium_host parameter is passed into the request" do
+          before do
+            request['selenium_host'] = "another-machine"
+            stub(driver).start
+            stub(driver).open
+          end
+
+          it "starts the Selenium Driver with the passed in selenium_host" do
+            mock(Selenium::SeleniumDriver).new('another-machine', 4444, '*firefox', 'http://127.0.0.1:8080') do
+              driver
+            end
+            runner.post(request, response)
+          end
+        end
+
+        describe "when a selenium_host parameter is not passed into the request" do
+          before do
+            request['selenium_host'].should be_nil
+            stub(driver).start
+            stub(driver).open
+          end
+
+          it "starts the Selenium Driver from localhost" do
+            mock(Selenium::SeleniumDriver).new('localhost', 4444, '*firefox', 'http://127.0.0.1:8080') do
+              driver
+            end
+            runner.post(request, response)
+          end
+        end
+
+        describe "when a selenium_port parameter is passed into the request" do
+          before do
+            request['selenium_port'] = "4000"
+            stub(driver).start
+            stub(driver).open
+          end
+
+          it "starts the Selenium Driver with the passed in selenium_port" do
+            mock(Selenium::SeleniumDriver).new('localhost', 4000, '*firefox', 'http://127.0.0.1:8080') do
+              driver
+            end
+            runner.post(request, response)
+          end
+        end
+
+        describe "when a selenium_port parameter is not passed into the request" do
+          before do
+            request['selenium_port'].should be_nil
+            stub(driver).start
+            stub(driver).open
+          end
+
+          it "starts the Selenium Driver from localhost" do
+            mock(Selenium::SeleniumDriver).new('localhost', 4444, '*firefox', 'http://127.0.0.1:8080') do
+              driver
+            end
+            runner.post(request, response)
+          end
+        end
+
+        describe "when a spec_url is passed into the request" do
+          before do
+            request['spec_url'] = "http://another-host:8080/specs/subdir"
+          end
+
+          it "runs Selenium with the passed in host and part to run the specified spec suite in Firefox" do
+            mock(Selenium::SeleniumDriver).new('localhost', 4444, '*firefox', 'http://another-host:8080') do
+              driver
+            end
+            mock(driver).start
+            mock(driver).open("http://another-host:8080/specs/subdir?guid=#{runner.guid}")
+
+            runner.post(request, response)
+          end
+        end
+
+        describe "when a spec_url is not passed into the request" do
+          before do
+            request['spec_url'].should be_nil
+            mock(Selenium::SeleniumDriver).new('localhost', 4444, '*firefox', 'http://127.0.0.1:8080') do
+              driver
+            end
+          end
+
+          it "uses Selenium to run the entire spec suite in Firefox" do
+            mock(driver).start
+            mock(driver).open("http://127.0.0.1:8080/specs?guid=#{runner.guid}")
+
+            runner.post(request, response)
+          end
+        end
+      end
+
+      describe "#finalize" do
+        before do
+          @request = Rack::Request.new( Rack::MockRequest.env_for('/runners/firefox') )
+          @response = Rack::Response.new
+          @runner = Runners::FirefoxRunner.new(request, response)
+          stub(driver).start
+          stub(driver).open
+          runner.post(request, response)
+        end
+
+        it "kills the browser, sends the response body, and close the connection" do
+          mock(driver).stop
+          data = ""
+          stub(EventMachine).send_data do |signature, data, data_length|
+            data << data
+          end
+          mock(connection).close_connection_after_writing
+
+          runner.finalize("The text")
+          data.should include("The text")
+        end
+      end
+    end
+  end
+end

data/spec/unit/js_spec/resources/spec_dir_runner_spec.rb

@@ -0,0 +1,48 @@
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
+
+module JsSpec
+  module Resources
+    describe SpecDirRunner do
+      attr_reader :dir, :runner
+
+      before do
+        stub(EventMachine).send_data
+        stub(EventMachine).close_connection
+        @dir = Dir.new(spec_root_path, '/specs')
+        @runner = SpecDirRunner.new(dir)
+      end
+
+      describe "#spec_files" do
+        it "returns a File for each *_spec.js file in the directory" do
+          spec_files = runner.spec_files
+
+          spec_files.should contain_spec_file_with_correct_paths("/failing_spec.js")
+          spec_files.should contain_spec_file_with_correct_paths("/foo/failing_spec.js")
+          spec_files.should contain_spec_file_with_correct_paths("/foo/passing_spec.js")
+        end
+      end
+
+      describe "#get" do
+        attr_reader :html, :doc
+        before do
+          request = request('get', '/specs')
+          response = Rack::Response.new
+          runner.get(request, response)
+          @html = response.body
+          @doc = Hpricot(html)
+        end
+
+        it "returns script tags for each test javascript file" do
+          doc.at("script[@src='/specs/failing_spec.js']").should exist
+        end
+
+        it "returns the js specs template" do
+          doc.at("link[@href='/core/JSSpec.css']").should exist
+          doc.at("script[@src='/core/JSSpec.js']").should exist
+          doc.at("script[@src='/core/JSSpecExtensions.js']").should exist
+          doc.at("body").inner_html.should be_empty
+        end
+      end
+    end
+  end
+end

data/spec/unit/{resources → js_spec/resources}/spec_file_runner_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
 
 module JsSpec
   module Resources

data/spec/unit/{resources → js_spec/resources}/suite_finish_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
 
 module JsSpec
   module Resources
@@ -21,9 +21,9 @@ module JsSpec
           body = "The text in the POST body"
           request = Rack::Request.new({'rack.input' => StringIO.new("text=#{body}")})
           request.body.string.should == "text=#{body}"
-          stub.proxy(Server).request {request}
+          response = Rack::Response.new
 
-          suite_finish.post
+          suite_finish.post(request, response)
           stdout.string.should == "#{body}\n"
         end
 
@@ -31,9 +31,9 @@ module JsSpec
           body = "The text in the POST body"
           request = Rack::Request.new({'rack.input' => StringIO.new("text=#{body}")})
           request.body.string.should == "text=#{body}"
-          stub.proxy(Server).request {request}
+          response = Rack::Response.new
 
-          suite_finish.post.should == ""
+          suite_finish.post(request, response).should == ""
         end
       end
     end

data/spec/unit/{resources → js_spec/resources}/suite_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
 
 module JsSpec
   module Resources

data/spec/unit/{resources → js_spec/resources}/web_root_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path("#{File.dirname(__FILE__)}/../unit_spec_helper")
+require File.expand_path("#{File.dirname(__FILE__)}/../../unit_spec_helper")
 
 module JsSpec
   module Resources