jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrea Turli <notificati...@github.com>
Subject Re: [jclouds] replace ExpectTests with MockWebServerTests (#727)
Date Wed, 13 May 2015 19:07:27 GMT
@nacx thanks for the review.

I think we agreed on not writing tests for pre-defined fallbacks as those fallbacks are already
tested in isolation. At this regard, what's the rule-of-thumb for `list` methods: is it always
needed to have something like `@Fallback(EmptySetOnNotFoundOr404.class)` to avoid NPE or it
depends on the API?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/727#issuecomment-101779494
Mime
View raw message