cypress-rails 0.3.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6cbbbc003d33e8f71d926860e428093061ffddf5c10adf67e4da34f4a4a5a763
-  data.tar.gz: f9e5ce0881ccd57b8afa2d6b9839269f5999ac905cb3c2cd06d44db2a36a53ef
+  metadata.gz: a89d2ae87c67c2df6c45dd8e3642a9ca52b0f3a88b29b4d7e7569dc22579ff76
+  data.tar.gz: 94d308d83c47086800430e3c8954e068637575f7c4d9e3f4327b1f325600c8dd
 SHA512:
-  metadata.gz: d1568bc421e21b5a61d3f35a133fe5e0bafcbf7830164457ea37f4c3d9a1f0c8e194b0446d93912a816a29bf0dce2404c48f1d848f33bda152cc75d8ea7c28a9
-  data.tar.gz: 2408279b528acb1139dca550f19fae8cb5d4e5fb116d9eb26512c012d4549c44afe98d45ef29972aff96d3724c2afc63304e5c303cbe7f3f3d88668fbf590974
+  metadata.gz: b3630cfd6de6541c551163a34c6e513865743b3733a603a492a4328efd5a827d68925757eba5c0de3d7b095b7597f357f653a52e5040542e8e6c979c869805db
+  data.tar.gz: 0f80be8f0c63d8de21917913b02d83a59d6a436da223c9838b1a0421ca3317bbd5d71289fa60f5021e69d94167453c6b015d4469b61c209e05464449e1bb4034

data/.github/workflows/main.yml

@@ -0,0 +1,21 @@
+name: CI
+
+on: [push, pull_request, workflow_dispatch]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        ruby-version: ['2.7', '3.0']
+
+    steps:
+      - uses: actions/checkout@v2
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+          bundler-cache: true
+
+      - name: Run tests
+        run: ./script/test

data/.standard.yml

@@ -1,4 +1,4 @@
 ignore:
   - 'vendor/bundle/**/*'
-  - 'example/an_app/node_modules/**/*'
-  - 'example/an_app/vendor/bundle/**/*'
+  - 'example/node_modules/**/*'
+  - 'example/vendor/bundle/**/*'

data/CHANGELOG.md

@@ -1,5 +1,35 @@
 # CHANGELOG
 
