watir-rails 2.0.0 → 2.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 20dfc23f553782527d053042e02c2e58b2661c96
-  data.tar.gz: 22654fad486f48652f1a184ece70a3d54352b444
+SHA256:
+  metadata.gz: '0890e02a5cef434105541f81252240ad701f28bb828643f84aaf6b6745eb6cbf'
+  data.tar.gz: b13169b1ea1a2bbfa49b79968eb5568e3137e2319e48a5172e70ab7f0f50572d
 SHA512:
-  metadata.gz: cd85bf3cf5e6a8c31d6f152f1deaece79d3519164f0bb608f9b6deb24dc57e1c0a1351651c606c7dbdf0c9142ad64bf58092510ce2a654b2ebcd9b43ac9cbbdc
-  data.tar.gz: 90a968a7118048bb196507509a8e2e089e3ed668b8a23cb3353d0908a10f4596451708971d2f558347eb872eec947fc52e67169643026d0b773a3a45d5131c0d
+  metadata.gz: 9aa0f04dfa0f53060fd48765c040f26158e647254bc9ca049c8ebc2bca4f89ef3bd2c117c7ccc2bc9c05a94b1a4a4c47a04c2f93eda59aeb835d72963d6624d7
+  data.tar.gz: 264a48bb5458b09926950e32709184d909ce4e15123e90bc8ff2ae0f6091c16f4e1aab39b2d421ba1f087f86a7ae9f3f5cf59bfbcb4c352ab9b95b39830b8c16

data/.travis.yml

@@ -1,14 +1,14 @@
 rvm:
- - 2.3.1
- - 2.2.5
- - ruby-head
+  - 2.6.3
+  - ruby-head
 gemfile:
   - gemfiles/Gemfile.rails-2.x
   - gemfiles/Gemfile.rails-3.x
+  - gemfiles/Gemfile.rails-4.x
   - Gemfile
 notifications:
   recipients:
-    - jarmo.p@gmail.com
+    - jarmo@jarmopertman.com
 matrix:
   allow_failures:
     - rvm: ruby-head

data/CHANGES.md

@@ -1,37 +1,58 @@
-### 2.0.0 - 2016/09/24
-
-* Add support for Watir 6.0.
-
-### 1.2.1 - 2016/06/15
-
-* Fix Browser#add_checker deprecation warning. PR #18 by Christophe Bliard.
-
-### 1.2.0 - 2016/01/16
-
-* Add support for running Rails on Puma server. PR #15 by Andrey Koleshko.
-
-### 1.1.0 - 2015/07/22
-
-* Remove strict `mime-types` dependency. See #13
-
-### 1.0.4 - 2015/02/28
-
-* Allow to set Watir::Rails.ignore_exceptions to false. PR #8 by Andrey Koleshko.
-
-### 1.0.3 - 2013/11/02
-
-* Make watir-rails working with Rails 2.3.x too.
-
-### 1.0.2 - 2013/11/02
-
-* Make sure that newest Rails is going to be installed on an empty system.
-
-### 1.0.1 - 2013/11/01
-
-* Add license to gemspec.
-
-### 1.0.0 - 2013/10/05
-
-* Add watir as a dependency to make it possible to use with watir-classic too.
-* Add specs to keep the quality high.
-* Documentation fixes.
+### 2.2.3 - 2021/04/16
+
+* Fix a possible initialization problem with Watir 7.0.0+. See #32.
+ 
+### 2.2.2 - 2020/06/04
+
+* Fix a situation where checking if server is running without using specified port. PR #27 by Stefan Rotariu.
+
+### 2.2.1 - 2019/05/12
+
+* Fix EOFError on some environments. PR #25 by Slava Kardakov.
+
+### 2.2.0 - 2019/05/08
+
+* Add support for specifying custom server for test Rails instance. PR #23 by Alex Rodionov.
+* Add support for cleaning up when test Rails server instance has served all requests. PR #23 by Alex Rodionov.
+
+### 2.1.0 - 2019/03/19
+
+* Add support for specifying Rails test-server port. PR #22 by Bartek Wilczek.
+
+### 2.0.0 - 2016/09/24
+
+* Add support for Watir 6.0.
+
+### 1.2.1 - 2016/06/15
+
+* Fix Browser#add_checker deprecation warning. PR #18 by Christophe Bliard.
+
+### 1.2.0 - 2016/01/16
+
+* Add support for running Rails on Puma server. PR #15 by Andrey Koleshko.
+
+### 1.1.0 - 2015/07/22
+
+* Remove strict `mime-types` dependency. See #13
+
+### 1.0.4 - 2015/02/28
+
+* Allow to set Watir::Rails.ignore_exceptions to false. PR #8 by Andrey Koleshko.
+
+### 1.0.3 - 2013/11/02
+
+* Make watir-rails working with Rails 2.3.x too.
+
+### 1.0.2 - 2013/11/02
+
+* Make sure that newest Rails is going to be installed on an empty system.
+
+### 1.0.1 - 2013/11/01
+
+* Add license to gemspec.
+
+### 1.0.0 - 2013/10/05
+
+* Add watir as a dependency to make it possible to use with watir-classic too.
+* Add specs to keep the quality high.
+* Documentation fixes.

