I like it. This is the right approach, adding tests for both when it is set and when it's omitted. Too often, the unused case doesn't get a unit test.