Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix & cleanup integration test #264

Merged
merged 5 commits into from
Oct 16, 2018
Merged

Conversation

errordeveloper
Copy link
Contributor

Description

Fix & cleanup integration test

Checklist

  • Code compiles correctly (i.e make build)
  • Added tests that cover your change (if possible)
  • All tests passing (i.e. make test)
  • Added/modified documentation as required (such as the README)
  • Added yourself to the humans.txt file

errordeveloper and others added 2 commits October 16, 2018 08:50
- use single Describe block to avoid ordering issues
- rename variables and functions to match overall style
@errordeveloper errordeveloper force-pushed the improve-integration-test branch 2 times, most recently from ea2bdb5 to 263661e Compare October 16, 2018 08:17
@errordeveloper errordeveloper force-pushed the improve-integration-test branch 3 times, most recently from 5141295 to a0c7e2e Compare October 16, 2018 10:46
- cosmetic style changes and refactoring
- get rid of nested directory
- remove dead code
- unexport matcher struct
- use canonical tag key
- update suite name
@errordeveloper errordeveloper force-pushed the improve-integration-test branch from a0c7e2e to 36cde0a Compare October 16, 2018 11:42
@errordeveloper errordeveloper changed the title WIP: Fix & cleanup integration test Fix & cleanup integration test Oct 16, 2018
Copy link

@Lazyshot Lazyshot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@errordeveloper errordeveloper merged commit d435854 into master Oct 16, 2018
@errordeveloper errordeveloper deleted the improve-integration-test branch October 16, 2018 13:15
Copy link
Contributor

@richardcase richardcase left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants