Rename `Integration#can_test?` to `Integration#testable?`
Generally, can_
, is_
and has_
prefixes should be avoided in predicate methods.
The following discussion from !65113 (merged) should be addressed:
-
@alexkalderimis started a discussion: (+1 comment) I think if we leave the
subject
as the integration, then this can be written as:it { is_expected.to be_can_test }
which (while a little awkward, language-wise), is better since the expectation can provide nicer failure messages.
(I note that this points us towards perhaps wanting to rename this method to
#testable?
or similar, which would make forit { is_expected.to be_testable }
)WDYT?