appendto/amplify — comment on commit 80676fd7106cce7794ced5850f7f8906302e3cc2
File:  line 0
By gitgrimbo:

Hi, I'm not suggesting the fix is incorrect, just that the "unsubscribe during publish" test only seems to test that the unsubscription does not produce a runtime error.

It does not actually seem to test that the racer() function was successfully unsubscribed - in the sense that it will not be called again should the "racy" topic be published again. So maybe putting a test in racer and publishing "racy" a second time within the test:

```
var alreadyCalled = false;
function racer() {
ok( !alreadyCalled, "second" );
amplify.unsubscribe( "racy", racer );
alreadyCalled = true;
}
```