I'm fine with this approach. I probably would have done separate tests for true and false. As you have it you're calling the method under test twice in one test.