aeolusproject / conductor

Aeolus Conductor
http://www.aeolusproject.org/
Apache License 2.0
32 stars 27 forks source link

Deployables create improve test coverage #357

Closed hacxman closed 11 years ago

hacxman commented 11 years ago

some (2) of those fails though, I believe it is due to current implementation being incorrect 'fixes' #322

n1zyy commented 11 years ago

@hacxman - This looks like it needs to be rebased against master.

hacxman commented 11 years ago

rebased & fixed whitespace error

martinpovolny commented 11 years ago

@hacxman so to merge this we would probably like to fix the implementation in the same pull-request

or do you have different idea?

martinpovolny commented 11 years ago

nice tests, code finally fixed ;-)

ack and merging