Remove AcreBitcoinDepositorHarness contract

We no longer need to define the test contract to expose internal
functions so we remove it also from the deployment scripts.

core/contracts/test/AcreBitcoinDepositorHarness.sol

@@ -9,17 +9,6 @@ import {IBridgeTypes} from "@keep-network/tbtc-v2/contracts/integrator/IBridge.s
 
 import {TestERC20} from "./TestERC20.sol";
 
-/// @dev A test contract to expose internal function from AcreBitcoinDepositor contract.
-///      This solution follows Foundry recommendation:
-///      https://book.getfoundry.sh/tutorials/best-practices#internal-functions
-contract AcreBitcoinDepositorHarness is AcreBitcoinDepositor {
-    function exposed_finalizeBridging(
-        uint256 depositKey
-    ) external returns (uint256 amountToStake, address staker) {
-        return finalizeBridging(depositKey);
-    }
-}
-
 /// @dev A test contract to stub tBTC Bridge contract.
 contract BridgeStub is MockBridge {}
 

core/deploy/03_deploy_acre_bitcoin_depositor.ts

@@ -15,10 +15,6 @@ const func: DeployFunction = async (hre: HardhatRuntimeEnvironment) => {
   const [, acreBitcoinDepositorDeployment] = await helpers.upgrades.deployProxy(
     "AcreBitcoinDepositor",
     {
-      contractName:
-        process.env.HARDHAT_TEST === "true"
-          ? "AcreBitcoinDepositorHarness"
-          : "AcreBitcoinDepositor",
       factoryOpts: {
         signer: deployer,
       },

core/package.json

@@ -26,7 +26,7 @@
     "lint:sol:fix": "solhint 'contracts/**/*.sol' --fix && prettier --write 'contracts/**/*.sol'",
     "lint:config": "prettier --check '**/*.@(json)'",
     "lint:config:fix": "prettier --write '**/*.@(json)'",
-    "test": "HARDHAT_TEST=true hardhat test"
+    "test": "hardhat test"
   },
   "devDependencies": {
     "@keep-network/hardhat-helpers": "^0.7.1",