route_mechanic 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a2378e095ad88def369adb3b2b271a9e324a853b535670e3b1be95f4e98d2ee0
-  data.tar.gz: 12f172bb4021e0c56e4ea740a269d10e41a2c67d2f3f040137882dc414ffe448
+  metadata.gz: edfa9dd2a475e94ffb5131cfb367f430a838ecc12d001e45414955b1f0d84ec5
+  data.tar.gz: '039c3bbbc1f86d16336db76fb1e6b0879ee3e2d2b8bc8f64731988c64301c48c'
 SHA512:
-  metadata.gz: 3ee458593e8f2e095c11cb2ce1f36095c50d53eacab10e72032429b4514641d1afe676a37082121c629ba7c23e1ede37362a3ee0ce563caf35f76bf0760bb2ae
-  data.tar.gz: 3079f7b95a27e7c2c3bddd735352283f85cd6f83e362398406c70206160783dba53a83cfe35daf42ab2965857b445dac148d53cbf1cff29fbbb338443c9f7bde
+  metadata.gz: ae134daadd7209a3d67e5e3c9b7e2615b168693f034d3abd34915568de2333b5ef0955f0b7f57a76734c0e06d56145cc8bf861f156567f9437ed5bd02199e1fd
+  data.tar.gz: 97d0c73f8d437837742470795f4f9414c73af178f911d27e49eb5c5edbba5ebf51a9f7e0b0d1155e66f1f348a1aaf8ba1333513d48de6c41fe36afe8c0ac0e7f

data/README.md

@@ -33,7 +33,7 @@ Just add a test file which has only one test case using `have_valid_routes` matc
 ```ruby
 RSpec.describe 'Rails.application', type: :routing do
   it "fails if application does not have valid routes" do
-   expect(Rails.application.routes).to have_valid_routes
+   expect(Rails.application).to have_valid_routes
   end
 end
 ```

data/lib/route_mechanic/rspec/matchers.rb

@@ -8,7 +8,7 @@ module RouteMechanic
         include ::RSpec::Matchers::Composable
         include RouteMechanic::Testing::Methods
 
-        # @param [ActionDispatch::Routing::RouteSet] expected
+        # @param [Rails::Application] expected
         def initialize(expected)
           @expected = expected
         end
@@ -42,8 +42,8 @@ module RouteMechanic
         end
       end
 
-      def have_valid_routes(routes=Rails.application.routes)
-        HaveValidRoutes.new(routes)
+      def have_valid_routes(application=Rails.application)
+        HaveValidRoutes.new(application)
       end
     end
   end

data/lib/route_mechanic/testing/methods.rb

@@ -10,12 +10,10 @@ module RouteMechanic
       include MinitestAssertionAdapter if defined?(RSpec)
       include ActionDispatch::Assertions
 
-      # @param [ActionDispatch::Routing::RouteSet] routes
+      # @param [Rails::Application] application
       # @raise [Minitest::Assertion]
-      def assert_all_routes(routes=Rails.application.routes)
-        # assert_routing expect @routes to exists as like this class inherits ActionController::TestCase.
-        # If user already defines @routes, do not override
-        @routes ||= routes
+      def assert_all_routes(application=Rails.application)
+        @application = application
 
         # Instead of including ActionController::TestCase::Behavior, set up
         # https://github.com/rails/rails/blob/5b6aa8c20a3abfd6274c83f196abf73cacb3400b/actionpack/lib/action_controller/test_case.rb#L519-L520
@@ -29,6 +27,20 @@ module RouteMechanic
 
       private
 
+      def routes
+        # assert_routing expect @routes to exists as like this class inherits ActionController::TestCase.
+        # If user already defines @routes, do not override
+        @routes ||= @application.routes
+
+        return @routes if @routes.routes.size > 0
+
+        # If routes setting is not loaded when running test, it automatically loads config/routes as Rails does.
+        load_path = "#{Rails.root.join('config/routes.rb')}"
+        @application.routes_reloader.paths << load_path unless @application.routes_reloader.paths.include? load_path
+        @application.reload_routes!
+        @routes
+      end
+
       # @param [RouteMechanic::Testing::RouteWrapper] wrapper
       # @raise [Minitest::Assertion]
       def assert_routes(wrapper)
@@ -37,7 +49,7 @@ module RouteMechanic
         end
 
         base_option = { controller: wrapper.controller, action: wrapper.action }
-        url = @routes.url_helpers.url_for(
+        url = routes.url_helpers.url_for(
           base_option.merge({ only_path: true }).merge(required_parts))
         expected_options = base_option.merge(required_parts)
 
@@ -62,7 +74,7 @@ module RouteMechanic
 
       # @return [Array<ActionDispatch::Journey::Route>]
       def target_routes
-        @routes.routes.reject do |journey_route|
+        routes.routes.reject do |journey_route|
           # Skip internals, endpoints that Rails adds by default
           # Also Engines should be skipped since Engine's tests should be done in Engine
           wrapper = RouteWrapper.new(journey_route)

data/lib/route_mechanic/version.rb

@@ -1,3 +1,3 @@
 module RouteMechanic
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: route_mechanic
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - ohbarye