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

test: delete system tests #523

Closed
wants to merge 1 commit into from
Closed

test: delete system tests #523

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. We need to delete these tests as it is blocking the migration to `google-cloud-python`, where we are trying to minimize handwritten code.
@parthea parthea requested a review from a team as a code owner September 15, 2023 16:44
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: dlp Issues related to the googleapis/python-dlp 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: dlp Issues related to the googleapis/python-dlp API. size: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant