describe TvShowParslet do # I probably would make this a longer method name, since it's a bit hard to follow # the below specs, not being the original writer of them without looking at what the # definition is. # Very minor nitpick # eg: # it { should p(" - ") } # versus # it { should parse(" - ") } def p(string) parse(string, :trace => true) end #.... describe :separator do # I'd put this all here, one less bit of indirection, so the reader doesn't have # to jump around to see what's going on. # I'm guessing though that parse somehow implicitly depends on the parser method? subject { TvShowParslet.new.separator } it { should p(" - ") } it { should p("-") } it { should p("_") } it { should p(".") } end end