Skip to content
Snippets Groups Projects

Added (again) database integration tests

Open Karsten Prehn requested to merge feature/integration-tests into main
1 unresolved thread

This tries to add tests of the database-module (integration tests) again. They have previously been removed due to issues with the gitlab-runner when waiting for the docker-in-docker (dind) service to start. Further explanation below.

\approve @ds @marius

(approval only if the issue can be fixed on gitlab)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
39 39 # Run the unit tests
40 40 - pytest tests/unit -s -v
41
42 tests::integration:
43 stage: test
44 interruptible: true
45 image: docker:20.10.16
46 tags:
47 - dind
48 variables:
49 DOCKER_HOST: tcp://docker:2375
50 DOCKER_DRIVER: overlay2
51 DOCKER_TLS_CERTDIR: ""
52 services:
53 # We rely on the docker-in-docker service
54 - name: docker:20.10.16-dind
  • I see there is now a new error message. Can you add an alias like this:

      services:
        # We rely on the docker-in-docker service
        - name: docker:20.10.16-dind
          alias: docker
  • Author Contributor

    Done.

    Locally, that new error didn't prevent the job from succeeding when run with the --docker-privileged flag.

  • Please register or sign in to reply
  • Karsten Prehn added 1 commit

    added 1 commit

    • 8d6865b7 - Added (again) database integration tests

    Compare with previous version

  • Please register or sign in to reply
    Loading