ONSdigital / dp-data-pipelines

Pipeline specific python scripts and tooling for automated website data ingress.
MIT License
1 stars 0 forks source link

issue #4 #26

Closed JimBryant7 closed 8 months ago

JimBryant7 commented 8 months ago

What

Added config.py which returns the identifier from the config dict, given from issue #4 Created unit tests to go along with this function. The "if" statement may be overkill but its included for potential different schemas to be added if needed.

How to review

Mainly a sanity check. Not super familiar with creating unit tests so maybe check that side a bit more, its a simple function and tests so shouldn't take much time. Also have created the tests in a new path, just check this path makes sense compared to original function.

Who can review

Anyone