Skip to content

Port schedule/duplicate_error test to all SDK languages#800

Merged
lina-temporal merged 8 commits into
mainfrom
sched-duplicate-error-all-langs
Apr 10, 2026
Merged

Port schedule/duplicate_error test to all SDK languages#800
lina-temporal merged 8 commits into
mainfrom
sched-duplicate-error-all-langs

Conversation

@lina-temporal

@lina-temporal lina-temporal commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Ports the Go schedule/duplicate_error feature test (from Add a schedule feature test for ErrScheduleAlreadyRunning #799) to TypeScript, Java, Python, PHP, and .NET
  • Each test creates a schedule, then attempts to create another with the same ID, verifying the SDK-specific "already running" error is returned
  • Adds Java registration in PreparedFeature.java

Language-specific error types

Language Error type
Go temporal.ErrScheduleAlreadyRunning
TypeScript ScheduleAlreadyRunning
Java ScheduleAlreadyRunningException
Python ScheduleAlreadyRunningError
PHP ServiceClientException with StatusCode::ALREADY_EXISTS
.NET ScheduleAlreadyRunningException

Test plan

  • Go build passes (verified locally)
  • TypeScript build + feature tests pass
  • Java build + feature tests pass
  • Python build + feature tests pass
  • PHP build + feature tests pass
  • .NET build + feature tests pass

@lina-temporal lina-temporal marked this pull request as ready for review April 8, 2026 23:15
@lina-temporal lina-temporal requested review from a team as code owners April 8, 2026 23:15

@Sushisource Sushisource left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@lina-temporal lina-temporal enabled auto-merge (squash) April 10, 2026 15:57
@lina-temporal lina-temporal merged commit 77f9b4a into main Apr 10, 2026
25 checks passed
@lina-temporal lina-temporal deleted the sched-duplicate-error-all-langs branch April 10, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants