hanami 2.0.3 → 2.1.0.beta2

data/spec/integration/view/config/scope_namespace_spec.rb

@@ -0,0 +1,103 @@
+# frozen_string_literal: true
+
+require "hanami"
+
+RSpec.describe "App view / Config / Scope namespace", :app_integration do
+  before do
+    with_directory(make_tmp_directory) do
+      write "config/app.rb", <<~RUBY
+        module TestApp
+          class App < Hanami::App
+          end
+        end
+      RUBY
+
+      write "app/view.rb", <<~RUBY
+        # auto_register: false
+
+        require "hanami/view"
+
+        module TestApp
+          class View < Hanami::View
+          end
+        end
+      RUBY
+
+      before_prepare if respond_to?(:before_prepare)
+      require "hanami/prepare"
+    end
+  end
+
+  subject(:scope_namespace) { view_class.config.scope_namespace }
+
+  describe "app view" do
+    let(:view_class) { TestApp::View }
+
+    describe "no scope namespace defined" do
+      it "is nil" do
+        expect(scope_namespace).to be nil
+      end
+    end
+
+    describe "scope namespace defined" do
+      def before_prepare
+        write "app/views/scopes/player.rb", <<~RUBY
+          module TestApp
+            module Views
+              module Scopes
+                class Player < Hanami::View::Scope
+                end
+              end
+            end
+          end
+        RUBY
+      end
+
+      it "is the Views::Scopes namespace within the app" do
+        expect(scope_namespace).to eq TestApp::Views::Scopes
+      end
+    end
+  end
+
+  describe "slice view" do
+    def before_prepare
+      write "slices/main/view.rb", <<~RUBY
+        # auto_register: false
+
+        module Main
+          class View < TestApp::View
+          end
+        end
+      RUBY
+    end
+
+    let(:view_class) { Main::View }
+
+    describe "no scope namespace defined" do
+      it "is nil" do
+        expect(scope_namespace).to be nil
+      end
+    end
+
+    describe "scope namespace defined" do
+      def before_prepare
+        super
+
+        write "slices/main/views/scopes/player.rb", <<~RUBY
+          module Main
+            module Views
+              module Scopes
+                class Player < Hanami::View::Scope
+                end
+              end
+            end
+          end
+        RUBY
+      end
+
+      it "is the Views::Scopes namespace within the slice" do
+        expect(scope_namespace).to eq Main::Views::Scopes
+      end
+    end
+  end
+end

data/spec/integration/view/config/template_spec.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require "hanami"
+
+RSpec.describe "App view / Config / Template", :app_integration do
+  before do
+    module TestApp
+      class App < Hanami::App
+        config.root = "/test_app"
+      end
+    end
+
+    Hanami.app.instance_eval(&app_hook) if respond_to?(:app_hook)
+    Hanami.app.register_slice :main
+    Hanami.app.prepare
+
+    module TestApp
+      class View < Hanami::View
+      end
+    end
+
+    module TestApp
+      module Views
+        module Article
+          class Index < TestApp::View
+          end
+        end
+      end
+    end
+  end
+
+  subject(:template) { view_class.config.template }
+  let(:view_class) { TestApp::Views::Article::Index }
+
+  it "configures the tempalte to match the class name" do
+    expect(template).to eq "article/index"
+  end
+end

data/spec/integration/view/context/assets_spec.rb

@@ -24,21 +24,15 @@ RSpec.describe "App view / Context / Assets", :app_integration do
 
   describe "#assets" do
     context "without assets provider" do
-      it "raises error" do
+      xit "raises error" do
+        allow(Hanami).to receive(:bundled?).with("hanami-assets").and_return(false)
+
         expect { context.assets }
           .to raise_error(Hanami::ComponentLoadError, /hanami-assets/)
       end
     end
 
     context "with assets provider" do
-      before do
-        Hanami.app.register_provider(:assets) do
-          start do
-            register "assets", Object.new
-          end
-        end
-      end
-
       it "is the app assets by default" do
         expect(context.assets).to be TestApp::App[:assets]
       end

data/spec/integration/view/context/request_spec.rb

@@ -20,14 +20,10 @@ RSpec.describe "App view / Context / Request", :app_integration do
   let(:context_class) { TestApp::Views::Context }
 
   subject(:context) {
-    context_class.new(
-      request: request,
-      response: response,
-    )
+    context_class.new(request: request)
   }
 
   let(:request) { double(:request) }
-  let(:response) { double(:response) }
 
   describe "#request" do
     it "is the provided request" do
@@ -51,10 +47,10 @@ RSpec.describe "App view / Context / Request", :app_integration do
     let(:flash) { double(:flash) }
 
     before do
