Create pytest infrastructure in `rbext create`

Review Request #13084 — Created May 31, 2023 and submitted

Information

Review Board
release-6.x

Reviewers

Now that we can use pytest in extensions, we want to create that as part
of creating a new extension. This change adds the machinery to generate
conftest.py and pyproject.toml files with the correct setup to run
tests.

Created a test extension and saw that it was set up to run tests.

Summary ID
Create pytest infrastructure in `rbext create`
Now that we can use pytest in extensions, we want to create that as part of creating a new extension. This change adds the machinery to generate `conftest.py` and `pyproject.toml` files with the correct setup to run tests. Testing Done: Created a test extension and saw that it was set up to run tests.
5288bb93231ac941318c8eee1da1324721d0d404
Description From Last Updated

We should probably generate a pyproject.toml instead, putting this there, and also stop generating a setup.py. I know that last …

chipx86chipx86

Are we able to set these in our fixtures file? Also wondering about that for many of these other options. …

chipx86chipx86

This is duplicated.

chipx86chipx86
chipx86
  1. 
      
  2. reviewboard/cmdline/rbext.py (Diff revision 1)
     
     
     
     
    Show all issues

    We should probably generate a pyproject.toml instead, putting this there, and also stop generating a setup.py. I know that last part is outside the scope of this change, but with setup.py going away, that'll be important.

  3. reviewboard/cmdline/rbext.py (Diff revision 1)
     
     
     
    Show all issues

    Are we able to set these in our fixtures file?

    Also wondering about that for many of these other options. I suspect some can be, but I don't know.

    1. Setting environment variables from a fixture doesn't seem to work.

  4. 
      
david
chipx86
  1. 
      
  2. reviewboard/cmdline/rbext.py (Diff revision 2)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
    Show all issues

    This is duplicated.

  3. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-6.x (81398ef)
Loading...