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

ASF GitHub Bot commented on STREAMS-458:
----------------------------------------

Github user smarthi commented on a diff in the pull request:

    https://github.com/apache/incubator-streams/pull/331#discussion_r89666745
  
    --- Diff: 
streams-contrib/streams-provider-google/google-gplus/src/test/java/org/apache/streams/gplus/test/processors/GooglePlusTypeConverterIT.java
 ---
    @@ -27,18 +27,19 @@
     import com.fasterxml.jackson.databind.ObjectMapper;
     import com.fasterxml.jackson.databind.module.SimpleModule;
     import com.google.api.services.plus.model.Person;
    +import com.google.gplus.processor.GooglePlusTypeConverter;
     import com.google.gplus.serializer.util.GPlusActivityDeserializer;
     import com.google.gplus.serializer.util.GPlusPersonDeserializer;
     import com.google.gplus.serializer.util.GooglePlusActivityUtil;
    -
     import org.apache.commons.lang.StringUtils;
    --- End diff --
    
    use lang3


> GPlusTypeConverter fails processing recent plus#person without a gender
> -----------------------------------------------------------------------
>
>                 Key: STREAMS-458
>                 URL: https://issues.apache.org/jira/browse/STREAMS-458
>             Project: Streams
>          Issue Type: Bug
>            Reporter: Steve Blackmon
>            Assignee: Steve Blackmon
>             Fix For: 0.5
>
>   Original Estimate: 4h
>  Remaining Estimate: 4h
>
> Running GPlusTypeConverterTest against the output of GPlusUserDataProviderIT:
> java.lang.NullPointerException: null
>       at 
> com.google.gplus.serializer.util.GPlusPersonDeserializer.deserialize(GPlusPersonDeserializer.java:65)
> Rework to ensure GPlusTypeConverter doesn't crash on valid input.



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

Reply via email to