Move shared Spark and Trainer test helpers to kubeflow.common

[Goku2099]

Summary

The Spark and Trainer SDKs currently maintain duplicate test utilities, including TestCase, SUCCESS, FAILED, and related helper functionality.

To reduce duplication and keep testing utilities consistent across SDK components, these common helpers should be moved to a shared location under kubeflow.common.

Motivation

Currently both components provide nearly identical implementations:

• kubeflow.spark.test.common
• kubeflow.trainer.test.common

Maintaining duplicate implementations increases maintenance overhead and can lead to inconsistencies over time.

Proposed Changes

• Create a shared test helper module under kubeflow.common.
• Move the common testing utilities into the shared module.
• Update Spark tests to import the shared implementation.
• Update Trainer tests to import the shared implementation.
• Remove duplicate implementations where appropriate.

Acceptance Criteria

• Shared test helpers are available under kubeflow.common.
• Spark and Trainer tests use the shared implementation.
• Existing test behavior remains unchanged.
• All affected test suites continue to pass.

ref : #685 (comment)

[Goku2099]

/good-first-issue

[google-oss-prow]

@Goku2099:
This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

Details

In response to this:

/good-first-issue

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[mehulkumaryadav]

Hi @Goku2099, I did like to work on this issue if it's available. I will move the shared Spark and Trainer test helpers into kubeflow.common, update both SDKs to use the shared implementation, and ensure all existing tests continue to pass. Could you please assign it to me?

[ChandelAnushka]

/assign

[ChandelAnushka]

Hi @Goku2099 ,

I've opened a PR #701 for this issue. Please review when available. Thanks!