mirror of
https://github.com/hasura/graphql-engine.git
synced 2024-12-16 01:44:03 +03:00
80c977da85
We currently have a fairly intricate way of running our PostgreSQL and MSSQL integration tests (not the API tests). By splitting them out, we can simplify this a lot. Most prominently, we can rely on Cabal to be our argument parser instead of writing our own. We can also simplify how they're run in CI. They are currently (weirdly) run alongside the Python integration tests. This breaks them out into their own jobs for better visibility, and to avoid conflating the two. The changes are as follows: - The "unit" tests that rely on a running PostgreSQL database are extracted out to a new test directory so they can be run separately. - Most of the `Main` module comes with them. - We now refer to these as "integration" tests instead. - Likewise for the "unit" tests that rely on a running MS SQL Server database. These are a little simpler and we can use `hspec-discover`, with a `SpecHook` to extract the connection string from an environment variable. - Henceforth, these are the MS SQL Server integration tests. - New CI jobs have been added for each of these. - There wasn't actually a job for the MS SQL Server integration tests. It's pretty amazing they still run well. - The "haskell-tests" CI job, which used to run the PostgreSQL integration tests, has been removed. - The makefiles and contributing guide have been updated to run these. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6912 GitOrigin-RevId: 67bbe2941bba31793f63d04a9a693779d4463ee1
43 lines
1.1 KiB
Makefile
43 lines
1.1 KiB
Makefile
# Use the Azure SQL Edge image instead of the SQL Server image on arm64.
|
|
# The latter doesn't work yet.
|
|
ifeq ($(shell uname -m),arm64)
|
|
MSSQL_IMAGE=mcr.microsoft.com/azure-sql-edge
|
|
else
|
|
MSSQL_IMAGE= # allow the Docker Compose file to set the image
|
|
endif
|
|
|
|
export MSSQL_IMAGE
|
|
|
|
TEST_MSSQL_CONNECTION_STRING = Driver={ODBC Driver 18 for SQL Server};Server=localhost,65003;Uid=sa;Pwd=Password!;Encrypt=optional
|
|
TEST_POSTGRES_URL = postgres://hasura:hasura@localhost:65002/hasura
|
|
|
|
define stop_after
|
|
@ echo $1 >&2
|
|
@ $1 || EXIT_STATUS=$$?; \
|
|
if [[ -z "$${EXIT_STATUS:-}" ]]; then \
|
|
$(MAKE) stop-everything; \
|
|
else \
|
|
$(MAKE) stop-everything; \
|
|
exit $$EXIT_STATUS; \
|
|
fi
|
|
endef
|
|
|
|
.PHONY: build-backends
|
|
## build-backends: build Docker images for any backends that need them
|
|
build-backends:
|
|
docker compose build
|
|
|
|
.PHONY: start-backends
|
|
## start-backends: start all known backends in Docker and wait for them to be ready
|
|
start-backends: build-backends
|
|
docker compose up --detach --wait
|
|
|
|
.PHONY: stop-everything
|
|
## stop-everything: tear down test databases
|
|
stop-everything:
|
|
docker compose down --volumes
|
|
|
|
.PHONY: remove-tix-file
|
|
remove-tix-file:
|
|
@ find . -name '*.tix' -delete
|