Add tests upgrade to v2.2.0

[anajuliabit]

This PR adds integration tests to insure protocol integrity on upgrade from 2.1.0 to 2.2.0

Breaking changes

• PR: Allows DR creation without the need for activation by protocol admin by @anajuliabit in Allows DR creation without the need for activation by protocol admin #466
  • DR can be activated on creation.
• PR: Allowlist metatx methods by @zajck in Allowlist metatx methods #461
  • Meta transaction should work with allowlisted function
  • Meta transaction should fail when function name is not allowlisted
  • functionPointer positions should change
• PR: Raise and Escalate Dispute Orchestration by @cliffhall in Raise and Escalate Dispute Orchestration #489:
  • Test if facets address pointed to selectors from OrchestrationHandlerFacet were replaced for facets OrchestrationHandlerFacet1 and OrchestrationHandlerFacet2
• PR: New voucher name and symbol by @zajck in New voucher name and symbol #477
  • Test if new vouchers are created with new name and symbol
• PR: Rename operator to assistant Rename operator to assistant #556
  • Operator has been renamed to assistant

New methods

• PR: Allowlist metatx methods by @zajck in Allowlist metatx methods #461
  • setAllowlistedFunctions
  • isFunctionAllowlisted(bytes32)
  • isFunctionAllowlisted(string)
  • Meta transaction should fail when function name is not allowlisted
• PR: Add new facet ProtocolInitializationFacet by @anajuliabit in Add new facet ProtocolInitializationFacet #475
  • getVersion
• PR: Init v2_2_0 by @zajck in Init v2_2_0 #492
  • Should call initV2_2_0
• PR: Raise and Escalate Dispute Orchestration by @cliffhall in Raise and Escalate Dispute Orchestration #489
  • raiseAndEscalateDispute
• PR: Protocol support preminted voucher by @zajck in Protocol support preminted voucher #490
  • setMaxPremintedVouchers
  • getMaxPremintedVouchers
  • commitToPreMintedOffer
  • reserveRange
  • createPremintedOfferWithCondition
  • createPremintedOfferAddToGroup
  • createPremintedOfferAndTwinWithBundle
  • createPremintedOfferWithConditionAndTwinAndBundle
  • createSellerAndPremintedOffer
  • createSellerAndPremintedOfferWithCondition
  • createSellerAndPremintedOfferAndTwinWithBundle
  • createSellerAndPremintedOfferWithConditionAndTwinAndBundle
  • createOfferWithCondition
  • createOfferAddToGroup
  • createOfferAndTwinWithBundle
  • createOfferWithConditionAndTwinAndBundle
  • createSellerAndOffer
  • createSellerAndOfferWithCondition
  • createSellerAndOfferAndTwinWithBundle
  • createSellerAndOfferWithConditionAndTwinAndBundle
• PR: Allow vouchers to be preMinted to voucher contract Allow vouchers to be preMinted to voucher contract #563
  • reserveRange to contract

Bug fixes

• PR: Fix bug on createTwinInternal by @anajuliabit in Fix bug on createTwinInternal #469
  • Should ignore twin id set by seller and use nextAccountId on twin creation
• PR:Fix bug on tokenURI method on BosonVoucher Fix bug on tokenURI method on BosonVoucher #557
  • Should return the correct tokenURI when token is pre-minted

Notes

• Besides the missing checks above, we still have to investigate why upgrade-facets script is logging undefined: 0xa7ef6a7c when removing interfaces for DR, and why is removing DR interface

[zajck]

2. Investigate why upgrade-facets script is logging undefined: 0xa7ef6a7c when removing interfaces for DR, and why is removing DR interface

0xa7ef6a7c is activateDisputeResolver sigHash, which was removed in 2.2.0, so it's ok that's reported. It says undefined since upgrade script is not aware of actual function names that were previously registered on diamond. All it knows is that something was registered which is not in interface anymore and therefore needs to be removed.

[anajuliabit]

I can't approve but LGTM

[mischat]

Looks good, only a minor comment, which we should address in a subsequent version perhaps

[mischat]

I guess this should start with an underscore, _exchangeId ... but i don't want to block on this ... Can we changed in the next version

[mischat]

@anajuliabit