Closed GoogleCodeExporter closed 9 years ago
Issue 58 has been merged into this issue.
Original comment by avrah...@gmail.com
on 29 Oct 2014 at 7:41
A simple unit test would be useful.
Original comment by kkrugler...@transpac.com
on 22 Jan 2015 at 2:30
Well, do you really think that is needed a unit test in this case?
If the answer is yes, would be enough to test its compatibility:
@Test
public void testIsSerializable() {
assertTrue(SimpleRobotRules.class.isAssignableFrom(Serializable.class));
}
Thanks and regards,
Josep Miquel
El 22/1/15 15:30, "crawler-commons@googlecode.com"
<crawler-commons@googlecode.com> escribi�:
Original comment by josepmiq...@fav24.com
on 26 Jan 2015 at 12:43
Hi Josep - it's not a big deal, but in writing Cascading code I've been bitten
numerous times by either something not being serializable due to members not
being serializable, or when custom serialization was needed having this not
work properly. So I now always add a test to create & serialize via
ObjectOutputStream to a byte array, then de-serialize and compare. In any case,
I went ahead and added the test, and committed the mod.
Original comment by kkrugler...@transpac.com
on 26 Jan 2015 at 1:17
Committed revision 162
Original comment by kkrugler...@transpac.com
on 26 Jan 2015 at 1:19
Original issue reported on code.google.com by
josepmiq...@fav24.com
on 29 Oct 2014 at 7:05