-
Notifications
You must be signed in to change notification settings - Fork 6
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
Default NFT Balance #42
Conversation
run: npm run compile | ||
- name: Run Unit Tests | ||
run: | | ||
npm run test |
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.
👍
@@ -61,7 +61,7 @@ export interface INetworkProvider { | |||
/** | |||
* Fetches the state of a transaction. | |||
*/ | |||
getTransaction(txHash: string): Promise<TransactionOnNetwork>; | |||
getTransaction(txHash: string, withProcessStatus?: boolean): Promise<TransactionOnNetwork>; |
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.
I forgot that we have this interface in the package. In my opinion it should be removed (breaking change) - in a separate PR.
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.
Will keep it in mind, this will be changed once the specs for sdk-network-providers
are out.
assert.equal(apiResponse.RoundAtEpochStart, proxyResponse.RoundAtEpochStart); | ||
assert.equal(apiResponse.RoundsPerEpoch, proxyResponse.RoundsPerEpoch); | ||
// done this way because the nonces may change until both requests are executed | ||
assert.approximately(apiResponse.HighestFinalNonce, proxyResponse.HighestFinalNonce, 1); |
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.
Interesting :)
NFT Token balance set as
1
if nobalance
field provided.Also, added the
tests-devnet
to run as a github action and fixed the tests.