Skip to content

fix(e2e): update pr-mod test for signer-less dot mod#172

Closed
EnderOfWorlds007 wants to merge 1 commit into
mainfrom
fix/mod-signer-less-test
Closed

fix(e2e): update pr-mod test for signer-less dot mod#172
EnderOfWorlds007 wants to merge 1 commit into
mainfrom
fix/mod-signer-less-test

Conversation

@EnderOfWorlds007
Copy link
Copy Markdown
Collaborator

Summary

  • Since b228817 (feat: migrate to paseo-next-v2, 2026-05-13), dot mod uses getReadOnlyRegistryContract and never calls resolveSigner--suri is a deprecated no-op
  • The pr-mod CI cell was failing because the test "exits non-zero with signer suggestion when no signer available" asserted output.toContain("No signer available") — output that no longer exists
  • Updated test to reflect actual signer-less behavior: dot mod proceeds directly to the registry lookup, and an unknown domain produces "not found in registry"

Test plan

  • CI pr-mod cell passes

Since b228817 (feat: migrate to paseo-next-v2), dot mod uses
getReadOnlyRegistryContract and never calls resolveSigner. The
--suri flag is a deprecated no-op. The test "exits non-zero with
signer suggestion when no signer available" was asserting on
"No signer available" output that no longer exists.

Update the test to reflect the actual signer-less behavior:
dot mod proceeds directly to the registry lookup, and an unknown
domain produces "not found in registry" (not a signer error).
@github-actions
Copy link
Copy Markdown
Contributor

Dev build ready — try this branch:

curl -fsSL https://raw.githubusercontent.com/paritytech/playground-cli/main/install.sh | VERSION=dev/fix/mod-signer-less-test bash

@github-actions
Copy link
Copy Markdown
Contributor

E2E Test Pass · ❌ FAIL

Tag: e2e-ci-pr · Branch: fix/mod-signer-less-test · Commit: 9a50224 · Run logs

Cell Result Time
pr-preflight ❌ FAIL 2m45s
pr-init-session ✅ PASS 1m40s
pr-deploy-foundry ❌ FAIL 2m13s
pr-deploy-cdm ❌ FAIL 3m29s
pr-deploy-frontend ❌ FAIL 2m42s
pr-install ✅ PASS 0m49s
pr-mod ✅ PASS 1m16s
${{ matrix.cell }} ⏭️ SKIP 0m-1s
${{ matrix.cell }} ⏭️ SKIP 0m-1s
❌ Failed tests (1)
  • e2e/cli/deploy.test.ts › dot deploy — cdm (requires Paseo + IPFS) > CDM deploy completes end-to-end
    CDM deploy failed: Checking availability of e2ecdm00.dot… ✔ e2ecdm00.dot is available — Requires Proof of Personhood (Lite). Will be set up automatically. Deploying e2ecdm00.dot Signer Dev signer (no phone taps for upload) (5DtoCFYY2gqk65CkrJAc44LCPZDvrxQCkCvTc2AoX28bCu65) Build rebuild first Build dir /home/runner/_work/playground-cli/playground-cli/e2e/cli/fixtures/projects/rust-cdm/dist Publish Playground + your apps Moddable no Phone approvals required: 1 1. Publish to Playground registry ▸ contracts… ▸ build… > e2e-rust-cdm@0.0.1 build /home/runner/_work/playground-cli/playground-cli/e2e/cli/fixtures/projects/rust-cdm > mkdir -p dist && echo '

    rust cdm app

    ' > dist/index.html ✔ build ▸ storage-and-dotns… [+0.0s] ============================================================ [+0.0s] DEPLOYING TO TESTNET v0.7.20 [+0.0s] ============================================================ [+0.0s] Environment: Paseo Next v2 [+0.0s] Domain: e2ecdm00.dot [+0.0s] Tag: e2e-cli-ci-pr [+0.0s] Build dir: /home/runner/_work/playground-cli/playground-cli/e2e/cli/fixtures/projects/rust-cdm/dist [+0.0s] Runner: parity-default-96tvj-runner-mvq8l (self-hosted) [+0.0s] ============================================================ [+0.0s] Preflight [+0.0s] ============================================================ [+0.0s] SS58 Address: 5DtoCFYY2gqk65CkrJAc44LCPZDvrxQCkCvTc2AoX28bCu65 [+1.0s] H160 Address: 0x1fb8da400137178e872e3a9ff99f004e4d7966cf [+1.0s] Account: auto-mapped [+1.1s] DotNS: e2ecdm00.dot classifies as ProofOfPersonhoodLite ✖ storage-and-dotns: This environment does not allow DotNS self-attestation (Preview/Paseo Next v2 POP_RULES owner-gated setUserPopStatus). e2ecdm00.dot requires ProofOfPersonhoodLite, but this signer is NoStatus. Use a NoStatus-compatible label (base length >= 9 with exactly two trailing digits, e.g. e2epoolns01.dot) or ask the environment operator to grant the signer the required PoP status. ✖ This environment does not allow DotNS self-attestation (Preview/Paseo Next v2 POP_RULES owner-gated setUserPopStatus). e2ecdm00.dot requires ProofOfPersonhoodLite, but this signer is NoStatus. Use a NoStatus-compatible label (base length >= 9 with exactly two trailing digits, e.g. e2epoolns01.dot) or ask the environment operator to grant the signer the required PoP status.: expected 1 to be +0 // Object.is equality

Sentry traces: view spans for this run

@EnderOfWorlds007
Copy link
Copy Markdown
Collaborator Author

Closing in favor of #171 which now includes both the domain rename and the mod test fix (cherry-picked from this branch).

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