Okay, these tests are nearly there. I notice that you're still spying on `triggerMethod` with this style `sinon.spy(region, "triggerMethod");`. 

we've been using this style throughout our test suite:

```js
closeHandler = sinon.spy();
region.on("close", closeHandler);
expect(closeHandler).toHaveBeenCalled();
```

Here's another example of how we do it in our [module suite](https://github.com/marionettejs/backbone.marionette/blob/master/spec/javascripts/module.spec.js#L334-L343). 

I think we've been over this a couple times, but we don't spy on `triggerMethod` because it's an internal implementation detail that's subject to change. Don't worry about testing both triggerMethod's event and hook, we generally test one, and in most cases the event, because it's easier to bind to. 

Sorry, that this is a sticking point, but I checked and `triggerMethod` isn't spied on anywhere else and even though it's a relatively small deal, I would prefer not to start doing it in this one place. Once these tests are consistent, I think it will be ready to be merged in.