Fix #323 regression. Configure application inline without proc

Author: bogdan

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## v2.3.1
 
 * Add timestamp on when routes where generated into banner
+* Fix application specified directly without proc. [#323](https://github.com/railsware/js-routes/issues/323)
 
 ## v2.3.0
 

lib/js_routes/instance.rb

@@ -117,7 +117,8 @@ def wrapper_variable
 
     sig { returns(Application) }
     def application
-      @configuration.application.call
+      app = @configuration.application
+      app.is_a?(Proc) ? app.call : app
     end
 
     sig { params(value: T.untyped).returns(String) }

lib/js_routes/types.rb

@@ -17,7 +17,7 @@ module Types
     JourneyRoute = T.type_alias{ActionDispatch::Journey::Route}
     RouteSpec = T.type_alias {T.untyped}
     Application = T.type_alias { T.any(T::Class[Rails::Engine], Rails::Application) }
-    ApplicationCaller = T.type_alias { T.proc.returns(Application) }
+    ApplicationCaller = T.type_alias { T.any(Application, T.proc.returns(Application)) }
     Clusivity = T.type_alias { T.any(Regexp, T::Array[Regexp]) }
     FileName = T.type_alias { T.any(String, Pathname, NilClass) }
     ConfigurationBlock = T.type_alias { T.proc.params(arg0: JsRoutes::Configuration).void }

spec/js_routes/options_spec.rb

@@ -482,10 +482,21 @@
   end
 
   describe "when application is specified" do
-    let(:_options) { {application: -> {BlogEngine::Engine}} }
+    context "as proc" do
+      let(:_options) { {application: -> {BlogEngine::Engine}} }
+
+      it "should include specified engine route" do
+        expectjs("Routes.posts_path()").not_to be_nil
+      end
+    end
+
+    context "directly" do
+      let(:_options) { {application: BlogEngine::Engine} }
+
+      it "should include specified engine route" do
+        expectjs("Routes.posts_path()").not_to be_nil
+      end
 
-    it "should include specified engine route" do
-      expectjs("Routes.posts_path()").not_to be_nil
     end
   end