Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upbpo-39299: Add more tests for mimetypes and its cli. #17949
Conversation
This comment has been minimized.
This comment has been minimized.
|
Ah, the logic to load the initial database is with known types, common types and the list of files from I guess my tests can be rewritten to make sure the tests are not machine dependent based on the files. |
* Set knownfiles for mimetypes to be empty list only during setup so that it doesn't leak into other tests. * Move the __main__ code to separate function for better testing.
This comment has been minimized.
This comment has been minimized.
|
I have moved the code for |
This comment has been minimized.
This comment has been minimized.
|
I am slightly confused since importing |
This comment has been minimized.
This comment has been minimized.
|
I resorted to using |
|
LGTM! Thanks @tirkarthi |


tirkarthi commentedJan 11, 2020
•
edited by bedevere-bot
read_mime_types.https://bugs.python.org/issue39299