Fix: Add support for rendering multiple scopes by passing string as n…

…ames

Resolves: hanami#252

lib/hanami/view/scope_builder.rb

@@ -31,7 +31,7 @@ def scope_class(name = nil, rendering:)
  elsif name.is_a?(Class)
  name
  else
- View.cache.fetch_or_store(:scope_class, rendering.config) do
+ View.cache.fetch_or_store("scope_class: #{name}", rendering.config) do
  resolve_scope_class(name: name, rendering: rendering)
  end
  end
@@ -40,7 +40,7 @@ def scope_class(name = nil, rendering:)
  def resolve_scope_class(name:, rendering:)
  name = rendering.inflector.camelize(name.to_s)
 
- namespace = rendering.config.scope_namespace
+ namespace = rendering.config.scope_namespace || Object
 
  # Give autoloaders a chance to act
  begin

spec/integration/scope_builder_spec.rb

@@ -27,5 +27,31 @@
 
  expect(scope).to be_an_instance_of scope_class
  end
+
+ context 'when multiple scopes are rendered in the same view' do
+
+ let(:builder) { described_class.new }
+
+ it 'allows to build scopes with different classes' do
+ FirstScopeClass = Class.new(Hanami::View::Scope)
+ SecondScopeClass = Class.new(Hanami::View::Scope)
+
+ view = Class.new(Hanami::View) do
+ config.paths = SPEC_ROOT.join("__ignore__")
+ config.template = "__ignore__"
+
+ config.scope_class = FirstScopeClass
+ end.new
+
+ scope = view.rendering.scope({})
+ expect(scope).to be_an_instance_of FirstScopeClass
+
+ scope = view.rendering.scope('SecondScopeClass', {})
+ expect(scope).to be_an_instance_of SecondScopeClass
+
+ scope = view.rendering.scope('FirstScopeClass', {})
+ expect(scope).to be_an_instance_of FirstScopeClass
+ end
+ end
  end
 end