Skip to content

refactor(core): simplify integration test fixtures#33292

Merged
thdxr merged 6 commits into
devfrom
core-test-fixtures
Jun 22, 2026
Merged

refactor(core): simplify integration test fixtures#33292
thdxr merged 6 commits into
devfrom
core-test-fixtures

Conversation

@thdxr

@thdxr thdxr commented Jun 22, 2026

Copy link
Copy Markdown
Member

Summary

  • run core tests through the existing Turbo CI unit graph
  • default core tests to an in-memory database via Bun preload
  • rename credential value and stored-record types for clarity
  • simplify integration connection resolution and catalog availability
  • replace internal service mocks with real default/location layers
  • add a shared real PluginTestLayer and migrate provider plugin tests to the production plugin host path

Validation

  • repository-wide typecheck passes across 23 packages
  • migrated provider test groups pass
  • core service and integration test groups pass

Status

Draft while the remaining bespoke provider tests are migrated off provider-helper.ts.

@thdxr thdxr marked this pull request as ready for review June 22, 2026 01:47

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@thdxr thdxr enabled auto-merge (squash) June 22, 2026 01:48
@thdxr thdxr merged commit 2bb4311 into dev Jun 22, 2026
8 checks passed
@thdxr thdxr deleted the core-test-fixtures branch June 22, 2026 04:15
markjaquith pushed a commit to markjaquith/opencode that referenced this pull request Jun 23, 2026
BenGu3 pushed a commit to BenGu3/opencode that referenced this pull request Jun 27, 2026
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.

1 participant