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

Reimplement scheduler unit tests #321

Closed
philwinder opened this issue Sep 21, 2015 · 2 comments
Closed

Reimplement scheduler unit tests #321

philwinder opened this issue Sep 21, 2015 · 2 comments
Assignees
@mwl
Labels
refactoring
Milestone
0.4.2

Comments

@philwinder
Copy link
Contributor

For a previous version the scheduler unit tests were commented out due to refactoring. Reimplement these tests.

See: https://github.com/mesos/elasticsearch/blob/master/scheduler/src/test/java/org/apache/mesos/elasticsearch/scheduler/ElasticsearchSchedulerTest.java
@philwinder philwinder added this to the 0.4.2 milestone Sep 21, 2015
@mwl mwl self-assigned this Sep 23, 2015
@mwl
Copy link
Contributor

mwl commented Sep 23, 2015

Will be solved by #325

@mwl mwl mentioned this issue Sep 23, 2015
Merged
@mwl
Copy link
Contributor

mwl commented Sep 23, 2015

Fixed by #325.

@mwl mwl closed this as completed Sep 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwl mwl

Labels
refactoring
Projects
None yet
Milestone
0.4.2
Development

No branches or pull requests
2 participants
@mwl @philwinder