kubeflow/sdk

Move shared Spark and Trainer test helpers to kubeflow.common

开放

#700 创建于 2026年7月31日

 (5 条评论) (1 个反应) (1 位负责人)Python (196 个派生)auto 404
good first issuehelp wanted

仓库指标

星标
 (124 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

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 : https://github.com/kubeflow/sdk/pull/685#discussion_r3683712958

贡献者指南