data/gemfiles/Gemfile.rails-4.x

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gemspec path: ".."
+
+gem "coveralls", require: false
+gem "rails", "~> 4.0"

data/lib/watir/rails.rb

@@ -18,17 +18,20 @@ module Watir
     class << self
       private :new
       attr_reader :port, :middleware
-      attr_writer :ignore_exceptions
+      attr_writer :ignore_exceptions, :server
 
       # Start the Rails server for tests.
       # Will be called automatically by {Watir::Browser#initialize}.
-      def boot
+      #
+      # @param [Integer] port port for the Rails up to run on. If omitted random port will be picked.
+      def boot(port: nil)
+        @port = port || find_available_port
+
         unless running?
           @middleware = Middleware.new(app)
-          @port = find_available_port
 
           @server_thread = Thread.new do
-            run_default_server @middleware, @port
+            server.call @middleware, @port
           end
 
           Timeout.timeout(boot_timeout) { @server_thread.join(0.1) until running? }
@@ -65,6 +68,13 @@ module Watir
         @middleware.error
       end
 
+      # Returns true if there are pending requests to server.
+      #
+      # @return [Boolean]
+      def pending_requests?
+        @middleware.pending_requests?
+      end
+
       # Set error rescued by the middleware.
       #
       # @param value
@@ -103,7 +113,7 @@ module Watir
         if res.is_a?(Net::HTTPSuccess) or res.is_a?(Net::HTTPRedirection)
           return res.body == @app.object_id.to_s
         end
-      rescue Errno::ECONNREFUSED, Errno::EBADF
+      rescue Errno::ECONNREFUSED, Errno::EBADF, EOFError
         return false
       end
 
@@ -137,22 +147,24 @@ module Watir
         server.close if server
       end
 
-      def run_default_server(app, port)
-        begin
-          require 'rack/handler/thin'
-          Thin::Logging.silent = true
-          return Rack::Handler::Thin.run(app, :Port => port)
-        rescue LoadError
-        end
+      def server
+        @server ||= lambda do |app, port|
+          begin
+            require 'rack/handler/thin'
+            Thin::Logging.silent = true
+            return Rack::Handler::Thin.run(app, :Port => port)
+          rescue LoadError
+          end
 
-        begin
-          require 'rack/handler/puma'
-          return Rack::Handler::Puma.run(app, :Port => port, :Silent => true)
-        rescue LoadError
-        end
+          begin
+            require 'rack/handler/puma'
+            return Rack::Handler::Puma.run(app, :Port => port, :Silent => true)
+          rescue LoadError
+          end
 
-        require 'rack/handler/webrick'
-        Rack::Handler::WEBrick.run(app, :Port => port, :AccessLog => [], :Logger => WEBrick::Log::new(nil, 0))
+          require 'rack/handler/webrick'
+          Rack::Handler::WEBrick.run(app, :Port => port, :AccessLog => [], :Logger => WEBrick::Log::new(nil, 0))
+        end
       end
 
       def legacy_rails?

data/lib/watir/rails/middleware.rb

