This repository was archived by the owner on Aug 30, 2026. It is now read-only.

Repository navigation

This repository was archived by the owner on Aug 30, 2026. It is now read-only.

Split `tests/system.py` into separate modules, refactor to use `pytest` fixtures. #391

Closed

Assignees

Labels

api: bigtabletype: process

opened

on Jul 21, 2021

The tests/system.py module is huge:

$ wc -l tests/system.py
1504 tests/system.py

and has complicated, idiosyncratic setup / teardown (it long predates our use of pytest idioms).

For reference, see googleapis/python-storage#476.

tseaver
added
type: processA process-related concern. May include testing, release, or the like.
on Jul 21, 2021
tseaver
self-assigned this
on Jul 21, 2021
product-auto-label
added
api: bigtableIssues related to the googleapis/python-bigtable API.
on Jul 21, 2021
tseaver
added a commit that references this issue on Jul 22, 2021
tests: remove remaining monolith fixtures
6b4121c
tseaver
added a commit that references this issue on Sep 14, 2021
tests: remove remaining monolith fixtures
00da0ae
tseaver
closed this as completedin #393 on Sep 15, 2021
tseaver
added a commit that references this issue on Sep 15, 2021
tests: refactor systests into separate modules w/ pytest fixtures (#393)
83958df

to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

tseaver

Labels

api: bigtabletype: process

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests