Github user moshebla commented on a diff in the pull request: https://github.com/apache/lucene-solr/pull/410#discussion_r201222488 --- Diff: solr/core/src/test/org/apache/solr/update/TestNestedUpdateProcessor.java --- @@ -120,25 +122,41 @@ public void before() throws Exception { @Test public void testDeeplyNestedURPGrandChild() throws Exception { + final String[] tests = { + "/response/docs/[0]/id=='" + grandChildId + "'", + "/response/docs/[0]/" + IndexSchema.NEST_PATH_FIELD_NAME + "=='children#0/grandChild#'" + }; indexSampleData(jDoc); - assertJQ(req("q", IndexSchema.NEST_PATH_FIELD_NAME + ":*" + PATH_SEP_CHAR + "grandChild" + NUM_SEP_CHAR + "*" + NUM_SEP_CHAR, + assertJQ(req("q", IndexSchema.NEST_PATH_FIELD_NAME + ":*" + PATH_SEP_CHAR + "grandChild" + NUM_SEP_CHAR + "*", "fl","*", "sort","id desc", "wt","json"), - "/response/docs/[0]/id=='" + grandChildId + "'"); + tests); } @Test public void testDeeplyNestedURPChildren() throws Exception { --- End diff -- I added a new sanity unit test for this URP
--- --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org