Don't allow redefining the default view

Signed-off-by: Jordan Hollinger <[email protected]>

lib/blueprinter/v2/view_builder.rb

@@ -26,7 +26,10 @@ def initialize(parent)
       # @param definition [Proc]
       #
       def []=(name, definition)
-        @pending[name.to_sym] = definition
+        name = name.to_sym
+        raise Errors::InvalidBlueprint, 'You may not redefine the default view' if name == :default
+
+        @pending[name] = definition
       end
 
       #

spec/v2/view_builder_spec.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe "Blueprinter::V2::ViewBuilder" do
+describe Blueprinter::V2::ViewBuilder do
   let(:builder) do
     Blueprinter::V2::ViewBuilder.new(blueprint)
   end
@@ -60,4 +60,17 @@
     keys = builder.each.map { |name, _| name }
     expect(keys.sort).to eq %i(default foo bar).sort
   end
+
+  it "doesn't throw an error if you try to redefine an existing view" do
+    builder[:foo] = proc { field :description }
+    expect do
+      builder[:foo] = proc { field :description }
+    end.to_not raise_error
+  end
+
+  it "throws an error if you try to define the default view" do
+    expect {
+      builder[:default] = proc { field :description }
+    }.to raise_error Blueprinter::Errors::InvalidBlueprint
+  end
 end