Skip to content
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 warnings in VRFCoordinatorV2Mock #9151

Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
12 changes: 6 additions & 6 deletions contracts/src/v0.8/mocks/VRFCoordinatorV2Mock.sol
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ contract VRFCoordinatorV2Mock is VRFCoordinatorV2Interface {

function getConfig()
external
view
pure
returns (
uint16 minimumRequestConfirmations,
uint32 maxGasLimit,
Expand All @@ -250,7 +250,7 @@ contract VRFCoordinatorV2Mock is VRFCoordinatorV2Interface {

function getFeeConfig()
external
view
pure
returns (
uint32 fulfillmentFlatFeeLinkPPMTier1,
uint32 fulfillmentFlatFeeLinkPPMTier2,
Expand All @@ -276,19 +276,19 @@ contract VRFCoordinatorV2Mock is VRFCoordinatorV2Interface {
);
}

function getFallbackWeiPerUnitLink() external view returns (int256) {
function getFallbackWeiPerUnitLink() external pure returns (int256) {
return 4000000000000000; // 0.004 Ether
}

function requestSubscriptionOwnerTransfer(uint64 _subId, address _newOwner) external pure override {
function requestSubscriptionOwnerTransfer(uint64, address) external pure override {
revert("not implemented");
}

function acceptSubscriptionOwnerTransfer(uint64 _subId) external pure override {
function acceptSubscriptionOwnerTransfer(uint64) external pure override {
revert("not implemented");
}

function pendingRequestExists(uint64 subId) public view override returns (bool) {
function pendingRequestExists(uint64) public pure override returns (bool) {
revert("not implemented");
}
}