search

gchq / gaffer-tools

gaffer-tools is deprecated. Use https://github.com/gchq/gafferpy instead
Apache License 2.0
50 stars 29 forks source link

Gh 845 is operation supported null input #846

Closed j8934893 closed 4 years ago

j8934893 commented 4 years ago

Covers issue #845

Updated is_operation_supported url generation to handle operation=None.

Added doc strings for the execute_get and is_operation_supported methods with expected inputs and examples.

Commit 5f5bb4b - Changed the outputs of the two methods from strings to json.loads() of those strings - happy to revert this if it is breaking something elsewhere I couldn't see. All usage of these methods (*_it tests) uses an immediate json.loads so it seemed logical to handle that within the method.

j8934893 commented 4 years ago

Reverted some of the commits to avoid breaking changes. Operation parameter for is_operation_supported method has been made mandatory.