[ https://issues.apache.org/jira/browse/ATLAS-300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15003602#comment-15003602 ]
Shwetha G S commented on ATLAS-300: ----------------------------------- 1. notificationConsumer.reset(); - add to before test method 2. helper methods to make API calls - use AtlasClient instead 3. Please use 4 spaces for formatting. Can you fix EntityNotificationImplTest, EntityNotificationIT and any other classes from last patch with 2 spaces 4. Can you modify testAddTrait to use super types and also test getAllTraits() For the updated patch, can you add it to review board. Thanks > Need additional integration test coverage for entity notifications > ------------------------------------------------------------------ > > Key: ATLAS-300 > URL: https://issues.apache.org/jira/browse/ATLAS-300 > Project: Atlas > Issue Type: Task > Reporter: Tom Beerbower > Assignee: Tom Beerbower > Attachments: ATLAS-300.patch > > > Add tests for other operations - entity update, trait add/delete. -- This message was sent by Atlassian JIRA (v6.3.4#6332)