styling tweaks

lib/paperclip/has_attached_file.rb

@@ -92,7 +92,9 @@ def add_active_record_callbacks
       @klass.send(:after_save) { send(name).send(:save) }
       @klass.send(:before_destroy) { send(name).send(:queue_all_for_delete) }
       if @klass.respond_to?(:after_commit)
-        @klass.send(:after_commit, :on => :destroy) { send(name).send(:flush_deletes) }
+        @klass.send(:after_commit, on: :destroy) do
+          send(name).send(:flush_deletes)
+        end
       else
         @klass.send(:after_destroy) { send(name).send(:flush_deletes) }
       end

spec/paperclip/has_attached_file_spec.rb

@@ -38,12 +38,13 @@
       assert_adding_attachment('avatar').defines_callback('after_commit')
     end
 
-    it 'defines an after_commit callback if the class allows after_commit callbacks' do
-      assert_adding_attachment('avatar').defines_callback('after_commit')
-    end
-
-    it 'defines an after_destroy callback if the class does not allow after_commit callbacks' do
-      assert_adding_attachment('avatar', :unstub_methods => [:after_commit]).defines_callback('after_destroy')
+    context 'when the class does not allow after_commit callbacks' do
+      it 'defines an after_destroy callback' do
+        assert_adding_attachment(
+          'avatar',
+          unstub_methods: [:after_commit]
+        ).defines_callback('after_destroy')
+      end
     end
 
     it 'defines the Paperclip-specific callbacks' do