json: Improve error handling and adding ability to limit log output.

recipe_modules/json/test_api.py

Index: recipe_modules/json/test_api.py
diff --git a/recipe_modules/json/test_api.py b/recipe_modules/json/test_api.py
index e260238a82aaaac0d8235923dcc271c1fd8fc996..f15b264c6ac0cb44cfa5987e733a05a4992ce14e 100644
--- a/recipe_modules/json/test_api.py
+++ b/recipe_modules/json/test_api.py
@@ -26,6 +26,14 @@ class JsonTestApi(recipe_test_api.RecipeTestApi):
     ret.placeholder_data[(self._module.NAME, 'output', name)] = placeholder_data
     return ret
 
+  def not_found(self, retcode=None, name=None):
+    """Can be used to have the json.output file be missing."""
+    ret = recipe_test_api.StepTestData()
+    ret.retcode=retcode
+    # FIXME: How do I get None into the data?

[Paweł Hajdan Jr., 2017/03/23 12:22:12]

What happens if you just use None below instead of '' ?

[iannucci, 2017/03/23 19:44:13]

That's what I would expect as well... however I would also expect that passing
None to invalid() above will also work?

+    ret.placeholder_data[(self._module.NAME, 'output', name)] = ''
+    return ret
+
   def output_stream(self, data, stream='stdout', retcode=None, name=None):
     assert stream in ('stdout', 'stderr')
     ret = recipe_test_api.StepTestData()