Summary

Add --pytest argument to `rbext test`.

Review Request #13077 — Created May 30, 2023 and submitted June 5, 2023, 9:26 a.m.

Information

Owner

david

Repository

Review Board

Branch

release-6.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

This adds a flag for opting into the new pytest-based test runner.

Testing Done


Ran rbext test -m rbintegrations and saw it run the tests with the

  old nose runner, showing a warning about the change in the future.
Ran rbext test -m rbintegrations --pytest and saw it run all of the

  same tests via pytest.

Commits

Summary	ID
Add --pytest argument to `rbext test`. This adds a flag for opting into the new pytest-based test runner. Testing Done: - Ran `rbext test -m rbintegrations` and saw it run the tests with the old nose runner, showing a warning about the change in the future. - Ran `rbext test -m rbintegrations --pytest` and saw it run all of the same tests via pytest.	13641b3a33c9063c765d72dfd8e30ba86a253075

Issues

Description	From	Last Updated
Awesome! How about we make this default today if a conftest.py is present, and then generate an empty one (or, …	chipx86	May 31, 2023, 9:36 a.m.

flake8 passed.

JSHint passed.

The issue has been resolved. Show all issues

Awesome! How about we make this default today if a conftest.py is present, and then generate an empty one (or, well, one with a comment probably) when generating a source tree?

david May 31, 2023, 9:36 a.m.

I'll add the creation part in a separate change. We'll also want a few small bits in setup.cfg to match.

Commits:

	Summary	ID
	Add --pytest argument to `rbext test`. This adds a flag for opting into the new pytest-based test runner. Testing Done: - Ran `rbext test -m rbintegrations` and saw it run the tests with the old nose runner, showing a warning about the change in the future. - Ran `rbext test -m rbintegrations --pytest` and saw it run all of the same tests via pytest.	463c5f2e23bd591941de8294489cd4ee3af9a667
	Add --pytest argument to `rbext test`. This adds a flag for opting into the new pytest-based test runner. Testing Done: - Ran `rbext test -m rbintegrations` and saw it run the tests with the old nose runner, showing a warning about the change in the future. - Ran `rbext test -m rbintegrations --pytest` and saw it run all of the same tests via pytest.	13641b3a33c9063c765d72dfd8e30ba86a253075

Diff:

Revision 2 (+42 -8)

Show changes

reviewboard/cmdline/rbext.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-6.x (02d4fed)