Re: DrillRestServer Tests

2017-08-25 Thread Timothy Farkas
Hey Kunal, I tried rest-assured out for my rest api unit tests and It works. I'll include it as part of my PR for DRILL-5723. Thanks, Tim From: Timothy Farkas Sent: Friday, August 25, 2017 2:46:53 PM To: dev@drill.apache.org Subject: Re: DrillRestServer Tests

Re: DrillRestServer Tests

2017-08-25 Thread Timothy Farkas
From: Kunal Khatua <kkha...@mapr.com> Sent: Friday, August 25, 2017 10:45:58 AM To: dev@drill.apache.org Subject: RE: DrillRestServer Tests Krystal pointed out to an existing tool that allows testing for REST APIs. https://github.com/rest-assured/rest-assured Unfortu

RE: DrillRestServer Tests

2017-08-25 Thread Kunal Khatua
: Paul Rogers [mailto:prog...@mapr.com] Sent: Friday, August 25, 2017 10:01 AM To: dev@drill.apache.org Subject: Re: DrillRestServer Tests To simplify code reviews in this area, should we add tests? Might be mighty hard to “mentally execute” in lieu of tests… - Paul > On Aug 25, 2017, at 9:40

Re: DrillRestServer Tests

2017-08-25 Thread Paul Rogers
To simplify code reviews in this area, should we add tests? Might be mighty hard to “mentally execute” in lieu of tests… - Paul > On Aug 25, 2017, at 9:40 AM, Parth Chandra wrote: > > Don't think there are any unit tests, and again there is no rest api > client. The UI is

Re: DrillRestServer Tests

2017-08-25 Thread Parth Chandra
Don't think there are any unit tests, and again there is no rest api client. The UI is the only test tool, sadly. On Thu, Aug 24, 2017 at 12:55 PM, Timothy Farkas wrote: > Hi All, > > I want to add some unit tests for a new rest endpoint I added to the > DrillRestServer. I've