proceane opened a new pull request, #5021:
URL: https://github.com/apache/zeppelin/pull/5021

   ### What is this PR for?
   Since there is currently no test class that can test AngularInterpreter, I 
added unit tests for `org.apache.zeppelin.angular.AngularInterpreter` to cover 
the core behavior.
   - Verify interpret echoes input with Type.ANGULAR and Code.SUCCESS
   - Verify empty input returns Type.ANGULAR with empty data
   - Verify cancel does not throw
   - Verify completion returns empty list
   - Verify getFormType() returns FormType.NATIVE
   - Verify getProgress() returns 0
   - Verify scheduler is FIFOScheduler and name contains AngularInterpreter
   
   
   ### What type of PR is it?
   Improvement
   
   ### What is the Jira issue?
   https://issues.apache.org/jira/browse/ZEPPELIN-6273
   
   ### How should this be tested?
   * In the angular module, run test.
   
   ### Questions:
   * Does the license files need to update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to