Remove runtime-test that was misguided

Turns out that other code was expecting to just get a an empty string if there ware no activity.
BloomBooks · Jul 15, 2022 · 7a8f3a8 · 7a8f3a8
1 parent 25d1564
commit 7a8f3a8
Showing 1 changed file with 3 additions and 12 deletions.
diff --git a/src/activities/activityManager.ts b/src/activities/activityManager.ts
@@ -98,20 +98,11 @@ export class ActivityManager {
         // the moment the html data-analyticscategories attribute is plural,
         // but here in code we know that we currently only handle the whole
         // thing as a single string, so we just call it "category".
-        const category =
+        return (
             pageDiv.getAttribute("data-analyticscategories") ||
             pageDiv.getAttribute("data-analyticsCategories") ||
-            "";
-        if (!category) {
-            window.alert(
-                "Activity pages must have a data-analyticscategories attribute."
-            );
-            throw new Error(
-                "Activity pages must have a data-analyticscategories attribute."
-            );
-        } else {
-            return category;
-        }
+            ""
+        );
     }
 
     public processPage(