I think the ## should be :# so we don't imply it's an instance method. It might be a class method, we don't know.

See the above discussion about instance method

    I think #{@target_method} should also be prefixed with a :.
    RSpec adds "should" before the description for us, so this shouldn't be prefixed with should.

Agreed. Both fixed.