+## 0.5.1
+
+* Sprinkles two instance variables to the custom transaction manager that cribs
+  its implementation from ActiveRecord::TestFixtures (see f75f280)
+  [#88](https://github.com/testdouble/cypress-rails/issues/88)
+  [#89](https://github.com/testdouble/cypress-rails/pull/89)
+
+## 0.5.0
+
+* Add hook `after_server_start`
+  [#63](https://github.com/testdouble/cypress-rails/pull/63)
+* Fix namespace bug
+  [#64](https://github.com/testdouble/cypress-rails/pull/64)
+
+## 0.4.2
+
+* Add support to Rails 6.1 ([#52](https://github.com/testdouble/cypress-rails/issue/52))
+
+## 0.4.1
+
+* Add backcompat for Ruby 2.4
+  ([#47](https://github.com/testdouble/cypress-rails/pull/47))
+
+## 0.4.0
+
+* Add a `CYPRESS_RAILS_HOST` option that allows a hostname to be specified (as
+opposed to 127.0.0.1). Puma will still bind to 127.0.0.1, but Cypress will use
+the hostname in its `baseUrl`, which may be necessary for some folks' tests to
+work
+
 ## 0.3.0
 
 * Add a `CYPRESS_RAILS_BASE_PATH` option which will be appended to the

data/Gemfile.lock

@@ -1,55 +1,56 @@
 PATH
   remote: .
   specs:
-    cypress-rails (0.3.0)
+    cypress-rails (0.5.1)
       puma (>= 3.8.0)
       railties (>= 5.2.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    actionpack (6.0.3)
-      actionview (= 6.0.3)
-      activesupport (= 6.0.3)
-      rack (~> 2.0, >= 2.0.8)
+    actionpack (6.1.4)
+      actionview (= 6.1.4)
+      activesupport (= 6.1.4)
+      rack (~> 2.0, >= 2.0.9)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actionview (6.0.3)
-      activesupport (= 6.0.3)
+    actionview (6.1.4)
+      activesupport (= 6.1.4)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    activesupport (6.0.3)
+    activesupport (6.1.4)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.2, >= 2.2.2)
-    ast (2.4.0)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
+    ast (2.4.2)
     builder (3.2.4)
-    concurrent-ruby (1.1.6)
+    concurrent-ruby (1.1.9)
     crass (1.0.6)
-    erubi (1.9.0)
-    i18n (1.8.2)
+    erubi (1.10.0)
+    i18n (1.8.10)
       concurrent-ruby (~> 1.0)
-    jaro_winkler (1.5.4)
-    loofah (2.5.0)
+    loofah (2.10.0)
       crass (~> 1.0.2)
       nokogiri (>= 1.5.9)
     method_source (1.0.0)
-    mini_portile2 (2.4.0)
-    minitest (5.14.0)
-    nio4r (2.5.2)
-    nokogiri (1.10.9)
-      mini_portile2 (~> 2.4.0)
-    parallel (1.19.1)
-    parser (2.7.1.2)
-      ast (~> 2.4.0)
-    puma (4.3.3)
+    mini_portile2 (2.5.3)
+    minitest (5.14.4)
+    nio4r (2.5.7)
+    nokogiri (1.11.7)
+      mini_portile2 (~> 2.5.0)
+      racc (~> 1.4)
+    parallel (1.20.1)
+    parser (3.0.1.1)
+      ast (~> 2.4.1)
+    puma (5.3.2)
       nio4r (~> 2.0)
-    rack (2.2.2)
+    racc (1.5.2)
+    rack (2.2.3)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
     rails-dom-testing (2.0.3)
@@ -57,35 +58,39 @@ GEM
       nokogiri (>= 1.6)
     rails-html-sanitizer (1.3.0)
       loofah (~> 2.3)
-    railties (6.0.3)
-      actionpack (= 6.0.3)
-      activesupport (= 6.0.3)
+    railties (6.1.4)
+      actionpack (= 6.1.4)
+      activesupport (= 6.1.4)
       method_source
-      rake (>= 0.8.7)
-      thor (>= 0.20.3, < 2.0)
+      rake (>= 0.13)
+      thor (~> 1.0)
     rainbow (3.0.0)
-    rake (13.0.1)
-    rexml (3.2.4)
-    rubocop (0.82.0)
-      jaro_winkler (~> 1.5.1)
+    rake (13.0.4)
+    regexp_parser (2.1.1)
+    rexml (3.2.5)
+    rubocop (1.18.2)
       parallel (~> 1.10)
-      parser (>= 2.7.0.1)
+      parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
       rexml
+      rubocop-ast (>= 1.7.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 2.0)
-    rubocop-performance (1.5.2)
-      rubocop (>= 0.71.0)
-    ruby-progressbar (1.10.1)
-    standard (0.4.1)
-      rubocop (~> 0.82.0)
-      rubocop-performance (~> 1.5.2)
-    thor (1.0.1)
-    thread_safe (0.3.6)
-    tzinfo (1.2.7)
-      thread_safe (~> 0.1)
-    unicode-display_width (1.7.0)
-    zeitwerk (2.3.0)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.7.0)
+      parser (>= 3.0.1.1)
+    rubocop-performance (1.11.3)
+      rubocop (>= 1.7.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    ruby-progressbar (1.11.0)
+    standard (1.1.4)
+      rubocop (= 1.18.2)
+      rubocop-performance (= 1.11.3)
+    thor (1.1.0)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.0.0)
+    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby
@@ -98,4 +103,4 @@ DEPENDENCIES
   standard (>= 0.2.0)
 
 BUNDLED WITH
-   2.1.4
+   2.2.15

data/README.md

@@ -1,6 +1,6 @@
 # cypress-rails
 
-[![CircleCI](https://circleci.com/gh/testdouble/cypress-rails/tree/master.svg?style=svg)](https://circleci.com/gh/testdouble/cypress-rails/tree/master)
+[![CircleCI](https://circleci.com/gh/testdouble/cypress-rails/tree/main.svg?style=svg)](https://circleci.com/gh/testdouble/cypress-rails/tree/main)
 
 This is a simple gem to make it easier to start writing browser tests with
 [Cypress](http://cypress.io) for your [Rails](https://rubyonrails.org) apps,
@@ -102,7 +102,7 @@ like:
 return unless Rails.env.test?
 
 CypressRails.hooks.before_server_start do
-  # Called once, before ether the transaction or the server is started
+  # Called once, before either the transaction or the server is started
 end
 
 CypressRails.hooks.after_transaction_start do
@@ -120,7 +120,7 @@ end
 
 (You can find [an
 example
-initializer](/example/an_app/config/initializers/cypress_rails_initializer.rb)
+initializer](/example/config/initializers/cypress_rails_initializer.rb)
 in this repo.)
 
 The gem also provides a special route on the test server:
@@ -138,9 +138,9 @@ This way, you can easily instruct the server to reset its test state from your
 Cypress tests like so:
 
 ```js
-beforeEach(() =>
+beforeEach(() => {
   cy.request('/cypress_rails_reset_state')
-)
+})
 ```
 
 (Remember, in Cypress, `before` is a before-all hook and `beforeEach` is run
@@ -158,6 +158,7 @@ preferred environment variables project-wide using a tool like
 
 
 * **CYPRESS_RAILS_DIR** (default: `Dir.pwd`) the directory of your project
+* **CYPRESS_RAILS_HOST** (default: `"127.0.0.1"`) the hostname to bind to
 * **CYPRESS_RAILS_PORT** (default: _a random available port_) the port to run
   the Rails test server on
 * **CYPRESS_RAILS_BASE_PATH** (default: `"/"`) the base path for all Cypress's
@@ -169,7 +170,24 @@ preferred environment variables project-wide using a tool like
   general this means anything done during `cypress open` or `cypress run` will
   be rolled back on exit (similar to running a Rails System test)
 * **CYPRESS_RAILS_CYPRESS_OPTS** (default: _none_) any options you want to
-  forward to the Cypress CLI when running its `open` or `run` commands
+  forward to the Cypress CLI when running its `open` or `run` commands.
+
+#### Example: Running a single spec from the command line
+
+It's a little verbose, but an example of using the above options to run a single
+Cypress test would look like this:
+
+```
+$ CYPRESS_RAILS_CYPRESS_OPTS="--spec cypress/integration/a_test.js" bin/rake cypress:run
+```
+
+#### Example: Running your tests in Chromium
+
+By default, Cypress will run its tests in its packaged Electron app, unless you've configured it globally. To choose which browser it will run from the command line, try this:
+
+```
+$ CYPRESS_RAILS_CYPRESS_OPTS="--browser chromium" bin/rake cypress:run
+```
 
 ### Initializer hooks
 
@@ -180,6 +198,11 @@ will execute before the server or any transaction has been started. If you use
 Rails fixtures, it may make sense to load them here, so they don't need to be
 re-inserted for each request
 
+### after_server_start
+
+Pass a block to `CypressRails.hooks.after_server_start` to register a hook that
+will execute after the server has booted.
+
 ### after_transaction_start
 
 If there's any custom behavior or state management you want to do inside the
@@ -202,6 +225,22 @@ pollute other test suites or scripts, you can use the `before_server_stop` to
 of your test database. To set up the hook, pass a block to
 `CypressRails.hooks.before_server_stop`.
 
+## Configuring Rails
+
+Beyond the configuration options above, you'll probably also want to disable caching
+in your Rails app's [config/environments/test.rb](/example/config/environments/test.rb#L9)
+file, so that changes to your Ruby code are reflected in your tests while you
+work on them with `rake cypress:open`. (If either option is set to
+`true`, any changes to your Ruby code will require a server restart to be reflected as you work
+on your tests.)
+
+To illustrate, here's what that might look like in `config/environments/test.rb`:
+
+```ruby
+config.cache_classes = false
+config.action_view.cache_template_loading = false
+```
+
 ## Setting up continuous integration
 
 #### Circle CI
@@ -278,7 +317,7 @@ experience. In particular:
   and more information dense than using Capybara and Selenium. Rather than
   running a test from the command line, seeing it fail, then adding a debug
   breakpoint to a test to try to manipulate the browser or tweaking a call to a
-  Capybara API method, failures to be rather obvious when using Cypress and
+  Capybara API method, failures tend to be rather obvious when using Cypress and
   fixing it is usually as easy as tweaking a command, hitting save, and watching
   it re-run
 * With very few exceptions, a Cypress test that works in a browser window will

data/lib/cypress-rails/config.rb

@@ -1,15 +1,23 @@
 require_relative "env"
 
 module CypressRails
-  class Config < Struct.new(:dir, :port, :base_path, :transactional_server, :cypress_cli_opts, keyword_init: true)
+  class Config
+    attr_accessor :dir, :host, :port, :base_path, :transactional_server, :cypress_cli_opts
+
     def initialize(
       dir: Env.fetch("CYPRESS_RAILS_DIR", default: Dir.pwd),
+      host: Env.fetch("CYPRESS_RAILS_HOST", default: "127.0.0.1"),
       port: Env.fetch("CYPRESS_RAILS_PORT"),
       base_path: Env.fetch("CYPRESS_RAILS_BASE_PATH", default: "/"),
       transactional_server: Env.fetch("CYPRESS_RAILS_TRANSACTIONAL_SERVER", type: :boolean, default: true),
       cypress_cli_opts: Env.fetch("CYPRESS_RAILS_CYPRESS_OPTS", default: "")
     )
-      super
+      @dir = dir
+      @host = host
+      @port = port
+      @base_path = base_path
+      @transactional_server = transactional_server
+      @cypress_cli_opts = cypress_cli_opts
     end
 
     def to_s
@@ -18,6 +26,7 @@ module CypressRails
         cypress-rails configuration:
         ============================
          CYPRESS_RAILS_DIR.....................#{dir.inspect}
+         CYPRESS_RAILS_HOST....................#{host.inspect}
          CYPRESS_RAILS_PORT....................#{port.inspect}
          CYPRESS_RAILS_BASE_PATH...............#{base_path.inspect}
          CYPRESS_RAILS_TRANSACTIONAL_SERVER....#{transactional_server.inspect}

data/lib/cypress-rails/env.rb

@@ -4,11 +4,8 @@ module CypressRails
       return default unless ENV.key?(name)
 
       if type == :boolean
-        if ["", "0", "n", "no", "false"].include?(ENV.fetch(name))
-          false
-        else
-          true
-        end
+        no_like_flag = ["", "0", "n", "no", "false"].include?(ENV.fetch(name))
+        !no_like_flag
       else
         ENV.fetch(name)
       end

data/lib/cypress-rails/init.rb

@@ -24,7 +24,7 @@ module CypressRails
     end
 
     def merge_existing_with_defaults(json_path)
-      Hash[JSON.parse(File.read(json_path)).merge(DEFAULT_CONFIG).sort]
+      JSON.parse(File.read(json_path)).merge(DEFAULT_CONFIG).sort.to_h
     end
   end
 end

data/lib/cypress-rails/initializer_hooks.rb

@@ -12,6 +12,10 @@ module CypressRails
       register(:before_server_start, blk)
     end
 
+    def after_server_start(&blk)
+      register(:after_server_start, blk)
+    end
+
     def after_transaction_start(&blk)
       register(:after_transaction_start, blk)
     end

data/lib/cypress-rails/launches_cypress.rb

@@ -20,6 +20,7 @@ module CypressRails
         @manages_transactions.begin_transaction
       end
       server = @starts_rails_server.call(
+        host: config.host,
         port: config.port,
         transactional_server: config.transactional_server
       )

data/lib/cypress-rails/manages_transactions.rb

@@ -20,7 +20,7 @@ module CypressRails
 
           begin
             connection = ActiveRecord::Base.connection_handler.retrieve_connection(spec_name)
-          rescue ConnectionNotEstablished
+          rescue ActiveRecord::ConnectionNotEstablished
             connection = nil
           end
 
@@ -68,19 +68,10 @@ module CypressRails
     # need to share a connection pool so that the reading connection
     # can see data in the open transaction on the writing connection.
     def setup_shared_connection_pool
-      return unless ActiveRecord::Base.respond_to?(:connection_handlers)
-
-      writing_handler = ActiveRecord::Base.connection_handler
-      ActiveRecord::Base.connection_handlers.values.each do |handler|
-        if handler != writing_handler
-          handler.connection_pool_list.each do |pool|
-            name = pool.spec.name
-            writing_connection = writing_handler.retrieve_connection_pool(name)
-            return nil unless writing_connection
-            handler.send(:owner_to_pool)[name] = writing_connection
-          end
-        end
-      end
+      @legacy_saved_pool_configs ||= Hash.new { |hash, key| hash[key] = {} }
+      @saved_pool_configs ||= Hash.new { |hash, key| hash[key] = {} }
+
+      ActiveRecord::TestFixtures.instance_method(:setup_shared_connection_pool).bind_call(self)
     end
   end
 end

data/lib/cypress-rails/server.rb

@@ -3,6 +3,7 @@
 require "uri"
 require "net/http"
 require "rack"
+require_relative "initializer_hooks"
 require_relative "server/middleware"
 require_relative "server/checker"
 require_relative "server/timer"
@@ -16,11 +17,11 @@ module CypressRails
       end
     end
 
-    attr_reader :app, :port, :host
+    attr_reader :app, :host, :port
 
     def initialize(app,
+      host:,
       port:,
-      host: "127.0.0.1",
       reportable_errors: [Exception],
       extra_middleware: [])
       @app = app
@@ -32,6 +33,7 @@ module CypressRails
       @port ||= Server.ports[port_key]
       @port ||= find_available_port(host)
       @checker = Checker.new(@host, @port)
+      @initializer_hooks = InitializerHooks.instance
     end
 
     def reset_error!
@@ -78,6 +80,7 @@ module CypressRails
           raise "Rack application timed out during boot" if timer.expired?
 
           @server_thread.join(0.1)
+          @initializer_hooks.run(:after_server_start)
         end
       end
 

data/lib/cypress-rails/starts_rails_server.rb

@@ -3,10 +3,10 @@ require_relative "server"
 
 module CypressRails
   class StartsRailsServer
-    def call(port:, transactional_server:)
+    def call(host:, port:, transactional_server:)
       configure_rails_to_run_our_state_reset_on_every_request!(transactional_server)
       app = create_rack_app
-      Server.new(app, port: port).tap do |server|
+      Server.new(app, host: host, port: port).tap do |server|
         server.boot
       end
     end

data/lib/cypress-rails/version.rb

@@ -1,3 +1,3 @@
 module CypressRails
-  VERSION = "0.3.0"
+  VERSION = "0.5.1"
 end

data/script/test

@@ -0,0 +1,19 @@
+#!/usr/bin/env bash
+
+PS4='[script/test:${LINENO}] $ '
+set -euo pipefail
+set -x
+
+echo "---> Installing dependencies"
+bundle
+cd example
+bundle
+yarn
+cd ..
+
+echo "---> Running tests"
+bundle exec rake
+./script/test_example_app
+
+echo "---> Job's done!"
+

data/script/test_example_app

@@ -4,7 +4,7 @@ set -e
 
 directory=$1
 
-cd example/an_app
+cd example
 bundle
 yarn install
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cypress-rails
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Justin Searls
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-05-15 00:00:00.000000000 Z
+date: 2021-07-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -94,7 +94,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.2.0
-description: 
+description:
 email:
 - searls@gmail.com
 executables:
@@ -102,7 +102,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- ".circleci/config.yml"
+- ".github/workflows/main.yml"
 - ".gitignore"
 - ".standard.yml"
 - ".travis.yml"
@@ -137,12 +137,13 @@ files:
 - lib/cypress-rails/starts_rails_server.rb
 - lib/cypress-rails/tracks_resets.rb
 - lib/cypress-rails/version.rb
+- script/test
 - script/test_example_app
 homepage: https://github.com/testdouble/cypress-rails
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -157,8 +158,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.2.15
+signing_key:
 specification_version: 4
 summary: Helps you write Cypress tests of your Rails app
 test_files: []

data/.circleci/config.yml

@@ -1,53 +0,0 @@
-version: 2
-jobs:
-  build:
-    docker:
-      - image: circleci/ruby:2.6-node-browsers
-    steps:
-      - checkout
-
-      # Bundle install dependencies
-      - type: cache-restore
-        key: v1-main-{{ checksum "Gemfile.lock" }}
-
-      - run: gem install bundler --version `tail -1 Gemfile.lock`
-      - run: bundle install --path vendor/bundle
-
-      - type: cache-save
-        key: v1-main-{{ checksum "Gemfile.lock" }}
-        paths:
-          - vendor/bundle
-
-      # Bundle install dependencies for example app
-      - type: cache-restore
-        key: v1-example-{{ checksum "example/an_app/Gemfile.lock" }}
-
-      - run: |
-          cd example/an_app
-          bundle install --path vendor/bundle
-
-      - type: cache-save
-        key: v1-example-{{ checksum "example/an_app/Gemfile.lock" }}
-        paths:
-          - example/an_app/vendor/bundle
-
-      # Yarn dependencies
-      - restore_cache:
-          keys:
-            - v2-yarn-{{ checksum "example/an_app/package.json" }}
-            # fallback to using the latest cache if no exact match is found
-            - v2-yarn-
-
-      - run: |
-          cd example/an_app
-          yarn install
-
-      - save_cache:
-          paths:
-            - example/an_app/node_modules
-            - ~/.cache
-          key: v2-yarn-{{ checksum "example/an_app/package.json" }}
-
-      - run: bundle exec rake
-      - run: ./script/test_example_app
-