-
Notifications
You must be signed in to change notification settings - Fork 22
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 faucet limitation #571
Conversation
849af39
to
4af84d0
Compare
@@ -9,6 +9,8 @@ defmodule ArchethicWeb.FaucetControllerTest do | |||
PubSub | |||
} | |||
|
|||
alias Archethic.BeaconChain.ReplicationAttestation |
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.
alias Archethic.{
BeaconChain.ReplicationAttestation,??
Crypto,
P2P,
P2P.Node,
PubSub
}
alias ArchethicWeb.TransactionSubscriber | ||
alias ArchethicWeb.FaucetRateLimiter |
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.
alias ArchethicWeb.{TransactionSubscriber,FaucetRateLimiter}??
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 👍 works for me.
@@ -50,7 +53,12 @@ defmodule ArchethicWeb.FaucetControllerTest do | |||
|
|||
describe "create_transfer/2" do | |||
test "should show success flash with tx URL on valid transaction", %{conn: conn} do |
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.
MockCrypto.get_storage_nonce/0 ?? Maybe we can have this , for more deterministic test?
Call to TransactionSubscriber.get_max_confirmations requires storage nonce?I am not sure,
it may require more Mocks.
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.
This is the point to avoid blocking address , so if any address is not used it should not be blocked like the next test
Description
Fix the limitation of the faucet
Type of change
How Has This Been Tested?
Request funds more than 3 times on the same genesis address should be prevented
Checklist: