-
Notifications
You must be signed in to change notification settings - Fork 1
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
Data model , proof and request #11
Conversation
* call. | ||
*/ | ||
|
||
async createProof(inputVariables?: createProof_mutationMutationVariables) { |
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.
it should not be optional i guess
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.
that is already optional , as u can see ?
}); | ||
|
||
expect(createDataModel?.title).toEqual('Test'); | ||
}, 40000); |
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.
the timeout should be a variable which is constant
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.
nopes , every test cases takes differenet amount of time , so we can't store into a variable.
const { issuersByDataModel } = await api.dataModel.getIssuersByDataModel( | ||
process.env.DATAMODEL_ID as string, | ||
); | ||
|
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.
expect missing?
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.
lemme fix
await api.dataModel.getTotalofIssuersByDataModel( | ||
process.env.DATAMODEL_ID as string, | ||
); | ||
|
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.
expect missing?
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.
lemme fix
const { issuersByDataModelCount } = | ||
await api.dataModel.getIssuersByDataModelCount( | ||
process.env.DATAMODEL_ID as string, | ||
); |
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.
expect missing?
|
||
describe('proofs related test', () => { | ||
it('create proof', async () => { | ||
const {} = await api.proof.createProof({}); |
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.
expect missing?
return await this.sdk.createDataModel_mutation({ | ||
input: createModelInput, | ||
}); | ||
} catch (error: any) { |
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.
remove :any part from all catch blocks just to maintain consistency
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.
cool
* @returns The `getDataModelsCount` function is returning the result of the `dataModelsCount_query` | ||
* method call, which is a promise. | ||
*/ | ||
|
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.
run pnpm format missing lines in test files as well
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.
cool
with test cases