[FVM] Recover NFT contract

[turbolent]

Work towards onflow/cadence#3480

Add support for recovering contracts conforming to the pre-Cadence 1.0 NFT standard.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 15.96639% with 100 lines in your changes missing coverage. Please review.

Project coverage is 41.49%. Comparing base (4df1bf3) to head (ab9e300).

Files	Patch %	Lines
fvm/environment/program_recovery.go	6.18%	91 Missing ⚠️
cmd/util/ledger/migrations/migrator_runtime.go	71.42%	2 Missing and 2 partials ⚠️
...til/ledger/migrations/account_storage_migration.go	0.00%	2 Missing ⚠️
cmd/util/ledger/migrations/cadence.go	50.00%	1 Missing ⚠️
...util/ledger/migrations/cadence_values_migration.go	50.00%	1 Missing ⚠️
...til/ledger/migrations/fix_broken_data_migration.go	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6388      +/-   ##
==========================================
- Coverage   41.51%   41.49%   -0.03%     
==========================================
  Files        2013     2013              
  Lines      143518   143577      +59     
==========================================
- Hits        59577    59572       -5     
- Misses      77761    77830      +69     
+ Partials     6180     6175       -5     

Flag	Coverage Δ
unittests	41.49% <15.96%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[joshuahannan]

Is "Recovered Contract" the best name for it? I worry that developers who are unaware of this will be confused by the error messages. Maybe we just need to update the error messages to have more detail, like, "{Contract Name} is no longer functional. A version of the contract has been recovered to allow access to a small subset of functionality. isAvailableToWithdraw is not available in recovered program."

[turbolent]

@joshuahannan Great idea! I tried to improve the error messages in 2844c97

@SupunS PTAL: I realized there was a bug when printing the errors from the panics and opened onflow/cadence#3548. I also refactored the "finalize and merge" of the changes in the migration in 2844c97 (introduces the Commit function) and 1c21cd1 (uses it in more migrations)

[SupunS]

Looks good!

[bluesign]

I haven't check too much, but collection seems missing, I think adding also collection is important (with ownedNFTs)

[turbolent]

@bluesign right, that's planned to be added next