Skip to content

Add stdlib/@tests? #11762

Closed
Closed

Description

@srittau

There's a slight inconsistency that's been bugging me: Third-party stubs – sensibly – can have a @tests directory, which contains the stubtest allowlists and regression tests. On the other hand, the allowlists for the standard library are in tests/stubtest_allowlists and the test cases in test_cases/stdlib. (The test_cases directory is otherwise empty, except for a README file.) For consistency's sake and easier discoverability, I suggest we add a stdlib/@tests directory with the same structure as the corresponding stubs/*/@tests directories and move the existing files there.

Activity

  1. srittau commented on Apr 14, 2024

    @srittau
    CollaboratorAuthor

    Type checker authors would be encouraged to omit that directory from their distributions, although I don't see harm in including it.

  2. added a commit that references this issue on May 15, 2024
  3. linked a pull request that will close this issueMove stdlib allowlists to stdlib/@tests #11982on May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions