Summary:

Switch to pytest as the test runner.

Review Request #12487 — Created July 25, 2022 and submitted July 26, 2022, 6:33 p.m.

Information
Owner:	chipx86
Repository:	beanbag-docutils
Branch:	master
Bugs:
Depends On:
Reviewers
Groups:	beanbag-misc
People:

Description

This updates the codebase to use pytest instead of nose for running
unit tests. This helps us with testing on newer versions of Python, and
is consistent with our other codebases.

The old tests/runtests.py script will now wrap pytest.

Testing Done

Ran unit tests on all supported versions of Python.

Commits

Summary
	Switch to pytest as the test runner.

Screenshots

Issues

Description	From	Last Updated
Do we not want to have a runtests.py file that just invokes pytest?	david	July 25, 2022, 11:48 p.m.
It looks like tests/runtests.py is still deleted? There's no difference between diffs 1 and 2.	maubin	July 26, 2022, 5:59 p.m.

flake8 passed.

JSHint passed.

Show all issues

Do we not want to have a runtests.py file that just invokes pytest?

Change Summary:

Restored runtests.py, having it wrap pytest.

Description:

		This updates the codebase to use `pytest` instead of `nose` for running
		unit tests. This helps us with testing on newer versions of Python, and
		is consistent with our other codebases.
	+
	+	The old `tests/runtests.py` script will now wrap `pytest`.

Commits:

	Summary
-		Switch to pytest as the test runner.
+		Switch to pytest as the test runner.

Diff:

Revision 2 (+16 -56)

Show changes

	dev-requirements.txt
	tests/runtests.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Show all issues

It looks like tests/runtests.py is still deleted? There's no difference between diffs 1 and 2.

chipx86 July 26, 2022, 5:59 p.m.
```
Yyyyep. Oops. Fixing.
```

Change Summary:

Actually added back runtests.py.

Commits:

	Summary
-		Switch to pytest as the test runner.
+		Switch to pytest as the test runner.

Diff:

Revision 3 (+22 -44)

Show changes

	dev-requirements.txt
	tests/runtests.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to master (7927ed8)