spec icon indicating copy to clipboard operation
spec copied to clipboard

Deprecate should_not raise_error in MSpec and replaces usages in ruby/spec

Open eregon opened this issue 5 years ago • 0 comments

See https://github.com/ruby/spec/pull/753/files#r376792963

should_not raise_error is the same as testing nothing. Each example already implicitly test no exception is raised. There might be a few legit cases, but I would expect the vast majority could be removed and replaced with useful expectations actually testing it works as expected.

Currently there are about 176 should_not raise_error in ruby/spec.

eregon avatar Feb 09 '20 15:02 eregon