Github user merrimanr commented on the issue:

    https://github.com/apache/metron/pull/1140
  
    That's fine with me.  Just wanted to point out that this test is still 
deficient because the pcap topology part really isn't tested that well and the 
only coverage we have right now is that it sets up other tests.  It also 
doesn't match the way we integration test other topologies or DAOs in Metron.  
For example, our indexing integration test and search DAO integration tests are 
completely separate from each other.
    
    Maybe that's something we improve in the future.  I imagine it will make 
sense to do it when (if) the pcap topology is updated.  +1


---

Reply via email to