Skip to content

Usdk-e2e status check; testapp honors redisKeyPrefix - #100

Merged
bpapillon merged 5 commits into
mainfrom
sdk-e2e-status-check
Sep 6, 2026
Merged

Usdk-e2e status check; testapp honors redisKeyPrefix#100
bpapillon merged 5 commits into
mainfrom
sdk-e2e-status-check

Conversation

@bpapillon

@bpapillon bpapillon commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

For the SDK E2E suite (SchematicHQ/actions#565):

  • sdk-e2e-status.yml posts an sdk-e2e commit status on every PR: pending on update-wasm-v* bump branches until the schematic-api E2E run reports back, success immediately on anything else. This lets sdk-e2e become a required check without blocking ordinary PRs.

  • The testapp honors redisKeyPrefix so the new replicator-docs E2E mode can run the README configuration.

  • Sample app fix: withReplicatorMode(replicatorUrl) passed the bare base URL as the health URL. That is a 404, the SDK read it as not ready, and every replicator-mode check fell back to the REST API, so the Java replicator E2E job has only ever tested REST. Now replicatorUrl + "/ready" like the other testapps. Found by the strict replicator mode in actions#565.

  • Sample app fix: entity caches now keep the SDK default TTL (the other testapps made the same change). The short flag-check TTL on the datastream caches expired the replicator-owned company entry after track wrote it back, and every later check logged Company not found in cache.

  • Sample app logs the SDK at debug so the E2E job's test app log shows fallbacks and replicator health.

Posts an sdk-e2e commit status: pending on update-wasm-v* bump branches
until the schematic-api SDK E2E run reports back, success on every other
PR, so sdk-e2e can be a required check without blocking normal PRs. The
testapp passes redisKeyPrefix through so the E2E replicator-docs mode
runs the README configuration.
The sample app passed the bare replicator URL as the health URL. That
is a 404, the SDK read it as not ready, and every replicator-mode flag
check fell back to the REST API; the E2E replicator job passed on REST
values. The strict replicator E2E mode (SchematicHQ/actions#565) caught
it.
@schematic-bot

Copy link
Copy Markdown
Contributor

SDK E2E failure for sdk-e2e-status-check at 68f58bdrun

mode result failed step
replicator-docs ❌ failure Run E2E tests
replicator ❌ failure Run E2E tests

The sdk-e2e status on this PR reflects this run. A new push resets it to pending; re-run sdk_e2e.yml with sdk-ref=sdk-e2e-status-check to report on the new head.

@schematic-bot

Copy link
Copy Markdown
Contributor

SDK E2E failure for sdk-e2e-status-check at ba9052brun

mode result failed step
replicator ❌ failure Run E2E tests

The sdk-e2e status on this PR reflects this run. A new push resets it to pending; re-run sdk_e2e.yml with sdk-ref=sdk-e2e-status-check to report on the new head.

Same fix as the other testapps: the short flag-check TTL on the entity
caches expired the replicator-owned company entry after the SDK wrote
it back on track, and every later check missed the company.
@schematic-bot

Copy link
Copy Markdown
Contributor

SDK E2E success for sdk-e2e-status-check at 7115b8drun

mode result failed step
replicator-docs ✅ passed
replicator ✅ passed

The sdk-e2e status on this PR reflects this run. A new push resets it to pending; re-run sdk_e2e.yml with sdk-ref=sdk-e2e-status-check to report on the new head.

@bpapillon
bpapillon marked this pull request as ready for review September 6, 2026 05:11
@bpapillon
bpapillon requested a review from a team as a code owner September 6, 2026 05:11
@bpapillon
bpapillon merged commit 36925bf into main Sep 6, 2026
3 checks passed
@bpapillon
bpapillon deleted the sdk-e2e-status-check branch September 6, 2026 05:11
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