Skip to content

tests/basics: Added test for len dict_view exception. #12386

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

eleciawhite
Copy link
Contributor

Test for issue 12385

@jimmo
Copy link
Member

jimmo commented Sep 11, 2023

Thanks for the PR @eleciawhite

We don't really have a concept of "tests that are known to fail", rather we would either create an issue to track it, or in the case of a difference in behavior to CPython (or unimplemented behavior like this one), we would also create a test in tests/cpydiff that documents the difference. These tests include a small amount of metadata at the top that is used to automatically generate https://docs.micropython.org/en/latest/genrst/index.html

I think this test would be a good candidate for that. Then if we later decide to fix it, the test can be moved into e.g. tests/basics/object_dict.py as part of the fix.

@dpgeorge dpgeorge added the tests Relates to tests/ directory in source label Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Relates to tests/ directory in source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy