thepraandClaude Opus 5.5
d2f0c7a14e
Infrastructure/Data/Indexes runs at every start, after the migrations: unique on Post/DmPost ObjectURI, ForeignAvatar ActorURI, the Follower triple, RootToAvatar, RootUser UserName and ReservedName; plain indexes on the lookups the services actually make (PublicKeyId, author and group post listings, ParticipantsKey, the delivery queue). Migration _001 runs first and removes the duplicates the races could already have left (keeping the newest actor row, the oldest post, the accepted follower), then fills ReservedName from every avatar and group. ReservedName is one username space for personas, groups and the instance: a name is reserved by an insert the unique index arbitrates, before the avatar or group is saved, so two simultaneous sign-ups cannot both get it. A short list of names (admin, support, abuse, postmaster, ...) is never available. EntityMaps.Warm touches every entity's collection one at a time before anything else runs. MongoDB.Entities maps the Entity base class on first touch, and two types mapped at once throw "An item with the same key has already been added" and stay broken for the life of the process; the parallel test run hit it, and the delivery worker racing a request could have too. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012CzABvBkbcFqoHdmi8b9WB
63 lines
2.7 KiB
C#
63 lines
2.7 KiB
C#
using MongoDB.Driver;
|
|
using MongoDB.Entities;
|
|
|
|
using PrivaPub.Federation.Actors;
|
|
using PrivaPub.Infrastructure.Data;
|
|
using PrivaPub.Infrastructure.Data.Migrations;
|
|
using PrivaPub.Models;
|
|
using PrivaPub.Models.Federation;
|
|
using PrivaPub.Models.User;
|
|
using PrivaPub.StaticServices;
|
|
using PrivaPub.Tests.Support;
|
|
|
|
namespace PrivaPub.Tests.Infrastructure
|
|
{
|
|
[Trait("Category", "Integration")]
|
|
public class IndexTests
|
|
{
|
|
[Fact]
|
|
public async Task Duplicates_are_removed_and_the_unique_indexes_then_hold()
|
|
{
|
|
Assert.SkipUnless(MongoFixture.Enabled, MongoFixture.Skip);
|
|
var token = TestContext.Current.CancellationToken;
|
|
var actorUri = $"https://r.example/users/{Guid.NewGuid():N}";
|
|
var localId = Guid.NewGuid().ToString("N")[..24];
|
|
await DB.Default.SaveAsync(new[]
|
|
{
|
|
new ForeignAvatar { ActorURI = actorUri, Name = "old", UpdatedAt = DateTime.UtcNow.AddDays(-2) },
|
|
new ForeignAvatar { ActorURI = actorUri, Name = "new", UpdatedAt = DateTime.UtcNow }
|
|
}, token);
|
|
await DB.Default.SaveAsync(new[]
|
|
{
|
|
new Follower { LocalActorId = localId, ActorURI = actorUri, IsAccepted = false },
|
|
new Follower { LocalActorId = localId, ActorURI = actorUri, IsAccepted = true }
|
|
}, token);
|
|
|
|
await new _001_deduplicate_before_unique_indexes().UpgradeAsync();
|
|
await Indexes.Create(token);
|
|
|
|
Assert.Equal("new", (await DB.Default.Find<ForeignAvatar>().Match(a => a.ActorURI == actorUri).ExecuteSingleAsync(token)).Name);
|
|
Assert.True((await DB.Default.Find<Follower>().Match(f => f.LocalActorId == localId).ExecuteSingleAsync(token)).IsAccepted);
|
|
var duplicate = await Assert.ThrowsAsync<MongoWriteException>(async () =>
|
|
await DB.Default.SaveAsync(new ForeignAvatar { ActorURI = actorUri }, token));
|
|
Assert.Equal(ServerErrorCategory.DuplicateKey, duplicate.WriteError.Category);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task A_username_is_reserved_once_across_personas_and_groups()
|
|
{
|
|
Assert.SkipUnless(MongoFixture.Enabled, MongoFixture.Skip);
|
|
var token = TestContext.Current.CancellationToken;
|
|
await Indexes.Create(token);
|
|
var local = new LocalActorService(new DbEntities(), new StaticOptions<AppConfiguration>(new AppConfiguration { BackendBaseAddress = "https://privapub.test" }));
|
|
var name = $"n{Guid.NewGuid():N}"[..20];
|
|
|
|
Assert.True(await local.TryReserveUserName(name, LocalActorKind.Person, "a", token));
|
|
Assert.False(await local.TryReserveUserName(name.ToUpperInvariant(), LocalActorKind.Group, "b", token));
|
|
Assert.True(await local.IsUserNameTaken(name, token));
|
|
Assert.False(await local.TryReserveUserName("admin", LocalActorKind.Person, "c", token));
|
|
Assert.False(await local.TryReserveUserName(LocalActorService.InstanceUserName, LocalActorKind.Person, "c", token));
|
|
}
|
|
}
|
|
}
|