fix: Maintain indexes across schema versions

[AndrewSisley]

Relevant issue(s)

Resolves #3365

Description

Completes the fix started in #3366 - I missed a test case there and only tested with docs created after the patch. This adds a new test, testing with docs created before the patch, and then updates all index related references from col.ID to col.RootID.

Sorry about the bother.

[shahzadlone]

Good catch! LGTM

[nasdf]

LGTM

[codecov]

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 78.48%. Comparing base (628e03d) to head (72d1c46).
Report is 1 commits behind head on develop.

Files with missing lines	Patch %	Lines
internal/db/collection_index.go	66.67%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3367      +/-   ##
===========================================
+ Coverage    78.30%   78.48%   +0.18%     
===========================================
  Files          392      392              
  Lines        35688    35688              
===========================================
+ Hits         27944    28008      +64     
+ Misses        6096     6049      -47     
+ Partials      1648     1631      -17     

Flag	Coverage Δ
all-tests	78.48% <85.71%> (+0.18%)	⬆️

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

Files with missing lines	Coverage Δ
internal/db/fetcher/indexer_iterators.go	85.01% <100.00%> (ø)
internal/db/index.go	78.63% <100.00%> (ø)
internal/db/collection_index.go	87.50% <66.67%> (ø)

... and 16 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 628e03d...72d1c46. Read the comment docs.