Closed pont-us closed 4 years ago
@TonioF Thanks for the suggestions -- all implemented. I've put the new tests into a new ClientUrlTest
class, added docstrings, added setUp
and tearDown
functions, and done some other refactoring to remove duplication and improve code readability. Existing environment variables are now properly saved and restored.
This PR adds
cds_api_url
andcds_api_key
parameters to theCDSDataOpener
andCDSDataStore
constructors. Previously, the CDS API URl and key could only be specified via environment variables or a configuration file.