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

Basic tests #8

Merged
merged 5 commits into from Aug 9, 2019

Conversation

@ashcrow
Copy link
Contributor

commented Aug 9, 2019

Requires #7 (will rebase once merges)

  • Updated .gitignore
  • Initial tests
  • Fixes for items found by tests
  • Makefile for running flake8 and tests
  • Travis config for running each check (flake8 only at the moment) and test.

@ashcrow ashcrow force-pushed the ashcrow:basic-tests branch from da3be64 to 2089c12 Aug 9, 2019

ashcrow added some commits Aug 9, 2019

.gitignore: Ignore python generated files
Signed-off-by: Steve Milner <smilner@redhat.com>
server/test_app.py: Add initial tests
Signed-off-by: Steve Milner <smilner@redhat.com>
server/app.py: Fix items found by tests
Signed-off-by: Steve Milner <smilner@redhat.com>
Makefile: Add Makefile
Signed-off-by: Steve Milner <smilner@redhat.com>
.travis.yml: Add travis file for simple CI
Signed-off-by: Steve Milner <smilner@redhat.com>

@ashcrow ashcrow force-pushed the ashcrow:basic-tests branch from 2089c12 to ec6fd55 Aug 9, 2019

@ashcrow

This comment has been minimized.

Copy link
Contributor Author

commented Aug 9, 2019

Rebased

@ashcrow ashcrow changed the title WIP: Basic tests Basic tests Aug 9, 2019

@zonggen

This comment has been minimized.

Copy link
Owner

commented Aug 9, 2019

Merging! 👍

@zonggen zonggen merged commit ca02aa9 into zonggen:master Aug 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.