Skip to content

Commit

Permalink
Taking care of @houndci comments, but not sure to which line specific…
Browse files Browse the repository at this point in the history
…aly is referring too :)
  • Loading branch information
cavi21 authored and Jon Yurek committed Feb 6, 2015
1 parent e4c04c2 commit ba7cf29
Showing 1 changed file with 12 additions and 12 deletions.
24 changes: 12 additions & 12 deletions spec/paperclip/attachment_registry_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@
it 'calls the block with the class, attachment name, and options' do
foo = Class.new
expected_accumulations = [
[foo, :avatar, { yo: 'greeting' }],
[foo, :greeter, { ciao: 'greeting' }]
[foo, :avatar, { yo: "greeting" }],
[foo, :greeter, { ciao: "greeting" }]
]
expected_accumulations.each do |args|
Paperclip::AttachmentRegistry.register(*args)
Expand All @@ -50,19 +50,19 @@
context '.definitions_for' do
it 'produces the attachment name and options' do
expected_definitions = {
avatar: { yo: 'greeting' },
greeter: { ciao: 'greeting' }
avatar: { yo: "greeting" },
greeter: { ciao: "greeting" }
}
foo = Class.new
Paperclip::AttachmentRegistry.register(
foo,
:avatar,
{ yo: 'greeting' }
yo: "greeting"
)
Paperclip::AttachmentRegistry.register(
foo,
:greeter,
{ ciao: 'greeting' }
ciao: "greeting"
)

definitions = Paperclip::AttachmentRegistry.definitions_for(foo)
Expand All @@ -71,7 +71,7 @@
end

it 'produces defintions for subclasses' do
expected_definitions = { avatar: { yo: 'greeting' } }
expected_definitions = { avatar: { yo: "greeting" } }
foo = Class.new
bar = Class.new(foo)
Paperclip::AttachmentRegistry.register(
Expand All @@ -86,12 +86,12 @@
end

it 'produces defintions for subclasses but deep merging them' do
foo_definitions = { avatar: { yo: 'greeting' } }
bar_definitions = { avatar: { ciao: 'greeting' } }
foo_definitions = { avatar: { yo: "greeting" } }
bar_definitions = { avatar: { ciao: "greeting" } }
expected_definitions = {
avatar: {
yo: 'greeting',
ciao: 'greeting'
yo: "greeting",
ciao: "greeting"
}
}
foo = Class.new
Expand All @@ -118,7 +118,7 @@
foo = Class.new
Paperclip::AttachmentRegistry.register(foo,
:greeter,
{ ciao: 'greeting' })
{ ciao: "greeting" })

Paperclip::AttachmentRegistry.clear

Expand Down

0 comments on commit ba7cf29

Please sign in to comment.