Return empty state_details for deprecated bundle types.

codalab/lib/bundle_util.py

@@ -375,7 +375,7 @@ def get_bundle_state_details(bundle):
     type = bundle.get('bundle_type')
     state = bundle.get('state')
     state_details_by_type = {
-        'dataset': {
+        'dataset': {  # uploaded
             'created': 'Bundle has been created but its contents have not been uploaded yet.',
             'uploading': 'Bundle contents are being uploaded.',
             'ready': 'Bundle has finished uploading successfully, and is ready to be used for further runs.',
@@ -399,6 +399,10 @@ def get_bundle_state_details(bundle):
         },
     }
 
+    # We can remove the defensive checks below once program bundles are converted to dataset bundles.
+    # Related Issue: https://github.com/codalab/codalab-worksheets/issues/4235
+    state_details = state_details_by_type.get(type, {}).get(state, '')
+
     if state == 'preparing' or state == 'running':
         return run_status
-    return state_details_by_type[type][state]
+    return state_details

tests/unit/lib/bundle_util_test.py

@@ -3,6 +3,14 @@
 
 
 class GetBundleStateDetailsTest(unittest.TestCase):
+    def test_deprecated_bundle(self):
+        """
+        Returns an empty string for deprecated bundle types.
+        """
+        deprecated_bundle = {'bundle_type': 'program', 'state': 'ready'}
+        state_details = get_bundle_state_details(deprecated_bundle)
+        self.assertEqual(state_details, '')
+
     def test_running_bundle(self):
         """
         Returns `run_status` if state is `running`.