ApprovalTests.Ruby icon indicating copy to clipboard operation
ApprovalTests.Ruby copied to clipboard

A hash ( # ) should not be stripped from approval name

Open markijbema opened this issue 11 years ago • 3 comments

I noticed I test a lot of controller actions, so my names tend to be something like

"blogposts#show.json should stay the same"

I think we can do better handling of the # than stripping it. (this would mean a breaking change though, but I think this is still okay, as long as we add it to the changelog)

markijbema avatar Dec 18 '13 10:12 markijbema

We still haven't hit 1.0, so breaking changes are considered acceptable.

How do you want to handle it. Replace it with a hyphen, perhaps?

kytrinyx avatar Dec 18 '13 13:12 kytrinyx

Yeah, hyphen or underscore seems fine (i recall that everything is changed to underscore, but I don't have the conversion at hand).

Btw, I'm perfectly willing to implement this somewhere in the coming weeks, but I noticed it, so just jotted it down here, so I don't forget again (because I noticed it before ;) )

markijbema avatar Dec 18 '13 14:12 markijbema

Ok, cool -- and agreed, this is a good change.

kytrinyx avatar Dec 18 '13 14:12 kytrinyx