Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

segregate test files in a3p proposals #10853

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

segregate test files in a3p proposals #10853

wants to merge 7 commits into from

Conversation

turadg
Copy link
Member

@turadg turadg commented Jan 16, 2025

refs: Agoric/agoric-3-proposals#115

Description

A3P is getting this feature,

This PR takes advantage of it for a3p-integration tests.

Draft until it uses a new release of @agoric/synthetic-chain. Meanwhile the test refactoring should pass with the existing version.

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

Upgrade Considerations

Copy link

cloudflare-workers-and-pages bot commented Jan 16, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: dcfcb0b
Status:🚫  Deploy failed.

View logs

@turadg turadg added the force:integration Force integration tests to run on PR label Jan 16, 2025
@turadg turadg force-pushed the 115-a3p-tests branch 2 times, most recently from ea695b6 to d6940ba Compare January 16, 2025 19:37
To override 8.x in synthetic-chain that seems to be incompatible with SES:

#51 20.98 TypeError#1: Invalid property descriptor. Cannot both specify accessors and a value or writable attribute, [object Object]
#51 20.98     at InertConstructor.defineProperty (<anonymous>)
#51 20.98     at changeToString (file:///usr/src/proposals/n:upgrade-next/node_modules/mimic-fn/index.js:57:9)
#51 20.98     at mimicFunction (file:///usr/src/proposals/n:upgrade-next/node_modules/mimic-fn/index.js:68:2)
#51 20.98     at onetime (file:///usr/src/proposals/n:upgrade-next/node_modules/onetime/index.js:27:2)
#51 20.98     at execa (file:///usr/src/proposals/n:upgrade-next/node_modules/@agoric/synthetic-chain/node_modules/execa/index.js:160:28)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant