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.

Sign up for GitHub

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

Jump to bottom

Need tests for sync command. #27

Closed
bwbeach opened this issue Jan 4, 2016 · 3 comments
Closed

Need tests for sync command. #27

bwbeach opened this issue Jan 4, 2016 · 3 comments

Comments

@bwbeach
Copy link
Contributor

bwbeach commented Jan 4, 2016

There aren't any tests yet for the new sync code.
@bwbeach
Copy link
Contributor Author

bwbeach commented Jan 6, 2016

I'll be working on this today.

@ppolewicz
Copy link
Collaborator

Shouldn't refactoring (as per your TODO comments) come first? Or do you mean integration tests of console command?

@bwbeach
Copy link
Contributor Author

bwbeach commented Jan 6, 2016

I mean the integration tests. I'd like to have those in place before refactoring.

@bwbeach bwbeach mentioned this issue Jan 6, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bwbeach @ppolewicz