v2.3.0 migration script and upgrade test

contracts/protocol/facets/DisputeResolverHandlerFacet.sol

@@ -641,6 +641,7 @@ contract DisputeResolverHandlerFacet is IBosonAccountEvents, ProtocolBase {
     {
         (exists, disputeResolver, disputeResolverFees) = fetchDisputeResolver(_disputeResolverId);
         if (exists) {
+            disputeResolver.clerk = address(0);
             sellerAllowList = protocolLookups().allowedSellers[_disputeResolverId];
         }
     }

contracts/protocol/facets/ExchangeHandlerFacet.sol

@@ -870,7 +870,6 @@ contract ExchangeHandlerFacet is IBosonExchangeHandler, BuyerBase, DisputeBase {
                             // Token transfer order is descending
                             twinM.tokenId += twinM.supplyAvailable;
                         }
-
                         // ERC-721 style transfer
                         data = abi.encodeWithSignature(
                             "safeTransferFrom(address,address,uint256,bytes)",
@@ -890,7 +889,6 @@ contract ExchangeHandlerFacet is IBosonExchangeHandler, BuyerBase, DisputeBase {
                             ""
                         );
                     }
-
                     // Make call only if there is enough gas and code at address exists.
                     // If not, skip the call and mark the transfer as failed
                     twinM.tokenAddress = twinS.tokenAddress;

contracts/protocol/facets/ProtocolInitializationHandlerFacet.sol

@@ -155,6 +155,7 @@ contract ProtocolInitializationHandlerFacet is IBosonProtocolInitializationHandl
     function initV2_3_0(bytes calldata _initializationData) internal {
         // Current version must be 2.2.1
         require(protocolStatus().version == bytes32("2.2.1"), WRONG_CURRENT_VERSION);
+
         require(protocolCounters().nextTwinId == 1, TWINS_ALREADY_EXIST);
 
         // Decode initialization data

contracts/protocol/facets/SellerHandlerFacet.sol

@@ -496,7 +496,7 @@ contract SellerHandlerFacet is SellerBase {
 
         (exists, sellerId) = getSellerIdByAuthToken(_associatedAuthToken);
         if (exists) {
-            return fetchSeller(sellerId);
+            return fetchSellerWithoutClerk(sellerId);
         }
     }
 

hardhat.config.js

@@ -53,11 +53,12 @@ task(
 task("upgrade-facets", "Upgrade existing facets, add new facets or remove existing facets")
   .addParam("newVersion", "The version of the protocol to upgrade to")
   .addParam("env", "The deployment environment")
+  .addParam("functionNamesToSelector", "JSON list of function names to selectors")
   .addOptionalParam("facetConfig", "JSON list of facets to upgrade")
-  .setAction(async ({ env, facetConfig, newVersion }) => {
+  .setAction(async ({ env, facetConfig, newVersion, functionNamesToSelector }) => {
     const { upgradeFacets } = await lazyImport("./scripts/upgrade-facets.js");
 
-    await upgradeFacets(env, facetConfig, newVersion);
+    await upgradeFacets(env, facetConfig, newVersion, functionNamesToSelector);
   });
 
 task("upgrade-clients", "Upgrade existing clients")
@@ -115,9 +116,9 @@ task("migrate", "Migrates the protocol to a new version")
 
     if (dryRun) {
       const balanceAfter = await getBalance();
-      const etherSpent = balanceBefore.sub(balanceAfter);
+      const etherSpent = balanceBefore - balanceAfter;
 
-      const formatUnits = require("ethers").utils.formatUnits;
+      const formatUnits = require("ethers").formatUnits;
       console.log("Ether spent: ", formatUnits(etherSpent, "ether"));
     }
   });

package.json

@@ -31,7 +31,7 @@
     "tidy:contracts": "solhint --fix contracts/**/*.sol && prettier --write contracts/**",
     "tidy:scripts": "eslint --fix test/** scripts/** '*.js' && prettier --write test/** scripts/** '*.js'",
     "size": "npx hardhat size-contracts",
-    "coverage": "npx hardhat clean && npx hardhat coverage",
+    "coverage": "npx hardhat clean && npx hardhat coverage --testfiles '{test/access/*.js,test/protocol/*.js,test/protocol/clients/*.js}'",
     "deploy-suite:hardhat": "npx hardhat clean && npx hardhat compile && npx hardhat deploy-suite --network hardhat",
     "deploy-suite:local": "npx hardhat clean && npx hardhat compile && npx hardhat deploy-suite --network localhost",
     "deploy-suite:test": "npx hardhat clean && npx hardhat compile && npx hardhat deploy-suite --network test --env test >> logs/test.deploy.contracts.txt",
@@ -52,6 +52,7 @@
     "upgrade-facets:polygon:mumbai-test": "npx hardhat clean && npx hardhat compile && npx hardhat upgrade-facets --network mumbai --env test >> logs/mumbai-test.upgrade.contracts.txt",
     "upgrade-facets:polygon:mumbai-staging": "npx hardhat clean && npx hardhat compile && npx hardhat upgrade-facets --network mumbai --env staging >> logs/mumbai-staging.upgrade.contracts.txt",
     "upgrade-facets:polygon:mainnet": "npx hardhat clean && npx hardhat compile && npx hardhat upgrade-facets --network polygon --env prod >> logs/polygon.upgrade.contracts.txt",
+    "migrate": "npx hardhat clean && npx hardhat compile && npx hardhat migrate >> logs/${npm_config_env}-migrate.contracts.txt",
     "upgrade-clients:local": "npx hardhat clean && npx hardhat compile && npx hardhat upgrade-clients --network localhost --env ''",
     "upgrade-clients:test": "npx hardhat clean && npx hardhat compile && npx hardhat upgrade-clients --network test --env test >> logs/test.upgrade.contracts.txt",
     "upgrade-clients:ethereum:mainnet": "npx hardhat clean && npx hardhat compile && npx hardhat upgrade-clients --network mainnet --env prod >> logs/mainnet.upgrade.contracts.txt",

scripts/domain/Offer.js

@@ -115,6 +115,9 @@ class Offer {
       voided,
       collectionIndex,
     ] = struct;
+    if (!collectionIndex) {
+      collectionIndex = 0;
+    }
 
     return Offer.fromObject({
       id: id.toString(),

scripts/manage-roles.js

@@ -2,7 +2,7 @@ const hre = require("hardhat");
 const { getContractAt, provider } = hre.ethers;
 const network = hre.network.name;
 const { RoleAssignments } = require("./config/role-assignments");
-const { readContracts } = require("./util/utils");
+const { readContracts, listAccounts } = require("./util/utils");
 const environments = require("../environments");
 const Role = require("./domain/Role");
 
@@ -39,7 +39,7 @@ async function main(env) {
   console.log(`⛓  Network: ${hre.network.name}\n📅 ${new Date()}`);
 
   // Get the accounts
-  const accounts = await provider.listAccounts();
+  const accounts = await listAccounts();
   const admin = accounts[0];
   console.log("🔱 Admin account: ", admin ? admin : "not found" && process.exit());
   console.log(divider);

scripts/migrations/dry-run.js

@@ -29,7 +29,7 @@ async function setupDryRun(env) {
 
   await hre.changeNetwork("hardhat");
 
-  env = "upgrade-test";
+  env = `${env}-dry-run`;
 
   const { chainId } = await ethers.provider.getNetwork();
   if (chainId != "31337") process.exit(1); // make sure network is hardhat

scripts/migrations/migrate_2.2.1.js

@@ -41,6 +41,7 @@ async function migrate(env) {
 
     console.log("Compiling old contracts");
     await hre.run("clean");
+
     await hre.run("compile");
 
     const { chainId } = await provider.getNetwork();