Excluding attributes from serializers. Towards #26

app/serializers/board_serializer.rb

@@ -1,5 +1,5 @@
 class BoardSerializer
   include TalkSerializer
-  all_attributes
+  all_attributes except: :permissions
   can_include :discussions
 end

app/serializers/concerns/talk_serializer.rb

@@ -11,9 +11,10 @@ module TalkSerializer
   end
 
   module ClassMethods
-    def all_attributes
-      attrs = model_class.attribute_names rescue []
-      attributes *attrs
+    def all_attributes(except: [])
+      attrs = model_class.attribute_names.sort rescue []
+      except = Array.wrap(except).sort.collect &:to_s
+      attributes *(attrs.sort - except)
     end
   end
 

app/serializers/user_serializer.rb

@@ -1,4 +1,4 @@
 class UserSerializer
   include TalkSerializer
-  all_attributes
+  all_attributes except: :email
 end

spec/serializers/board_serializer_spec.rb

@@ -1,5 +1,5 @@
 require 'spec_helper'
 
 RSpec.describe BoardSerializer, type: :serializer do
-  it_behaves_like 'a talk serializer', exposing: :all, including: [:discussions]
+  it_behaves_like 'a talk serializer', exposing: :all, excluding: [:permissions], including: [:discussions]
 end

spec/serializers/user_serializer_spec.rb

@@ -1,5 +1,5 @@
 require 'spec_helper'
 
 RSpec.describe UserSerializer, type: :serializer do
-  it_behaves_like 'a talk serializer', exposing: :all
+  it_behaves_like 'a talk serializer', exposing: :all, excluding: [:email]
 end

spec/support/shared_examples_for_talk_serializer.rb

@@ -8,7 +8,8 @@
   let(:json){ serializer.as_json model_instance }
 end
 
-RSpec.shared_examples_for 'a talk serializer' do |exposing: nil, including: nil|
+RSpec.shared_examples_for 'a talk serializer' do |exposing: nil, excluding: [], including: nil|
+  [exposing, excluding, including].compact.each{ |opt| opt.map!(&:to_sym) if opt.is_a?(Array) }
   include_context 'a serializer'
   let(:model_instance){ defined?(object) ? object : instance }
 
@@ -17,14 +18,18 @@
 
     if exposing == :all
       described_class.model_class.attribute_names.each do |name|
-        it{ is_expected.to include name.to_sym }
+        it{ is_expected.to include name.to_sym } unless name.to_sym.in?(excluding)
       end
     elsif exposing
       exposing.each do |name|
         it{ is_expected.to include name.to_sym }
       end
     end
 
+    excluding.each do |name|
+      it{ is_expected.to_not include name.to_sym }
+    end
+
     it{ is_expected.to include :href }
   end