@Kristinita I think you accidentally included some unrelated changes in your last push

Type: Reply 💬

@brianschubert, I’m sorry,I made a mistake in the test file and create the new commit that fix tests. I can’t push my changes to GitHub, so I rungit fetch,git rebase and then push changes to GitHub.

I can close this pull request,update my CPython fork and send a new pull request. Or should I do something else? I can’t find instructions for this case in Python Developer’s Guide.

I apologize for the concern.

Thanks.

No worries! And no need to open a new PR. You can undo the extra changes by runninggit restore -s HEAD~1 ./ (which will reset your working tree to what it was before the commit), then remake the intended change totest_mimetypes.py and create a new commit.

@Kristinita please read ourAI policy.

A

Type: Fixed ✔️

@brianschubert , done. All checks have passed now.

Thanks.

Please could you add it to What's New?

https://docs.python.org/3.14/whatsnew/3.14.html#mimetypes

And add a test for .yml?

Type: Question ❓

1. Question

And add a test for .yml?

@hugovk, what is the preferred format for tests for non-preferred extensions like.yml? I was looking for the answer to this in previous pull requests that add new media types, but I can’t find a conventional method.

2. Missing tests

Currently, the filetest_mimetypes.py in many cases contains tests solely for preferred extensions. Examples:

1. Foraudio/mpeg media type:test_mimetypes.py has the test for the.mp3 preferred extension, but not for the non-preferred extension.mp2.
2. Forvideo/mpeg media type:test_mimetypes.py has the test for the.mpeg preferred extension, but not for non-preferred extensions.m1v,.mpa,.mpe and.mpg.

3. Existing test for non-preferred extensions

Should I create a test like the test for thetext/plain media type?

4. Simpler adding tests

Possibly, it would be nice if adding tests for non-preferred extension will be simpler. For example, the filetest_mimetypes.py could have a function liketest_non_preferred_extensions to which users will be able to add all non-preferred extensions for media types. Example:

("application/yaml",".yml"),("audio/mpeg",".mp2"),("video/mpeg",".m1v"),("video/mpeg",".mpa"),("video/mpeg",".mpe"),("video/mpeg",".mpg")

Thanks.

We could add some a test case to check the output ofguess_file_type is as expected, for example:

deftest_guess_file_type(self):defcheck_file_type():formime_type,extin (                ("application/yaml",".yaml"),                ("application/yaml",".yml"),            ):withself.subTest(mime_type=mime_type,ext=ext):result,_=mimetypes.guess_file_type(f"filename{ext}")self.assertEqual(result,mime_type)check_file_type()mimetypes.init()check_file_type()

No need to exhaustively fill lots of other preferred+optional extensions, but we could include a few more, such as theaudio/mpeg andvideo/mpeg ones mentioned.

All commit authors signed the Contributor License Agreement.