Skip to content
This repository has been archived by the owner on Dec 31, 2023. It is now read-only.

test: delete system tests #435

Closed
wants to merge 1 commit into from
Closed

test: delete system tests #435

wants to merge 1 commit into from

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Sep 15, 2023

The system tests being deleted are handwritten but they are for a client which is fully autogenerated. Issue googleapis/gapic-generator-python#1771 has been filed to evaluate whether these tests can be migrated to gapic-showcase, which is closest to where the actual development occurs. We need to delete these tests as it is blocking the migration to google-cloud-python, where we are trying to minimize handwritten code.

The system tests being deleted are handwritten but they are for a client which is fully autogenerated. Issue googleapis/gapic-generator-python#1771 has been filed to evaluate whether these tests can be migrated to `gapic-showcase`, which is closest to where the actual development occurs.
@parthea parthea requested review from a team as code owners September 15, 2023 16:07
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: compute Issues related to the googleapis/python-compute API. labels Sep 15, 2023
@parthea parthea closed this Sep 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: compute Issues related to the googleapis/python-compute API. size: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant