27041931923
53%
master: 62%

Ran 05 Jun 2026 09:51PM UTC

Jobs 1

Files 2

Run time 1min

Badge

Embed ▾

Committed 05 Jun 2026 09:47PM UTC coverage: 52.972%. First build

Build # 27041931923

Build Type

Pull #9205

github

Committed by

brotherlogic

Commit Message

Scrub negative instance IDs from all cache maps on startup

On readRecordCollection, any negative key in the eight cache maps
(InstanceToFolder, InstanceToCategory, InstanceToUpdate,
InstanceToUpdateIn, InstanceToMaster, InstanceToId, InstanceToRecache,
InstanceToLastSalePriceUpdate) is now deleted before the collection is
returned. If any entries were removed the cleaned collection is saved
back to the keystore so the stale data does not reappear on the next
restart.

Previously only InstanceToCategory had a scrub loop. The other seven
maps could still contain legacy negative IDs caused by historical
int32 overflow, which would propagate into query results.

A generic helper negativeKeysToDelete[V any] avoids repeating the
delete-loop boilerplate for each map.

Tests:
- Added TestNegativeIdNotInCacheAfterRead to drive the new behaviour
- Updated TestQueryRecordsWithNegativeId to match: negative IDs saved
  directly into the keystore must not appear in QueryRecords results
  after the next readRecordCollection call.

Pull Request Pull Request #9205: Scrub negative instance IDs from all cache maps on startup

Coverage Stats

704 of 1329 relevant lines covered (52.97%)

0.61 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	27041931923.1	05 Jun 2026 09:51PM UTC	2	52.97	GitHub Action Run

brotherlogic / recordcollection / 27041931923
53%
master: 62%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 27041931923

brotherlogic / recordcollection / 27041931923 53% master: 62%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 27041931923

brotherlogic / recordcollection / 27041931923
53%
master: 62%

README BADGES
x