I think ITestDataSources shouldn't be concerned with how they are displayed.

I think it may be better to either remove GetDisplayName completely, or introduce it as a separate interface

To be investigated: what is the current behavior if an implementation of GetDisplayName returned null. Are we going to fallback to a "nice" display name that correctly takes into account the parameters of the test?