Last set of comments: * we check for "event ID \> 0" in a lot of places, but what does that mean? We don't actually check it's a *valid* ID, do we? What would a negative ID tell us? Parsing failure? * Small language thing: "Unexisting" -> "Nonexistent" in test names * There's a copy-paste error in a lot of the `fail(...)` messages which all talk about ""Transfer image should fail on 404")"
Nice work! --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/49#issuecomment-33270755
