From 9c4eab2d59ee6a73858c8cc87dacef1cde821462 Mon Sep 17 00:00:00 2001 From: David Meister Date: Sat, 15 Aug 2026 12:58:26 +0000 Subject: [PATCH] test: drop duplicate MockAddressRevertingFactory import MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `test/src/lib/LibRainDeploy.t.sol` imported `MockAddressRevertingFactory` from the same path at lines 9 and 12, byte-identically. solc tolerates a repeated identical declaration, so it compiled and nothing failed, which is why it survived — the cost was a reader having to diff two lines to conclude they are the same. The symbol is used, so this is a redundant duplicate line rather than a dead import. A repo-wide scan of imported symbols per file finds this as the only case. Closes #71 Co-Authored-By: Claude Opus 5 (1M context) --- test/src/lib/LibRainDeploy.t.sol | 1 - 1 file changed, 1 deletion(-) diff --git a/test/src/lib/LibRainDeploy.t.sol b/test/src/lib/LibRainDeploy.t.sol index 23d302c..c00a21b 100644 --- a/test/src/lib/LibRainDeploy.t.sol +++ b/test/src/lib/LibRainDeploy.t.sol @@ -9,7 +9,6 @@ import {AddressRegistry, ADDRESS_REGISTRY_ROOT} from "../../../src/concrete/Addr import {MockAddressRevertingFactory} from "../../concrete/MockAddressRevertingFactory.sol"; import {MockResolvedOwner} from "../../concrete/MockResolvedOwner.sol"; import {MockDirtyWordOwner} from "../../concrete/MockDirtyWordOwner.sol"; -import {MockAddressRevertingFactory} from "../../concrete/MockAddressRevertingFactory.sol"; import {MockDeployable} from "../../concrete/MockDeployable.sol"; import {MockDeployableV2} from "../../concrete/MockDeployableV2.sol"; import {MockReverter} from "../../concrete/MockReverter.sol";