Merge pull request #249 from pusher/include-urls-in-errors

Fix failing test, add test for cluster check
pusher · Sep 4, 2017 · 9de0a35 · 9de0a35
2 parents 56c2847 + 78a9356
commit 9de0a35
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/spec/javascripts/unit/core/pusher_spec.js b/spec/javascripts/unit/core/pusher_spec.js
@@ -76,9 +76,15 @@ describe("Pusher", function() {
 
     it("should allow a hex key", function() {
       spyOn(Logger, "warn");
-      var pusher = new Pusher("1234567890abcdef");
+      var pusher = new Pusher("1234567890abcdef", { cluster: "mt1" });
       expect(Logger.warn).not.toHaveBeenCalled();
     });
+
+    it("should warn if no cluster supplied", function() {
+      spyOn(Logger, "warn");
+      var pusher = new Pusher("1234567890abcdef");
+      expect(Logger.warn).toHaveBeenCalled();
+    });
   });
 
   describe("after construction", function() {