Improve tests for AbstractDataSource subclasses

These are fairly basic objects with almost no tests, and tests should be straightforward.
We are missing a few defaults: `metadata`, `persist_data`, `index_dimension`. Any reason not to add them?