Dev 330 (#99)

* INSIGHTS-12 Initial structure of insight functionality

* INSIGHTS-12 Added todo statements

* INSIGHTS-12 Moved Insights out of client

* INSIGHTS-12 Adjusted insight methods to reside in the client class. Removed the ability to collect insights before sending, now the every time the user invokes the collectInsights method, it will also send. This prevents any State issues with the algorithm.

* INSIGHTS-12 Added a todo. Tests fail for unknown reasons at this time

* INSIGHTS-12 Fixed method call. Added a todo to get url from config if necessary.

* INSIGHTS-12 Fixed method call.

* INSIGHTS-12 added json serialization. might not be needed

* INSIGHTS-12 commented test temporarily

* INSIGHTS-12 comment updates and json .encode change

* INSIGHTS-12 comment update

* INSIGHTS-12 changed method signatures to match documentation https://insights1.enthalpy.click/developers/clients/python#publishing-algorithmia-insights

* INSIGHTS-12 Added system property for queue reader url

* INSIGHTS-12 Fixed URL to not be https

* INSIGHTS-12 minor version update

* INSIGHTS-12 revert change

* INSIGHTS-12 removed todo

* INSIGHTS-12 uncommented test. May start failing again in the pipeline.

* INSIGHTS-12 commented test.

* INSIGHTS-12 changed version. Removed unused import.

* INSIGHTS-12 changed url to include /v1/

* Allow listing of non-data:// files on cli

* Allow catting non-data:// files on cli

* Fix tests

* restructured list_languages method

* return error instead of printing for unit test

Co-authored-by: robert-close <[email protected]>
Co-authored-by: Kenny Daniel <[email protected]>
Co-authored-by: Kenny Daniel <[email protected]>
Co-authored-by: John Bragg <[email protected]>

Author: 4 people

Algorithmia/CLI.py

@@ -296,7 +296,14 @@ def get_environment_by_language(self,language,client):
 
     def list_languages(self, client):
         response = client.get_supported_languages()
-        return response
+        table = []
+        if "error" not in response:
+            table.append("{:<25} {:<35}".format('Name','Description'))
+            for lang in response:
+                table.append("{:<25} {:<35}".format(lang['name'],lang['display_name']))
+        else:
+            table.append(json.dumps(response))
+        return table
 
 
     def getBuildLogs(self, user, algo, client):

Algorithmia/__main__.py

@@ -202,9 +202,8 @@ def main():
 
     elif args.cmd == 'languages':
         response = CLI().list_languages(client)
-        print("{:<25} {:<35}".format('Name','Description'))
-        for lang in response:
-            print("{:<25} {:<35}".format(lang['name'],lang['display_name']))
+        for line in response:
+            print(line)
 
     elif args.cmd == 'template':
         CLI().get_template(args.envid,args.dest,client)

Test/CLI_test.py

@@ -181,9 +181,9 @@ def test_get_environment(self):
 
 	def test_list_languages(self):
 		result = CLI().list_languages(self.client)
-		if("error" in result):
+		if("error" in result[0]):
 			print(result)
-		self.assertTrue(result is not None and "name" in result[0])
+		self.assertTrue(result is not None and "anaconda3" in result[1])
 
 
 	def test_rm(self):