-      allow(response).to receive(:flash) { flash }
+      allow(request).to receive(:flash) { flash }
     end
 
-    it "is the response's flash" do
+    it "is the request's flash" do
       expect(context.flash).to be flash
     end
   end

data/spec/integration/view/helpers/form_helper_spec.rb

@@ -0,0 +1,174 @@
+# frozen_string_literal: true
+
+require "rack/test"
+require "stringio"
+
+RSpec.describe "Helpers / FormHelper", :app_integration do
+  include Rack::Test::Methods
+  let(:app) { Hanami.app }
+
+  before do
+    with_directory(make_tmp_directory) do
+      write "config/app.rb", <<~RUBY
+        module TestApp
+          class App < Hanami::App
+            config.logger.stream = StringIO.new
+          end
+        end
+      RUBY
+
+      write "config/routes.rb", <<~RUBY
+        module TestApp
+          class Routes < Hanami::Routes
+            get "posts/:id/edit", to: "posts.edit"
+            put "posts/:id", to: "posts.update"
+          end
+        end
+      RUBY
+
+      write "app/action.rb", <<~RUBY
+        # auto_register: false
+
+        require "hanami/action"
+
+        module TestApp
+          class Action < Hanami::Action
+          end
+        end
+      RUBY
+
+      write "app/view.rb", <<~RUBY
+        # auto_register: false
+
+        require "hanami/view"
+
+        module TestApp
+          class View < Hanami::View
+            config.layout = nil
+          end
+        end
+      RUBY
+
+      write "app/actions/posts/edit.rb", <<~RUBY
+        module TestApp
+          module Actions
+            module Posts
+              class Edit < TestApp::Action
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "app/actions/posts/update.rb", <<~RUBY
+        module TestApp
+          module Actions
+            module Posts
+              class Update < TestApp::Action
+                def handle(request, response)
+                  if valid?(request.params[:post])
+                    response.redirect_to "/posts/x/edit"
+                  else
+                    response.render view
+                  end
+                end
+
+                private
+
+                def valid?(post)
+                  post.to_h[:title].to_s.length > 0
+                end
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "app/views/posts/edit.rb", <<~RUBY
+        module TestApp
+          module Views
+            module Posts
+              class Edit < TestApp::View
+                expose :post do
+                  Struct.new(:title, :body).new("Hello <world>", "This is the post.")
+                end
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "app/templates/posts/edit.html.erb", <<~ERB
+        <h1>Edit post</h1>
+
+        <%= form_for("/posts") do |f| %>
+          <div>
+            Title:
+            <%= f.text_field "post.title" %>
+          </div>
+          <div>
+            Body:
+            <%= f.text_area "post.body" %>
+          </div>
+        <% end %>
+      ERB
+
+      before_prepare if respond_to?(:before_prepare)
+      require "hanami/prepare"
+    end
+  end
+
+  it "does not have a _csrf_token field when no sessions are configured" do
+    get "/posts/123/edit"
+
+    html = Capybara.string(last_response.body)
+
+    expect(html).to have_no_selector("input[type='hidden'][name='_csrf_token']")
+  end
+
+  it "uses the value from the view's locals" do
+    get "/posts/123/edit"
+
+    html = Capybara.string(last_response.body)
+
+    title_field = html.find("input[name='post[title]']")
+    expect(title_field.value).to eq "Hello <world>"
+
+    body_field = html.find("textarea[name='post[body]']")
+    expect(body_field.value).to eq "This is the post."
+  end
+
+  it "prefers the values from the request params" do
+    put "/posts/123", post: {title: "", body: "This is the UPDATED post."}
+
+    html = Capybara.string(last_response.body)
+
+    title_field = html.find("input[name='post[title]']")
+    expect(title_field.value).to eq ""
+
+    body_field = html.find("textarea[name='post[body]']")
+    expect(body_field.value).to eq "This is the UPDATED post."
+  end
+
+  context "sessions enabled" do
+    def before_prepare
+      write "config/app.rb", <<~RUBY
+        module TestApp
+          class App < Hanami::App
+            config.logger.stream = StringIO.new
+            config.actions.sessions = :cookie, {secret: "xyz"}
+          end
+        end
+      RUBY
+    end
+
+    it "inserts a CSRF token field" do
+      get "/posts/123/edit"
+
+      html = Capybara.string(last_response.body)
+
+      csrf_field = html.find("input[type='hidden'][name='_csrf_token']", visible: false)
+      expect(csrf_field.value).to match(/[a-z0-9]{10,}/i)
+    end
+  end
+end

data/spec/integration/view/helpers/part_helpers_spec.rb

