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

test: delete system tests #451

Closed
wants to merge 1 commit into from
Closed

test: delete system tests #451

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 19:32
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: videointelligence Issues related to the googleapis/python-videointelligence 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: videointelligence Issues related to the googleapis/python-videointelligence API. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant