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
fix: typo on cache_test eth_call #9462
fix: typo on cache_test eth_call #9462
Conversation
WalkthroughWalkthroughThis update involves a modification in a test case within the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Interesting that the test didn't fail because of this |
@tynes it is because we return
|
Do you think returning nil,nil is an antipattern? It should be nil,ErrNotFound. imo we should never return nil,nil. It enforces two possibilities over 3 which is easier to reason about as a dev |
Personally I think so. cc @felipe-op will this break assumptions around how this cache is used? |
The expected semantic here is return a The reason why the test didn't break is because those methods are expected to have the same behaviour, which is never cache the result. The test is checking this exactly same behaviour in a set of methods. |
309c9bf
chore(op-dispute-mon): refactor output validation into a separate component feat(op-dispute-mon): game forecasting implementation minus testing fix(op-dispute-mon): bad merge chore(op-dispute-mon): add more tests
chore(op-dispute-mon): refactor output validation into a separate component feat(op-dispute-mon): game forecasting implementation minus testing fix(op-dispute-mon): bad merge chore(op-dispute-mon): add more tests
chore(op-dispute-mon): refactor output validation into a separate component feat(op-dispute-mon): game forecasting implementation minus testing fix(op-dispute-mon): bad merge chore(op-dispute-mon): add more tests
chore(op-dispute-mon): refactor output validation into a separate component feat(op-dispute-mon): game forecasting implementation minus testing fix(op-dispute-mon): bad merge chore(op-dispute-mon): add more tests
* fix: typo on cache_test eth_call (#9462) chore(op-dispute-mon): refactor output validation into a separate component feat(op-dispute-mon): game forecasting implementation minus testing fix(op-dispute-mon): bad merge chore(op-dispute-mon): add more tests * fix(op-dispute-mon): testing var and use updated log filtering * fix(op-dispute-mon): functional resolver * fix(op-dispute-mon): simply bidirectional game tree construction * fix(op-dispute-mon): remove left bond counter * fix(op-dispute-mon): construct resolver bidirectional tree --------- Co-authored-by: Oak <me+git@droak.sh>
Description
The method being called in
cache_test.go
foreth_call
waseth_gasPrice
. This PR fixes it