@jcredding the only thing I don't like about this is that the test doesn't match the description.  The above test implementations (with similar descriptions to this one but for `new?` and `valid?`) match the descriptions but this one implements a test more like the system test.  I think either this unit test implemention needs to switch to `assert_equal @fake_test_record.destroyed?, subject.destroyed?` or (if this is not a good test) the other tests need to be updated to have unit implementations more like their system test counterparts.

Does this make sense?  Thoughts?

I'm more in favor of changing this test's implementation to match the others since the system test covers this implementation.  I think the goal laid out in these unit tests (given their description) is that the model and record `destroyed?` methods are in sync.  This implementation doesn't test that.

Anyway, love to hear your take on this.  Also, no immediate action needed either way. :smile: