6de29c1cd2
* Fix flakey sytest 'Local device key changes get to remote servers' * Debug logs * Remove internal/test and use /test only Remove a lot of ancient code too. * Use FederationRoomserverAPI in more places * Use more interfaces in federationapi; begin adding regression test * Linting * Add regression test * Unbreak tests * ALL THE LOGS * Fix a race condition which could cause events to not be sent to servers If a new room event which rewrites state arrives, we remove all joined hosts then re-calculate them. This wasn't done in a transaction so for a brief period we would have no joined hosts. During this interim, key change events which arrive would not be sent to destination servers. This would sporadically fail on sytest. * Unbreak new tests * Linting
92 lines
3 KiB
Go
92 lines
3 KiB
Go
package tables_test
|
|
|
|
import (
|
|
"context"
|
|
"database/sql"
|
|
"fmt"
|
|
"testing"
|
|
|
|
"github.com/matrix-org/dendrite/internal/sqlutil"
|
|
"github.com/matrix-org/dendrite/setup/config"
|
|
"github.com/matrix-org/dendrite/syncapi/storage/postgres"
|
|
"github.com/matrix-org/dendrite/syncapi/storage/sqlite3"
|
|
"github.com/matrix-org/dendrite/syncapi/storage/tables"
|
|
"github.com/matrix-org/dendrite/syncapi/types"
|
|
"github.com/matrix-org/dendrite/test"
|
|
)
|
|
|
|
func newTopologyTable(t *testing.T, dbType test.DBType) (tables.Topology, *sql.DB, func()) {
|
|
t.Helper()
|
|
connStr, close := test.PrepareDBConnectionString(t, dbType)
|
|
db, err := sqlutil.Open(&config.DatabaseOptions{
|
|
ConnectionString: config.DataSource(connStr),
|
|
}, sqlutil.NewExclusiveWriter())
|
|
if err != nil {
|
|
t.Fatalf("failed to open db: %s", err)
|
|
}
|
|
|
|
var tab tables.Topology
|
|
switch dbType {
|
|
case test.DBTypePostgres:
|
|
tab, err = postgres.NewPostgresTopologyTable(db)
|
|
case test.DBTypeSQLite:
|
|
tab, err = sqlite3.NewSqliteTopologyTable(db)
|
|
}
|
|
if err != nil {
|
|
t.Fatalf("failed to make new table: %s", err)
|
|
}
|
|
return tab, db, close
|
|
}
|
|
|
|
func TestTopologyTable(t *testing.T) {
|
|
ctx := context.Background()
|
|
alice := test.NewUser(t)
|
|
room := test.NewRoom(t, alice)
|
|
test.WithAllDatabases(t, func(t *testing.T, dbType test.DBType) {
|
|
tab, db, close := newTopologyTable(t, dbType)
|
|
defer close()
|
|
events := room.Events()
|
|
err := sqlutil.WithTransaction(db, func(txn *sql.Tx) error {
|
|
var highestPos types.StreamPosition
|
|
for i, ev := range events {
|
|
topoPos, err := tab.InsertEventInTopology(ctx, txn, ev, types.StreamPosition(i))
|
|
if err != nil {
|
|
return fmt.Errorf("failed to InsertEventInTopology: %s", err)
|
|
}
|
|
// topo pos = depth, depth starts at 1, hence 1+i
|
|
if topoPos != types.StreamPosition(1+i) {
|
|
return fmt.Errorf("got topo pos %d want %d", topoPos, 1+i)
|
|
}
|
|
highestPos = topoPos + 1
|
|
}
|
|
// check ordering works without limit
|
|
eventIDs, err := tab.SelectEventIDsInRange(ctx, txn, room.ID, 0, highestPos, highestPos, 100, true)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to SelectEventIDsInRange: %s", err)
|
|
}
|
|
test.AssertEventIDsEqual(t, eventIDs, events[:])
|
|
eventIDs, err = tab.SelectEventIDsInRange(ctx, txn, room.ID, 0, highestPos, highestPos, 100, false)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to SelectEventIDsInRange: %s", err)
|
|
}
|
|
test.AssertEventIDsEqual(t, eventIDs, test.Reversed(events[:]))
|
|
// check ordering works with limit
|
|
eventIDs, err = tab.SelectEventIDsInRange(ctx, txn, room.ID, 0, highestPos, highestPos, 3, true)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to SelectEventIDsInRange: %s", err)
|
|
}
|
|
test.AssertEventIDsEqual(t, eventIDs, events[:3])
|
|
eventIDs, err = tab.SelectEventIDsInRange(ctx, txn, room.ID, 0, highestPos, highestPos, 3, false)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to SelectEventIDsInRange: %s", err)
|
|
}
|
|
test.AssertEventIDsEqual(t, eventIDs, test.Reversed(events[len(events)-3:]))
|
|
|
|
return nil
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("err: %s", err)
|
|
}
|
|
})
|
|
}
|