Skip to content

Commit

Permalink
Remove PK from .env, can use alternate method
Browse files Browse the repository at this point in the history
  • Loading branch information
azflin committed Aug 23, 2022
1 parent 7f09a6e commit 544fe03
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
1 change: 0 additions & 1 deletion .env.example
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
FOUNDRY_RPC_URL=https://mainnet.infura.io/v3/123456789
PRIVATE_KEY=77404248268428598442225233697536483359126021352749990200965399598120042358261
10 changes: 6 additions & 4 deletions test/executors/UniswapV3Executor.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -234,13 +234,15 @@ contract UniswapV3ExecutorIntegrationTest is Test, PermitSignature {
address weth = 0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2;
address usdc = 0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48;
address swapRouter02 = 0x68b3465833fb72A70ecDF485E0e4C7bD8665Fc45;
// .env's PRIVATE_KEY corresponds to this address
address maker = 0x3580F45Cd5DB00221A999ae2925122ACcccDf254;
address maker;
uint256 makerPrivateKey;
UniswapV3Executor uniswapV3Executor;
PermitPost permitPost;
DutchLimitOrderReactor dloReactor;

function setUp() public {
makerPrivateKey = 0x12341234;
maker = vm.addr(makerPrivateKey);
vm.createSelectFork(vm.envString("FOUNDRY_RPC_URL"), 15327550);
uniswapV3Executor = new UniswapV3Executor(swapRouter02);
permitPost = new PermitPost();
Expand Down Expand Up @@ -273,7 +275,7 @@ contract UniswapV3ExecutorIntegrationTest is Test, PermitSignature {
order: order,
sig: getPermitSignature(
vm,
vm.envUint("PRIVATE_KEY"),
makerPrivateKey,
address(permitPost),
Permit({
token: address(weth),
Expand Down Expand Up @@ -316,7 +318,7 @@ contract UniswapV3ExecutorIntegrationTest is Test, PermitSignature {
order: order,
sig: getPermitSignature(
vm,
vm.envUint("PRIVATE_KEY"),
makerPrivateKey,
address(permitPost),
Permit({
token: address(weth),
Expand Down

0 comments on commit 544fe03

Please sign in to comment.