commit 5da29c6b46c2a4670b53fd7a24f2ff8a512132a6
author Clemenz Portmann <portmannc@google.com> Mon May 17 19:28:13 2021 -0700
committer Clemenz Portmann <portmannc@google.com> Mon May 17 19:33:42 2021 -0700
tree a9682a581407fc394f526c57af6f60694d4bb03b
parent 349e30f4d866b9b01867604c4cc74f2ce435052c

ITS: fix div by 0 in test_param_tonemap_mode.py

bug: 188466351
Change-Id: I29d57421abc4667329c5d962560463cc3374abf7

apps/CameraITS/tests/scene1_2/test_param_tonemap_mode.py

diff --git a/apps/CameraITS/tests/scene1_2/test_param_tonemap_mode.py b/apps/CameraITS/tests/scene1_2/test_param_tonemap_mode.py
index 93147f1..257ff3e 100644
--- a/apps/CameraITS/tests/scene1_2/test_param_tonemap_mode.py
+++ b/apps/CameraITS/tests/scene1_2/test_param_tonemap_mode.py
@@ -116,10 +116,12 @@
         cap = cam.do_capture(req, fmt)
         img_name = '%s_n=%d.jpg' % (os.path.join(log_path, _NAME), n)
         means_1.append(compute_means_and_save(cap, img_name))
+      if 0.0 in means_1[1]:
+        raise AssertionError(f'0.0 value in test 1 means: {means_1[0]}')
       rgb_ratios = [means_1[1][i]/means_1[0][i] for i in range(_NUM_COLORS)]
       logging.debug('Test 1: RGB ratios: %s', str(rgb_ratios))
 
-      # assert proper behavior
+      # Assert proper behavior
       for i in range(_NUM_COLORS-1):
         if rgb_ratios[i+1]-rgb_ratios[i] < _MIN_RGB_RATIO_DIFF:
           raise AssertionError(