Move functions onto instance of Encryptor, not class

kerryb · Mar 24, 2013 · c6a1226 · c6a1226
1 parent abadf11
commit c6a1226
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/app/assets/javascripts/encryptor.coffee b/app/assets/javascripts/encryptor.coffee
@@ -1,10 +1,13 @@
-window.Encryptor = class Encryptor
-  @encrypt: =>
+class Encryptor
+  constructor: ->
+    $("#plain-text").keyup @encrypt
+
+  encrypt: =>
     plainText = $("#plain-text").val()
     $.get "/encrypt", {text: plainText}, @update
 
-  @update: (data) =>
+  update: (data) =>
     $("#encrypted-text").val data.text
 
 $ ->
-  $("#plain-text").keyup window.Encryptor.encrypt
+  window.encryptor = new Encryptor
diff --git a/spec/javascripts/encryptor_spec.coffee b/spec/javascripts/encryptor_spec.coffee
@@ -4,12 +4,12 @@ describe "Encryptor", ->
 
   describe ".update", ->
     it "sets the #encrypted-text element's value to the supplied data's text attribute", ->
-      window.Encryptor.update {text: "foo"}
+      window.encryptor.update {text: "foo"}
       expect($("#encrypted-text").val()).toEqual "foo"
 
   describe ".encrypt", ->
     it "requests encryption of the #plain-text field, calling .update on success", ->
       $("#plain-text").val "foo"
       spyOn $, "get"
-      window.Encryptor.encrypt()
-      expect($.get).toHaveBeenCalledWith "/encrypt", {text: "foo"}, window.Encryptor.update
+      window.encryptor.encrypt()
+      expect($.get).toHaveBeenCalledWith "/encrypt", {text: "foo"}, window.encryptor.update