-
Notifications
You must be signed in to change notification settings - Fork 75
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
chore: add precheck for receivers account in sendRawTransactionCheck #3310
chore: add precheck for receivers account in sendRawTransactionCheck #3310
Conversation
589773c
to
92c28db
Compare
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.
Great work! Some blocking requests
66403c6
to
7931099
Compare
Signed-off-by: Nadezhda Popova <[email protected]>
…nCheck Signed-off-by: Nadezhda Popova <[email protected]>
…nsactionCheck Signed-off-by: Nadezhda Popova <[email protected]>
Signed-off-by: Nadezhda Popova <[email protected]>
7931099
to
3b85b1a
Compare
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.
Great work! However, additional tests are still needed. Also a batch of unit tests is required in the precheck.spec.ts
file to functionally test the method.
Signed-off-by: Nadezhda Popova <[email protected]>
Signed-off-by: Nadezhda Popova <[email protected]>
Signed-off-by: Nadezhda Popova <[email protected]>
…heck Signed-off-by: Nadezhda Popova <[email protected]>
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.
LGTM, one small nit
…nt precheck Signed-off-by: Nadezhda Popova <[email protected]>
f43a76e
…s account precheck Signed-off-by: Nadezhda Popova <[email protected]>
…eceivers account precheck Signed-off-by: Nadezhda Popova <[email protected]>
2ccf78d
Quality Gate failedFailed conditions See analysis details on SonarQube Cloud Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE |
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.
1 Q other than that everything LGTM! Great work as always 🚀 💪🏼 🙌🏼
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3310 +/- ##
==========================================
- Coverage 84.91% 84.91% -0.01%
==========================================
Files 69 69
Lines 4655 4660 +5
Branches 1042 1044 +2
==========================================
+ Hits 3953 3957 +4
Misses 397 397
- Partials 305 306 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Description:
Added precheck for receiver's account in sendRawTransactionCheck
Fixes #3286