> @@ -69,7 +69,7 @@ public void testExtractArchive() throws Exception {
>                                                       
> .extractArchive(containerName, payload, "tar.gz");
>           assertEquals(extractResponse.getCreated(), OBJECT_COUNT);
>           assertTrue(extractResponse.getErrors().isEmpty());
> -         
> assertEquals(api.getContainerApi(regionId).get(containerName).getObjectCount(),
>  OBJECT_COUNT);
> +         
> assertEquals(api.getContainerApi(regionId).get(containerName).getObjectCount(),
>  Long.valueOf(OBJECT_COUNT));

I´m not completely sure about what you mean. At the very beginning I went for 
the easy way and kept the object count as long, and make it 0 when null. nacx 
was probably right saying that would not be correct so I made it Long. 
In the tests, sometimes constant OBJECT_COUNT is compared with 
extractResponse.getCreated() , and therefore is a long, and sometimes it 
compared with getObjectCount() and therefore we must convert it to Long. 
I wonder if your concern is about object count long vs Long , or about how we 
deal with constants in tests, as they sometimes are compared with long and with 
Long. 
Thanks again.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/923/files#r54422544

Reply via email to