@@ -0,0 +1,124 @@
+# frozen_string_literal: true
+
+require "ostruct"
+
+# rubocop:disable Style/OpenStructUse
+
+RSpec.describe "App view / Helpers / Part helpers", :app_integration do
+  before do
+    with_directory(make_tmp_directory) do
+      write "config/app.rb", <<~RUBY
+        module TestApp
+          class App < Hanami::App
+          end
+        end
+      RUBY
+
+      write "app/view.rb", <<~RUBY
+        # auto_register: false
+
+        require "hanami/view"
+
+        module TestApp
+          class View < Hanami::View
+            config.layout = nil
+          end
+        end
+      RUBY
+
+      before_prepare if respond_to?(:before_prepare)
+      require "hanami/prepare"
+    end
+  end
+
+  describe "app view and parts" do
+    def before_prepare
+      write "app/views/posts/show.rb", <<~RUBY
+        module TestApp
+          module Views
+            module Posts
+              class Show < TestApp::View
+                expose :post
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "app/views/parts/post.rb", <<~RUBY
+        module TestApp
+          module Views
+            module Parts
+              class Post < TestApp::Views::Part
+                def number
+                  format_number(value.number)
+                end
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "app/templates/posts/show.html.erb", <<~ERB
+        <h1><%= post.number %></h1>
+      ERB
+    end
+
+    it "makes default helpers available in parts" do
+      post = OpenStruct.new(number: 12_345)
+      output = TestApp::App["views.posts.show"].call(post: post).to_s.strip
+
+      expect(output).to eq "<h1>12,345</h1>"
+    end
+  end
+
+  describe "slice view and parts" do
+    def before_prepare
+      write "slices/main/view.rb", <<~RUBY
+        module Main
+          class View < TestApp::View
+          end
+        end
+      RUBY
+
+      write "slices/main/views/posts/show.rb", <<~RUBY
+        module Main
+          module Views
+            module Posts
+              class Show < Main::View
+                expose :post
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "slices/main/views/parts/post.rb", <<~RUBY
+        module Main
+          module Views
+            module Parts
+              class Post < Main::Views::Part
+                def number
+                  format_number(value.number)
+                end
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "slices/main/templates/posts/show.html.erb", <<~ERB
+        <h1><%= post.number %></h1>
+      ERB
+    end
+
+    it "makes default helpers available in parts" do
+      post = OpenStruct.new(number: 12_345)
+      output = Main::Slice["views.posts.show"].call(post: post).to_s.strip
+
+      expect(output).to eq "<h1>12,345</h1>"
+    end
+  end
+end
+
+# rubocop:enable Style/OpenStructUse

data/spec/integration/view/helpers/scope_helpers_spec.rb

@@ -0,0 +1,84 @@
+# frozen_string_literal: true
+
+RSpec.describe "App view / Helpers / Scope helpers", :app_integration do
+  before do
+    with_directory(make_tmp_directory) do
+      write "config/app.rb", <<~RUBY
+        module TestApp
+          class App < Hanami::App
+          end
+        end
+      RUBY
+
+      write "app/view.rb", <<~RUBY
+        # auto_register: false
+
+        require "hanami/view"
+
+        module TestApp
+          class View < Hanami::View
+            config.layout = nil
+          end
+        end
+      RUBY
+
+      before_prepare if respond_to?(:before_prepare)
+      require "hanami/prepare"
+    end
+  end
+
+  describe "app view" do
+    def before_prepare
+      write "app/views/posts/show.rb", <<~RUBY
+        module TestApp
+          module Views
+            module Posts
+              class Show < TestApp::View
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "app/templates/posts/show.html.erb", <<~ERB
+        <h1><%= format_number(12_345) %></h1>
+      ERB
+    end
+
+    it "makes default helpers available in templates" do
+      output = TestApp::App["views.posts.show"].call.to_s.strip
+      expect(output).to eq "<h1>12,345</h1>"
+    end
+  end
+
+  describe "slice view" do
+    def before_prepare
+      write "slices/main/view.rb", <<~RUBY
+        module Main
+          class View < TestApp::View
+          end
+        end
+      RUBY
+
+      write "slices/main/views/posts/show.rb", <<~RUBY
+        module Main
+          module Views
+            module Posts
+              class Show < Main::View
+              end
+            end
+          end
+        end
+      RUBY
+
+      write "slices/main/templates/posts/show.html.erb", <<~ERB
+        <h1><%= format_number(12_345) %></h1>
+      ERB
+    end
+
+    it "makes default helpers available in templates" do
+      output = Main::Slice["views.posts.show"].call.to_s.strip
+      expect(output).to eq "<h1>12,345</h1>"
+    end
+  end
+end