-
Notifications
You must be signed in to change notification settings - Fork 71
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
Hwrdtm/datil dev support #510
Conversation
For now, I'm going to delegate this task to someone else and they can pick up on the PR where I've left off. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good outside of the local-test
changes
// this._testnet = await this._shivaClient.startTestnetManager(); | ||
// // wait for the testnet to be active before we start the tests. | ||
// await this._testnet.pollTestnetForActive(); | ||
// await this._testnet.getTestnetConfig(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have a solution for this. could you revert this change?
@@ -380,7 +383,7 @@ export class TinnyEnvironment { | |||
* ==================================== | |||
*/ | |||
if (this.network === LIT_TESTNET.LOCALCHAIN) { | |||
const networkContext = this._testnet.ContractContext; | |||
// const networkContext = this._testnet.ContractContext; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
@@ -101,7 +103,7 @@ export class TinnyPerson { | |||
// * ==================================== | |||
// */ | |||
if (this.envConfig.network === LIT_TESTNET.LOCALCHAIN) { | |||
const networkContext = this.envConfig.contractContext; | |||
// const networkContext = this.envConfig.contractContext; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
@@ -15,9 +15,10 @@ import { TinnyEnvironment } from 'local-tests/setup/tinny-environment'; | |||
* - Then: The claim operation should successfully return signatures, derived key IDs, and validate the existence and structure of claimed results. | |||
* * | |||
* Test Commands: | |||
* ❌ Not supported in Cayenne |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we remove this comment?
@@ -9,9 +9,10 @@ import { log } from '@lit-protocol/misc'; | |||
|
|||
/** | |||
* Test Commands: | |||
* ❌ Not supported in Cayenne |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we remove this comment?
@@ -13,9 +13,10 @@ import { TinnyEnvironment } from 'local-tests/setup/tinny-environment'; | |||
* - Then: The claim operation should successfully return signatures, derived key IDs, and validate the existence and structure of claimed results. | |||
* * | |||
* Test Commands: | |||
* ❌ Not supported in Cayenne |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
@@ -7,9 +7,10 @@ import { TinnyEnvironment } from 'local-tests/setup/tinny-environment'; | |||
|
|||
/** | |||
* Test Commands: | |||
* ❌ Not supported on cayenne |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
@@ -5,9 +5,10 @@ import { TinnyEnvironment } from 'local-tests/setup/tinny-environment'; | |||
|
|||
/** | |||
* Test Commands: | |||
* ❌ Not supported on cayenne |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
Description
Work towards LIT-3404.
This PR:
datil-dev
as a Lit network, both in the core packages and also in thelocal-tests
(Tinny). More importantly, this PR is INCOMPLETEType of change
How Has This Been Tested?
Run
DEBUG=true NETWORK=localchain LIT_RPC_URL=https://vesuvius-rpc.litprotocol.com yarn test:local
to run against Datil-dev.Checklist:
Datil Dev Network Context
networkContext.datilDev.json