Skip to content

Port schedule/duplicate_error test to all SDK languages#800

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

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

Conversation

@lina-temporal
Copy link
Copy Markdown
Contributor

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

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
Copy link
Copy Markdown
Member

@Sushisource Sushisource left a comment

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