maybe consider an enum
 should this log?
is there any more color on this
might be nice to have the test explicitly check that the subscription is cleaned up (though might need to restructure how the closures work in order to make those assertions)
I couldn't coax any difference in behavior out in my local messing around when setting those