GameCult/CultLib
Watch
9
0
Fork
You've already forked CultLib
 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

Caching tests: a custom store cannot mark an entry it does not hold (red)

Browse source 
Finding cultlib-gaps:finding:cut-expect-persisted-schema.s2.markwritten-unscoped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Meta Vens 2026-10-21 21:58:54 +02:00
commit 2c8b44ac1d
1 changed files with 22 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

22
tests/GameCult.Caching.Tests/ConditionalCommitTests.cs
View file

@ -162,6 +162,26 @@ namespace GameCult.Caching.Tests
Assert.That(reader.Get<ConditionalDriftNarrow>(DriftKey)!.Value, Is.EqualTo(2));
}
// A store marks only what it holds: another cache's copy of the same record is refused and left as read.
[Test]
public void ACustomStoreCannotMarkAnEntryItDoesNotHold()
{
SeedDrift(directory: false);
using var owner = new CultCache(NarrowDrift);
var ownerStore = new WholeFileStore(PathOf("drift.cc"));
owner.AddBackingStore(ownerStore);
ownerStore.PullAll();
using var other = new CultCache(NarrowDrift);
var otherStore = new WholeFileStore(PathOf("drift.cc"));
other.AddBackingStore(otherStore);
otherStore.PullAll();
var foreign = owner.GetStored(DriftKey)!;
var persisted = foreign.PersistedSchemaId;
Assert.Throws<ArgumentException>(() => otherStore.Mark(foreign), "a store marked another cache's entry");
Assert.That(foreign.PersistedSchemaId, Is.EqualTo(persisted), "a refused mark changed the entry");
}
[Test]
public void OwnUnconditionalCommitKeepsDriftedExpectHolding()
{
@ -1062,6 +1082,8 @@ namespace GameCult.Caching.Tests
public override void Delete(CultStoredDocument entry) => throw new NotSupportedException();
public void Mark(CultStoredDocument entry) => MarkWritten(entry);
public override void PushAll() => Held(Write);
public override CultCommitOutcome CommitBatch(CultCommitRequest request, bool wait) => Held(() =>