Skip to content

API integration test#63

Merged
J12934 merged 20 commits intodevelopfrom
feature/integration-test
Feb 13, 2019
Merged

API integration test#63
J12934 merged 20 commits intodevelopfrom
feature/integration-test

Conversation

@J12934
Copy link
Copy Markdown
Member

@J12934 J12934 commented Feb 1, 2019

Added API tests.

The tests start the docker-compose stack and will then start securityTest and assert that the detected findings match the expectations.

Notes

  • the tests currently only run against the develop tag. The build uses a build matrix for the docker tag and can be expended for more tags by commenting out this line:
    # - DEFAULT_TAG=latest
  • The tests run on branch and pr's to this repo and will also be run once a day (via a travis chron)
  • Running docker-compose up will now wait for the engine healthcheck to complete before starting the scanner. This makes waiting for the engine to start up trivial, but unfortunatly also meant that i had to downgrade the docker-compose file version to 2.1, as 3 doesn't support conditionals in depends_on anymore.
  • The tests are written in js using jest as the test runner.

@J12934 J12934 requested a review from rfelber February 1, 2019 17:01
@J12934 J12934 added the enhancement New feature or request label Feb 1, 2019
@J12934 J12934 added this to the 1.0.0 milestone Feb 1, 2019
@J12934 J12934 merged commit 7e453d7 into develop Feb 13, 2019
@J12934 J12934 deleted the feature/integration-test branch February 13, 2019 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant