[
https://issues.apache.org/jira/browse/BEAM-6005?focusedWorklogId=163509&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-163509
]
ASF GitHub Bot logged work on BEAM-6005:
----------------------------------------
Author: ASF GitHub Bot
Created on: 07/Nov/18 17:25
Start Date: 07/Nov/18 17:25
Worklog Time Spent: 10m
Work Description: supercclank commented on a change in pull request
#6923: [BEAM-6005] PCollectionCustomCoderTest updates to fix test to actually
function.
URL: https://github.com/apache/beam/pull/6923#discussion_r231600012
##########
File path:
sdks/java/core/src/test/java/org/apache/beam/sdk/coders/PCollectionCustomCoderTest.java
##########
@@ -148,104 +161,179 @@ private void throwIfPresent(String exceptionClassName)
throws IOException {
@Test
@Category(NeedsRunner.class)
public void testDecodingIOException() throws Exception {
- thrown.expect(Exception.class);
- thrown.expectCause(instanceOf(IOException.class));
Pipeline p =
- runPipelineWith(new CustomTestCoder(IO_EXCEPTION, null, null, null,
EXCEPTION_MESSAGE));
+ pipelineWith(new CustomTestCoder(IO_EXCEPTION, null, null, null,
EXCEPTION_MESSAGE));
+ thrown.expect(Exception.class);
Review comment:
Is this able to be changed to "IOException.class"
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 163509)
> PCollectionCustomCoderTest passes spuriously.
> ---------------------------------------------
>
> Key: BEAM-6005
> URL: https://issues.apache.org/jira/browse/BEAM-6005
> Project: Beam
> Issue Type: Bug
> Components: testing
> Reporter: Jason Kuster
> Assignee: Jason Kuster
> Priority: Major
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> The test assertions trigger before the coder is used on the actual runner.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
