I usually try to reduce the scope of the execution context as much as possible, particularly when using the default one. People tend to not notice the file-level import (myself included) because some IDE collapse the imports. Explicitly declaring the execution context tend to raise awareness while it doesn't hurt usability.

In this case it probably doesn't make much difference as the enclosing trait is used by all tests. Do you feel I should change it?