[jira] [Commented] (IGNITE-321) CacheWriter methods should throw CacheWriterException

["Sergey Evdokimov (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/IGNITE-321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14342366#comment-14342366 ] 

Sergey Evdokimov commented on IGNITE-321:
-----------------------------------------

Following tests still broken:
CacheWriterTest.shouldWriteThoughUsingPutAll_partialSuccess	
CacheWriterTest.shouldWriteThroughRemoveAllSpecific_partialSuccess	
CacheWriterTest.shouldWriteThroughRemoveAll_partialSuccess

> CacheWriter methods should throw CacheWriterException
> -----------------------------------------------------
>
>                 Key: IGNITE-321
>                 URL: https://issues.apache.org/jira/browse/IGNITE-321
>             Project: Ignite
>          Issue Type: Bug
>          Components: cache
>            Reporter: Dmitriy Setrakyan
>            Assignee: Sergey Evdokimov
>            Priority: Blocker
>              Labels: wait_for_review
>             Fix For: sprint-2
>
>         Attachments: ignite-321.patch
>
>
> Tests where CacheWriter's methods write(All)/delete(All) throw an exception expect that Cache's methods put(All)/remove(All) throw CacheWriterException in this case.
> List of tests which expect CacheWriterException if CacheWriter throws an exception:
> {code}
> CacheWriterTest.shouldNotPutWhenWriteThroughFails
> CacheWriterTest.shouldWriteThoughUsingPutAll_partialSuccess
> CacheWriterTest.shouldNotRemoveWhenWriteThroughFails
> CacheWriterTest.shouldWriteThroughRemoveAll_partialSuccess
> CacheWriterTest.shouldWriteThroughRemoveAllSpecific_partialSuccess
> CacheWriterTest.shouldWrapWriterExceptions
> {code}
> We should make sure that these tests pass by catching IgniteException, calling hasCause(CacheWriterException.class), and if true, then throwing the CacheWriterException.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)