kubeflow / pipelines / 3586 / 1

0%

master: 0%

push

travis-ci-com

Add visualization server and unit tests for visualization server (#1647)

* Added visualization server

* Updated function names, added comments, and made serviceURL a property of VisualizationService

* Added unit tests for visualizaiton.go

* Updated BUILD.bazel

* Addressed PR comments made by @IronPan

* Wrapped input_path argument value in string when generating python arguments

* GenerateVisualizationFromRequest -> generateVisualizationFromRequest

* Addressed additional PR feedback from @IronPan

* Removed getArgumentsAsJSONFromRequest
* Removed createPythonArgumentsFromRequest
* Moved `fmt.Sprintf` to generateVisualizationFromRequest
* Updated tests to reflect changes

* Added missing word to comment

1556 of 1745 branches covered (89.17%)

Branch coverage included in aggregate %.

2654 of 2790 relevant lines covered (95.13%)

61.86 hits per line