runat-1128988: mockCommands added json converters that we added to basic

I'm not clear why we thought that being able to display json was only something you'd want to do during testing, but since we later added them to the default set of converters, we don't need duplicates in test code. Signed-off-by: Joe Walker <[email protected]>
joewalker · Mar 5, 2015 · 345fdda · bgrins · Mar 13, 2015 · bgrins
1 parent 08be168
commit 345fdda
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 14 deletions.
diff --git a/lib/gcli/converters/basic.js b/lib/gcli/converters/basic.js
@@ -57,9 +57,12 @@ exports.items = [
   {
     item: 'converter',
     from: 'json',
-    to: 'dom',
-    exec: function(json, conversionContext) {
-      return nodeFromDataToString(JSON.stringify(json), conversionContext);
+    to: 'view',
+    exec: function(json, context) {
+      var html = JSON.stringify(json, null, '&#160;').replace(/\n/g, '<br/>');
+      return {
+        html: '<pre>' + html + '</pre>'
+      };
     }
   },
   {

diff --git a/lib/gcli/test/mockCommands.js b/lib/gcli/test/mockCommands.js
@@ -45,21 +45,10 @@ function createExec(name) {
 mockCommands.items = [
   {
     item: 'converter',
-    from: 'json',
-    to: 'string',
-    exec: function(json, context) {
-      return JSON.stringify(json, null, '  ');
     }
   },
   {
     item: 'converter',
-    from: 'json',
-    to: 'view',
-    exec: function(json, context) {
-      var html = JSON.stringify(json, null, '&#160;').replace(/\n/g, '<br/>');
-      return {
-        html: '<pre>' + html + '</pre>'
-      };
     }
   },
   {