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

ASF GitHub Bot commented on METRON-1294:
----------------------------------------

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

    https://github.com/apache/metron/pull/827#discussion_r149177390
  
    --- Diff: 
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/SearchControllerIntegrationTest.java
 ---
    @@ -236,17 +236,14 @@ public void test() throws Exception {
                 
.andExpect(jsonPath("$.groupResults[0].groupResults[0].score").value(50));
     
         this.mockMvc.perform(post(searchUrl + 
"/column/metadata").with(httpBasic(user, 
password)).with(csrf()).contentType(MediaType.parseMediaType("application/json;charset=UTF-8")).content("[\"bro\",\"snort\"]"))
    -            .andExpect(status().isOk())
    -            
.andExpect(content().contentType(MediaType.parseMediaType("application/json;charset=UTF-8")))
    -            .andExpect(jsonPath("$.*", hasSize(2)))
    -            
.andExpect(jsonPath("$.bro.common_string_field").value("string"))
    -            
.andExpect(jsonPath("$.bro.common_integer_field").value("integer"))
    -            .andExpect(jsonPath("$.bro.bro_field").value("boolean"))
    -            .andExpect(jsonPath("$.bro.duplicate_field").value("date"))
    -            
.andExpect(jsonPath("$.snort.common_string_field").value("string"))
    -            
.andExpect(jsonPath("$.snort.common_integer_field").value("integer"))
    -            .andExpect(jsonPath("$.snort.snort_field").value("double"))
    -            .andExpect(jsonPath("$.snort.duplicate_field").value("long"));
    +        .andExpect(status().isOk())
    --- End diff --
    
    Yes the `SearchControllerIntegrationTest` does not actually exercise the 
ElasticsearchDAO and I find this confusing as well.  Really all the 
`SearchControllerIntegrationTest` tests is that serialization/deserialization 
happens correctly (which is still valuable).


> IP addresses are not formatted correctly in facet and group results
> -------------------------------------------------------------------
>
>                 Key: METRON-1294
>                 URL: https://issues.apache.org/jira/browse/METRON-1294
>             Project: Metron
>          Issue Type: Bug
>            Reporter: Ryan Merriman
>            Assignee: Ryan Merriman
>
> Fields that are of type IP address are being returned in numerical format 
> whereas they should be returned in string format.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to