----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57241/#review167678 -----------------------------------------------------------
webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java Line 273 (original) <https://reviews.apache.org/r/57241/#comment239596> How do we know that the exception was thrown ? Consider adding an assert in the catch block to ensure that exception was indeed caught. webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java Lines 401 (patched) <https://reviews.apache.org/r/57241/#comment239587> Sending time as long works, please check. webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java Line 418 (original), 419 (patched) <https://reviews.apache.org/r/57241/#comment239597> This was working on my system with some tweaks, let's discuss. webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java Line 761 (original), 762 (patched) <https://reviews.apache.org/r/57241/#comment239588> This test also works for me webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java Line 261 (original), 263 (patched) <https://reviews.apache.org/r/57241/#comment239589> Since this data provider is only providing one value, is better to refactor the test method to use only null and not rely on the data provider. webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java Lines 659 (patched) <https://reviews.apache.org/r/57241/#comment239598> Is this the correct fix for partial update scenario ? Looks like a full-update is being called here. Please review. - Apoorv Naik On March 2, 2017, 11:16 a.m., Sarath Subramanian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57241/ > ----------------------------------------------------------- > > (Updated March 2, 2017, 11:16 a.m.) > > > Review request for atlas. > > > Bugs: ATLAS-1621 > https://issues.apache.org/jira/browse/ATLAS-1621 > > > Repository: atlas > > > Description > ------- > > Fix for the failed ITs: > Failed tests: > EntityJerseyResourceIT.testAddNullPropertyValue:423 Expected > AtlasServiceException > EntityJerseyResourceIT.testAddReferenceProperty:449->addProperty:467 » > AtlasService > EntityJerseyResourceIT.testEntityDeduping:264 » AtlasService Metadata > service ... > EntityJerseyResourceIT.testPartialUpdate:739 » AtlasService Metadata service > A... > EntityV2JerseyResourceIT.testAddNullPropertyValue:335->createHiveTable:443->createHiveTable:452 > expected object to not be null > EntityV2JerseyResourceIT.testEntityDeduping:197 » AtlasService Metadata > servic... > org.apache.atlas.web.resources.EntityV2JerseyResourceIT.testEntityInvalidValue(org.apache.atlas.web.resources.EntityV2JerseyResourceIT) > Run 1: PASS > Run 2: EntityV2JerseyResourceIT.testEntityInvalidValue:272 expected:<null> > but was:<AtlasEntityHeader{guid='d8cd8064-25cf-490d-aa88-5a93392677de', > status=ACTIVE, displayText=null, classificationNames=[],, > AtlasStruct{typeName='hive_db_v2', attributes=[name:LMPplE6apZ]}}> > EntityV2JerseyResourceIT.testSubmitEntity:103->createDBAndTable:437->createHiveTable:443->createHiveTable:452 > expected object to not be null > EntityV2JerseyResourceIT.testSubmitEntityWithBadDateFormat:296 expected > object to not be null > MetadataDiscoveryJerseyResourceIT.setUp:64->BaseResourceIT.createInstance:234 > » AtlasService > Tests run: 135, Failures: 10, Errors: 0, Skipped: 17 > > > Diffs > ----- > > > repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java > 61657a1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java > a3d951d > webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java > 6d902db > webapp/src/test/java/org/apache/atlas/web/resources/BaseResourceIT.java > 0bcabb7 > > webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java > b37dbca > > webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java > e0e32a8 > > > Diff: https://reviews.apache.org/r/57241/diff/1/ > > > Testing > ------- > > mvn clean verify -pl webapp -DskipUTs - all ITs tests suceeded > mvn clean install (in progress) > > > Thanks, > > Sarath Subramanian > >