flake8
separate parsing from reading file contents, for easier testing
doctest for _get_metadata_from_file