I'm nitpicking, but I would prefer this line to be split in 2.
The call to `attached()` is really important, and I'm afraid that with everything on one line, somebody working on this code could remove it by error, and then spend a fair amount of time trying to fix the associated test. Putting it in a new statement will make it more visible.