Skip to content

Maybe outdated pytest command in README.md #157

@17999824wyj

Description

@17999824wyj

Hello, I noticed that the README.md file for the python-cffi library contains an outdated command in the "Testing/development tips" section. The script suggests running pytest c/ testing/, which seems to be no longer valid or up-to-date.

Actual Behavior

To easily descript, I make one assumption:

  • After git clone or wget && tar, we will get a directory called cffi or cffi-x.x.x. I call it repo-directory.

But in my computer(windows 11), when using the command pytest c/ testing/ in repo-directory, there is a mistake as below:

Image

Then I change directory to repo-directory/src and then use the same command:

Image

The command pytest c/ testing/ may not work as expected. But the command pytest src/c/ testing/ when in the repo-directory or the command pytest c/ ../testing/ in the repo-directory/src can both work.

So, I don't know if it's because I'm using it incorrectly or if the documentation is outdated... And I will give a PR to fix that by using pytest src/c/ testing/.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions