I filed https://issues.apache.org/jira/browse/GEODE-1620 for the unrecognized options issue.
On Tue, Jul 5, 2016 at 10:11 AM, Jinmei Liao <[email protected]> wrote: > Let's change the test first and then create a jira ticket to have gfsh fail > on unrecognized options. Thanks! > > On Tue, Jul 5, 2016 at 9:57 AM, Kirk Lund <[email protected]> wrote: > > > GfshHistoryJUnitTest is now failing after my commit on Friday. > > > > The test expects unrecognized options to cause the command to fail. > > > > Replacing our forked version of jopt-simple with a released version has > > caused us to lose the ability to fail any command that contains > > unrecognized arguments or unrecognized options. The latter could be > > restored but it currently causes incorrect behavior with auto-complete. > > > > I propose changing the test to match the new behavior. My opinion is that > > this is a better compromise that breaking auto-complete or spending > further > > unknown time devising additional workarounds. This is up for discussion > so > > please weigh in. > > > > -Kirk > > > > On Tuesday, July 5, 2016, Anthony Baker <[email protected]> wrote: > > > > > Thanks Jinmei! Can you share with the list the results of your > > discussion? > > > > > > “If it doesn’t happen on the mailing list, it didn’t happen.” [1] > > > —The Apache Way > > > > > > Anthony > > > > > > [1] https://community.apache.org/newbiefaq.html > > > > > > > On Jul 5, 2016, at 8:50 AM, Jinmei Liao <[email protected] > > > <javascript:;>> wrote: > > > > > > > > It has something to do with the 7/1 checkin on gfshParser. I'll talk > to > > > the > > > > team about if they want to revert the changes or fix the test if the > > > change > > > > is expected. > > > > > > > > On Tue, Jul 5, 2016 at 8:28 AM, Jinmei Liao <[email protected] > > > <javascript:;>> wrote: > > > > > > > >> I'll take a look. > > > >> > > > >> On Tue, Jul 5, 2016 at 8:24 AM, Anthony Baker <[email protected] > > > <javascript:;>> wrote: > > > >> > > > >>> This test is failing pretty regularly. Any volunteers to take a > > look? > > > >>> > > > >>> Anthony > > > >>> > > > >>> On Jul 5, 2016, at 7:20 AM, Apache Jenkins Server < > > > >>> [email protected] <javascript:;>> wrote: > > > >>> > > > >>> geode-core:integrationTest > > > >>> > > > >>> > > > > com.gemstone.gemfire.management.internal.cli.shell.GfshHistoryJUnitTest > > > > >>> testHistoryFileIsCreated FAILED > > > >>> org.junit.ComparisonFailure: expected:<[]connect --fake-param...> > > but > > > >>> was:<[// [failed] ]connect --fake-param...> > > > >>> at org.junit.Assert.assertEquals(Assert.java:115) > > > >>> at org.junit.Assert.assertEquals(Assert.java:144) > > > >>> at > > > >>> > > > > > > com.gemstone.gemfire.management.internal.cli.shell.GfshHistoryJUnitTest.testHistoryFileIsCreated(GfshHistoryJUnitTest.java:75) > > > >>> > > > >>> > > > > com.gemstone.gemfire.management.internal.cli.shell.GfshHistoryJUnitTest > > > > >>> testHistoryFileDoesNotContainPasswords FAILED > > > >>> org.junit.ComparisonFailure: expected:<[// [failed] ]connect > > > >>> --password=*...> but was:<[]connect --password=*...> > > > >>> at org.junit.Assert.assertEquals(Assert.java:115) > > > >>> at org.junit.Assert.assertEquals(Assert.java:144) > > > >>> at > > > >>> > > > > > > com.gemstone.gemfire.management.internal.cli.shell.GfshHistoryJUnitTest.testHistoryFileDoesNotContainPasswords(GfshHistoryJUnitTest.java:84) > > > >>> > > > >>> > > > >>> > > > >> > > > >> > > > >> -- > > > >> Cheers > > > >> > > > >> Jinmei > > > >> > > > > > > > > > > > > > > > > -- > > > > Cheers > > > > > > > > Jinmei > > > > > > > > > > > > -- > Cheers > > Jinmei >
