adding more debugging info

KarrLab · Jan 4, 2019 · a496554 · a496554
1 parent dac3ea3
commit a496554
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/karr_lab_build_utils/core.py b/karr_lab_build_utils/core.py
@@ -2330,9 +2330,10 @@ def trigger_tests_of_downstream_dependencies(self, config_filename='.karr_lab_bu
                            "current build cascade\n"
                            "\t\tbuild: {}\n"
                            "\t\tbuild time: {}\n"
+                           "\t\tbuild status: {}\n"
                            "\t\tupstream repo: {}\n"
                            "\t\tupstream build: {}\n"
-                           "\t\tupstream build time: {}").format(build['build_num'], build['start_time'],
+                           "\t\tupstream build time: {}").format(build['build_num'], build['start_time'], build['status'],
                                                                  upstream_repo_name, upstream_build_num, upstream_build_time_str)
                     not_triggered_packages[package] = msg
                     self.logger.info("\t{}: {}".format(package, msg))
@@ -2348,9 +2349,10 @@ def trigger_tests_of_downstream_dependencies(self, config_filename='.karr_lab_bu
                            "by the current build cascade\n"
                            "\t\tbuild: {}\n"
                            "\t\tbuild time: {}\n"
+                           "\t\tbuild status: {}\n"
                            "\t\tupstream repo: {}\n"
                            "\t\tupstream build: {}\n"
-                           "\t\tupstream build time: {}").format(build['build_num'], build['start_time'],
+                           "\t\tupstream build time: {}").format(build['build_num'], build['start_time'], build['status'],
                                                                  upstream_repo_name, upstream_build_num, upstream_build_time_str)
                     not_triggered_packages[package] = msg
                     self.logger.info("\t{}: {}".format(package, msg))
@@ -2368,9 +2370,10 @@ def trigger_tests_of_downstream_dependencies(self, config_filename='.karr_lab_bu
                            "the commit time of the current build cascade\n"
                            "\t\tbuild: {}\n"
                            "\t\tbuild time: {}\n"
+                           "\t\tbuild status: {}\n"
                            "\t\tupstream repo: {}\n"
                            "\t\tupstream build: {}\n"
-                           "\t\tupstream build time: {}").format(build['build_num'], build['start_time'],
+                           "\t\tupstream build time: {}").format(build['build_num'], build['start_time'], build['status'],
                                                                  upstream_repo_name, upstream_build_num, upstream_build_time_str)
                     not_triggered_packages[package] = msg
                     self.logger.info("\t{}: {}".format(package, msg))

diff --git a/tests/test_core.py b/tests/test_core.py
@@ -2145,6 +2145,7 @@ def test_trigger_tests_of_downstream_dependencies_with_upstream(self):
                         'UPSTREAM_BUILD_NUM': '1',
                     },
                     'start_time': '2016-01-01T01:01:01.001Z',
+                    'status': 'success',
                 }
             ],
         })
@@ -2186,7 +2187,9 @@ def test_trigger_tests_of_downstream_dependencies_trigger_original_upstream(self
 
         requests_get_1 = attrdict.AttrDict({
             'raise_for_status': lambda: None,
-            'json': lambda: {'start_time': '2017-01-01T01:01:01-05:00'},
+            'json': lambda: {
+                'start_time': '2017-01-01T01:01:01-05:00',                
+                },
         })
         requests_get_2 = attrdict.AttrDict({
             'raise_for_status': lambda: None,
@@ -2195,11 +2198,13 @@ def test_trigger_tests_of_downstream_dependencies_trigger_original_upstream(self
                     'build_num': 0,
                     'build_parameters': {},
                     'start_time': '2016-01-01T01:01:01.001Z',
+                    'status': 'success',
                 },
                 {
                     'build_num': 1,
                     'build_parameters': {},
                     'start_time': '2016-01-01T01:01:01.001Z',
+                    'status': 'success',
                 },
             ],
         })