@@ -1,26 +1,50 @@
-module Watir
-  class Rails
-    # @private
-    class Middleware
-      attr_accessor :error
-
-      def initialize(app)
-        @app = app
-      end
-
-      def call(env)
-        if env["PATH_INFO"] == "/__identify__"
-          [200, {}, [@app.object_id.to_s]]
-        else
-          begin
-            @app.call(env)
-          rescue => e
-            @error = e
-            raise e
-          end
-        end
-      end
-    end
-
-  end
-end
+module Watir
+  class Rails
+    # @private
+    class Middleware
+      class PendingRequestsCounter
+        attr_reader :value
+
+        def initialize
+          @value = 0
+          @mutex = Mutex.new
+        end
+
+        def increment
+          @mutex.synchronize { @value += 1 }
+        end
+
+        def decrement
+          @mutex.synchronize { @value -= 1 }
+        end
+      end
+
+      attr_accessor :error
+
+      def initialize(app)
+        @app = app
+        @counter = PendingRequestsCounter.new
+      end
+
+      def pending_requests?
+        @counter.value > 0
+      end
+
+      def call(env)
+        if env["PATH_INFO"] == "/__identify__"
+          [200, {}, [@app.object_id.to_s]]
+        else
+          @counter.increment
+          begin
+            @app.call(env)
+          rescue => e
+            @error = e
+            raise e
+          ensure
+            @counter.decrement
+          end
+        end
+      end
+    end
+  end
+end

data/lib/watir/{version.rb → rails/version.rb}

@@ -1,5 +1,5 @@
-module Watir
-  class Rails
-    VERSION = "2.0.0"
-  end
-end
+module Watir
+  class Rails
+    VERSION = "2.2.3"
+  end
+end

data/spec/watir/rails_spec.rb

@@ -9,9 +9,11 @@ describe Watir::Rails do
 
   context ".boot" do
     it "starts the server unless already running" do
+      server = ->(app, port) {}
       allow(described_class).to receive_messages(app: double("app"), find_available_port: 42)
       expect(described_class).to receive(:running?).twice.and_return(false, true)
-      expect(described_class).to receive(:run_default_server).once
+      expect(described_class).to receive(:server).and_return(server)
+      expect(server).to receive(:call).once
 
       described_class.boot
       wait_until_server_started
@@ -20,16 +22,18 @@ describe Watir::Rails do
     it "does nothing if server is already running" do
       allow(described_class).to receive_messages(app: double("app"), find_available_port: 42)
       expect(described_class).to receive(:running?).once.and_return(true)
-      expect(described_class).not_to receive(:run_default_server)
+      expect(described_class).not_to receive(:server)
 
       described_class.boot
     end
 
     it "raises an error if Rails won't boot with timeout" do
+      server = ->(app, port) {}
       allow(described_class).to receive_messages(app: double("app"),
         find_available_port: 42, boot_timeout: 0.01)
       expect(described_class).to receive(:running?).at_least(:twice).and_return(false)
-      expect(described_class).to receive(:run_default_server)
+      expect(described_class).to receive(:server).and_return(server)
+      expect(server).to receive(:call)
 
       expect {
         described_class.boot
@@ -41,6 +45,19 @@ describe Watir::Rails do
     end
   end
 
+  context ".server" do
+    it "allows to customize server" do
+      allow(described_class).to receive_messages(app: double("app"), find_available_port: 42)
+      allow(described_class).to receive(:running?).twice.and_return(false, true)
+
+      server = ->(app, port) {}
+      described_class.server = server
+      expect(server).to receive(:call)
+
+      described_class.boot
+    end
+  end
+
   context ".host" do
     it "@host if specified" do
       described_class.host = "my_host"

data/watir-rails.gemspec

@@ -1,5 +1,5 @@
 # -*- encoding: utf-8 -*-
-require File.expand_path('../lib/watir/version', __FILE__)
+require File.expand_path('../lib/watir/rails/version', __FILE__)
 
 Gem::Specification.new do |gem|
   gem.authors       = ["Jarmo Pertman"]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: watir-rails
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.2.3
 platform: ruby
 authors:
 - Jarmo Pertman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-24 00:00:00.000000000 Z
+date: 2021-04-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -111,10 +111,11 @@ files:
 - Rakefile
 - gemfiles/Gemfile.rails-2.x
 - gemfiles/Gemfile.rails-3.x
+- gemfiles/Gemfile.rails-4.x
 - lib/watir/rails.rb
 - lib/watir/rails/browser.rb
 - lib/watir/rails/middleware.rb
-- lib/watir/version.rb
+- lib/watir/rails/version.rb
 - spec/spec_helper.rb
 - spec/support/watir.rb
 - spec/watir/rails/browser_spec.rb
@@ -141,7 +142,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.4
+rubygems_version: 2.7.6.2
 signing_key: 
 specification_version: 4
 summary: Use Watir (http://github.com/watir/watir) in Rails.
@@ -151,4 +152,3 @@ test_files:
 - spec/watir/rails/browser_spec.rb
 - spec/watir/rails/middleware_spec.rb
 - spec/watir/rails_spec.rb
-has_rdoc: