9 commits

Author	SHA1	Message	Date
Till Faelligen	f4bc86adaf	Rever change	2022-05-18 13:16:43 +02:00
Till Faelligen	cda49e8d96	Check if createdb exists	2022-05-18 13:15:15 +02:00
Till Faelligen	09b6e2e877	Use fatalError in createLocalDB	2022-05-18 12:40:18 +02:00
Till Faelligen	d7e79a7d02	Really SKIP_NODB	2022-05-18 12:32:44 +02:00
kegsay	5c37f165ae	Errors from createdb are non-fatal (#2420) ... As they are expected if the database already exists.	2022-05-05 10:53:52 +01:00
Neil Alexander	e01d1e1f5b	Skip tests that require a database if we can't connect to one (#2413) ... * Skip tests that require a database if we can't connect to one * Add `DENDRITE_SKIP_DB_TESTS` environment variable to bring @kegsay joy * Call it `DENDRITE_TEST_SKIP_NODB` intead	2022-05-03 17:38:54 +01:00
kegsay	ea92f80c12	Add database namespacing for unit tests (#2340) ... * Add database namespacing for unit tests Background: Running `go test ./...` will run tests in different packages concurrently. This can be stopped or limited by using `-p 1` (no concurrency). We want concurrency, but this causes problems when running Postgres DBs in CI. The problem is that, in CI, we have 1x postgres server exposing 1x postgres DB, which we wipe clean at the end of each test via `defer close()`. When tests run concurrently, calls to `close()` will delete data/tables which other tests are currently using, causing havoc. Fix this by: - Creating a database per package. - Namespacing the database name by a hash of the current working directory (the directory containing those `_test.go` files) This is exactly what SQLite does, quite unintentionally, via the use of `file:dendrite_test.db`, which dumps the file into the current working directory which is the package running the tests, hence deleting the file is safe when running concurrently. * Linting * Don't create the database in a txn * dupe db is not an error	2022-04-11 10:23:01 +01:00
kegsay	6d25bd6ca5	syncapi: add more tests; fix more bugs (#2338) ... * syncapi: add more tests; fix more bugs bugfixes: - The postgres impl of TopologyTable.SelectEventIDsInRange did not use the provided txn - The postgres impl of EventsTable.SelectEvents did not preserve the ordering of the input event IDs in the output events slice - The sqlite impl of EventsTable.SelectEvents did not use a bulk `IN ($1)` query. Added tests: - `TestGetEventsInRangeWithTopologyToken` - `TestOutputRoomEventsTable` - `TestTopologyTable` * -p 1 for now	2022-04-08 17:53:24 +01:00
kegsay	7499147550	Add test infrastructure code for dendrite unit/integ tests (#2331) ... * Add test infrastructure code for dendrite unit/integ tests Start re-enabling some syncapi storage tests in the process. * Linting * Add postgres service to unit tests * dendrite not syncv3 * Skip test which doesn't work * Linting * Add `jetstream.PrepareForTests` Co-authored-by: Neil Alexander <neilalexander@users.noreply.github.com>	2022-04-08 10:12:30 +01:00