Display helpful message if attribute_name missing

lib/administrate/base_dashboard.rb

@@ -18,6 +18,18 @@ def attribute_types
       self.class::ATTRIBUTE_TYPES
     end
 
+    def attribute_type_for(attribute_name)
+      attribute_types.fetch(attribute_name) do
+        fail "Attribute #{attribute_name} not in #{self.class}::ATTRIBUTE_TYPES"
+      end
+    end
+
+    def attribute_types_for(*attribute_names)
+      attribute_names.each_with_object({}) do |name, attributes|
+        attributes[name] = attribute_type_for(name)
+      end
+    end
+
     def form_attributes
       self.class::FORM_ATTRIBUTES
     end

lib/administrate/page/base.rb

@@ -15,10 +15,8 @@ def resource_name
 
       def attribute_field(dashboard, resource, attribute_name, page)
         value = get_attribute_value(resource, attribute_name)
-
-        dashboard.
-          attribute_types[attribute_name].
-          new(attribute_name, value, page)
+        field = dashboard.attribute_type_for(attribute_name)
+        field.new(attribute_name, value, page)
       end
 
       def get_attribute_value(resource, attribute_name)

lib/administrate/page/collection.rb

@@ -14,7 +14,7 @@ def attributes_for(resource)
       end
 
       def attribute_types
-        dashboard.attribute_types.slice(*attribute_names)
+        dashboard.attribute_types_for(*attribute_names)
       end
 
       def ordered_html_class(attr)

spec/dashboards/customer_dashboard_spec.rb

@@ -24,6 +24,45 @@
     end
   end
 
+  describe "#attribute_type_for" do
+    context "for an existing attribute" do
+      it "returns the attribute field" do
+        dashboard = CustomerDashboard.new
+        field = dashboard.attribute_type_for(:name)
+        expect(field).to eq Administrate::Field::String
+      end
+    end
+
+    context "for a non-existent attribute" do
+      it "raises an exception" do
+        dashboard = CustomerDashboard.new
+        expect { dashboard.attribute_type_for(:foo) }.
+          to raise_error /Attribute foo not in CustomerDashboard/
+      end
+    end
+  end
+
+  describe "#attribute_types_for" do
+    context "for existing attributes" do
+      it "returns the attribute fields" do
+        dashboard = CustomerDashboard.new
+        fields = dashboard.attribute_types_for(:name, :email)
+        expect(fields).to match(
+          name: Administrate::Field::String,
+          email: Administrate::Field::Email,
+        )
+      end
+    end
+
+    context "for one non-existent attribute" do
+      it "raises an exception" do
+        dashboard = CustomerDashboard.new
+        expect { dashboard.attribute_types_for(:name, :foo) }.
+          to raise_error /Attribute foo not in CustomerDashboard/
+      end
+    end
+  end
+
   describe "#display_resource" do
     it "returns the customer's name" do
       customer = double(name: "Example Customer")