remove import console in Script.sol about forge-std HOT 9 CLOSED

Hi @ZeroEkkusu

Sorry, I just found the real problem.

that because forge is not case sensitive, and I import the console file withs "Console.sol" in ContactA.sol, then the error reproduce.

Maybe it’s not a big problem, thanks for your time

from forge-std.

Gm, @tonypig.

There should be no issues with importing/using console and console2. Can you show me an example setup?

The only case I can think of when you would get a DeclarationError is when you try to import console from console2.sol and vice-versa.

from forge-std.

Hi @ZeroEkkusu

My scenario is simple:

two contract:
ContractA.sol and ContractA.t.sol (test)

While import console in ContractA.sol, then forge build/test will throw an DeclarationError

that because forge-std/Test.sol also import the console (through Script.sol)

from forge-std.

@tonypig, like so? I can't reproduce the error.

ContractA.sol

import "forge-std/console.sol";
import "forge-std/console2.sol";

contract ContractA {
    function f() external {
        console.log("gm with console");
        console2.log("gm with console2");
    }
}

ContractA.t.sol

import "forge-std/Test.sol";

import "src/ContractA.sol";

contract ContractATest is Test {

    ContractA cA;

    function setUp() public {
        cA = new ContractA();
    }

    function testExample() public {
        cA.f();
        console.log("test");
        console2.log("test 2");
    }
}

from forge-std.

I think this still could be an issue for users who use Hardhat console in their production contracts, and use Forge Test in their test contracts.

import "hardhat/console.sol";

contract MyContract {}

import "forge-std/Test.sol";
import "contracts/MyContract.sol";

contract MyContractTest is Test {}

Compile it will throw error[2333]: DeclarationError: Identifier already declared.

Is there a way to import the library so that it only affects current file and it's child-contracts, but not sibling contracts in the inheritance chain?

from forge-std.

@ranmocy, import console from forge-std instead of hardhat in your production contracts:

import "forge-std/console.sol";

Another option is to use selective imports to import only MyContract in your test contracts:

import {MyContract} from "contracts/MyContract.sol";

from forge-std.

I also cannot get console2 to work on a particular project that I built from scratch. I've ran multiple repos locally that have had it implemented already, and have configured my repo in the same way as the ones that work.

foundry.toml remappings

remappings = [
    "forge-std/=lib/forge-std/src/",
    "@openzeppelin/=lib/openzeppelin-contracts/",
    "@uniswap/v2-core/=lib/v2-core/",
    "@uniswap/v3-core/=lib/v3-core/",
    "@uniswap/v3-periphery/=lib/v3-periphery/"
]

having trouble importing in both ContractTest.t.sol and ContractScript.s.sol files:

// have tried
import "forge-std/console2.sol";

// as well as
import {console2} from "forge-std/console2.sol";

// as well as
import {Script} from "forge-std/Script.sol";
import {console2} from "forge-std/console2.sol";
import {MyContract} from "../src/MyContract.sol";

getting this error on forge build:

Error (7576): Undeclared identifier. Did you mean "console"?
script/PriceOracle.s.sol:19:9: DeclarationError: Undeclared identifier. Did you mean "console"?
        console2.log("initializing price oracle...");
        ^------^

from forge-std.

I think the issue I was running into had to do with Solidity and my PriceOracle.sol contract being version ^0.7.6

was able to resolve the error using pragma abicoder v2 on the script/test contracts for PriceOracle

from forge-std.

Btw, no need to use console2; it's only there for legacy support. You can just use console.

from forge-std.