Add additional tests to cover more scenario for the new kfp-tekton v1 api

kubeflow / kfp-tekton

Kubeflow Pipelines on Tekton

https://developer.ibm.com/blogs/kubeflow-pipelines-with-tekton-and-watson/

Apache License 2.0

173 stars 123 forks source link

Add additional tests to cover more scenario for the new kfp-tekton v1 api #1181

Open Tomcli opened 1 year ago

Tomcli commented 1 year ago

/kind feature

Description: Right now we see some edge cases in the new v1 api changes were not able to cover with our e2e test. Therefore, we need to create a set of tests just to list and get runs and experiments from the Python sdk/cli.

Additional information: [Miscellaneous information that will assist in solving the issue.]

Tomcli commented 1 year ago

related #1180 #1179

Tomcli commented 1 year ago

/assign @yhwang

yhwang commented 1 year ago

In existing e2e test cases, the CLI is used to upload pipelines, create runs, and list runs. Seems those changes address bugs for experiment-related APIs. I believe adding some test cases to call experiment-related APIs should be good for now. Any suggestion?

Tomcli commented 1 year ago

yeah we need to have some tests on listing experiments and runs. In the case of listing runs we might want to cover few cases like listing run with different namespaces to cover each condition.