From bf4dabc4f49495c62f44546503a5dbf760f070e0 Mon Sep 17 00:00:00 2001
From: Dennis Collinson <dennis.collective@gmail.com>
Date: Tue, 6 Mar 2012 16:26:19 -0800
Subject: [PATCH] MS DC posting from post/new werks

---
 features/trumpeter.feature                    |  4 ++--
 .../javascripts/app/models/status_message.js  | 12 +++++++++-
 public/javascripts/app/pages/post-new.js      | 10 +++++---
 .../app/templates/post-form.handlebars        |  2 +-
 .../javascripts/app/views/post_form_view.js   | 23 ++++++++++++++----
 spec/javascripts/app/app_spec.js              | 10 +++-----
 .../app/models/status_message_spec.js         | 12 ++++++++++
 spec/javascripts/app/pages/post_new_spec.js   |  8 +++++++
 spec/javascripts/app/views/post_form_spec.js  | 24 +++++++++++++++++--
 9 files changed, 84 insertions(+), 21 deletions(-)
 create mode 100644 spec/javascripts/app/models/status_message_spec.js

diff --git a/features/trumpeter.feature b/features/trumpeter.feature
index 28a8b0288..7b42452d7 100644
--- a/features/trumpeter.feature
+++ b/features/trumpeter.feature
@@ -8,5 +8,5 @@ Feature: Creating a new post
     When I trumpet
     And I write "Rectangles are awesome"
     And I press "Share"
-#    When I go to the stream page
-#    Then I should see "Rectangles are awesome" as the first post in my stream
+    When I go to "/stream"
+    Then I should see "Rectangles are awesome" as the first post in my stream
diff --git a/public/javascripts/app/models/status_message.js b/public/javascripts/app/models/status_message.js
index 7a2c8120a..50190883d 100644
--- a/public/javascripts/app/models/status_message.js
+++ b/public/javascripts/app/models/status_message.js
@@ -1 +1,11 @@
-app.models.StatusMessage = app.models.Post.extend({ });
+app.models.StatusMessage = app.models.Post.extend({
+  url : function(){
+    return this.isNew() ? '/status_messages' : '/posts/' + this.get("id");
+  },
+
+  mungeAndSave : function(){
+    var mungedAttrs = {status_message : _.clone(this.attributes), aspect_ids : ["public"]}
+
+    this.save(mungedAttrs)
+  }
+});
diff --git a/public/javascripts/app/pages/post-new.js b/public/javascripts/app/pages/post-new.js
index 339f19424..0e2e446d1 100644
--- a/public/javascripts/app/pages/post-new.js
+++ b/public/javascripts/app/pages/post-new.js
@@ -4,9 +4,13 @@ app.pages.PostNew = app.views.Base.extend({
   subviews : { "#new-post" : "postForm"},
 
   initialize : function(){
-    console.log("In the page")
-
-    this.model = new app.models.Post()
+    this.model = new app.models.StatusMessage()
     this.postForm = new app.views.PostForm({model : this.model})
+
+    this.model.bind("setFromForm", this.saveModel, this)
+  },
+
+  saveModel : function(){
+    this.model.mungeAndSave();
   }
 })
diff --git a/public/javascripts/app/templates/post-form.handlebars b/public/javascripts/app/templates/post-form.handlebars
index 4d5253c6a..0c8ffb88c 100644
--- a/public/javascripts/app/templates/post-form.handlebars
+++ b/public/javascripts/app/templates/post-form.handlebars
@@ -1,4 +1,4 @@
 <form class="new-post">
     <label>text<textarea class="text"/></label>
-    <input type="submit" value="Share" class="btn-primary"></input>
+    <input type="submit" class="btn-primary" value="Share"></input>
 </form>
diff --git a/public/javascripts/app/views/post_form_view.js b/public/javascripts/app/views/post_form_view.js
index b451ae48b..e29bd54b9 100644
--- a/public/javascripts/app/views/post_form_view.js
+++ b/public/javascripts/app/views/post_form_view.js
@@ -1,12 +1,25 @@
 app.views.PostForm = app.views.Base.extend({
   templateName : "post-form",
 
-  initialize : function(){
-    console.log("In the form")
+  events :{
+    'submit form' : 'setModelAttributes'
   },
 
-  postRenderTemplate: function(){
-    console.log("I'm getting rendered")
-  }
+  formAttrs : {
+    ".text" : "text"
+  },
 
+  setModelAttributes : function(evt){
+    if(evt){ evt.preventDefault(); }
+
+    var form = this.$("form");
+
+    function setValueFromField(memo, attribute, selector){
+      memo[attribute] =  form.find(selector).val()
+      return memo
+    }
+
+    this.model.set(_.inject(this.formAttrs, setValueFromField, {}))
+    this.model.trigger("setFromForm")
+  }
 });
\ No newline at end of file
diff --git a/spec/javascripts/app/app_spec.js b/spec/javascripts/app/app_spec.js
index 6989301f4..9323020d9 100644
--- a/spec/javascripts/app/app_spec.js
+++ b/spec/javascripts/app/app_spec.js
@@ -3,20 +3,16 @@ describe("app", function() {
     it("sets the user if given one and returns the current user", function() {
       expect(app.user()).toBeFalsy()
     });
-    
+
     it("returns false if the current_user isn't set", function() {
       app._user = undefined;
-
       expect(app.user()).toEqual(false);
     });
-  });
 
-  describe('currentUser', function(){
     it("sets the user if given one and returns the current user", function() {
-      expect(app.currentUser.authenticated()).toBeFalsy()
+      expect(app.user()).toBeFalsy()
       app.user({name: "alice"});
-
       expect(app.user().get("name")).toEqual("alice");
     });
   });
-});
+})
diff --git a/spec/javascripts/app/models/status_message_spec.js b/spec/javascripts/app/models/status_message_spec.js
new file mode 100644
index 000000000..ad8c93a31
--- /dev/null
+++ b/spec/javascripts/app/models/status_message_spec.js
@@ -0,0 +1,12 @@
+describe("app.models.StatusMessage", function(){
+  describe("#url", function(){
+    it("is /status_messages when its new", function(){
+      var post = new app.models.StatusMessage()
+      expect(post.url()).toBe("/status_messages")
+    })
+
+    it("is /posts/id when it has an id", function(){
+      expect(new app.models.StatusMessage({id : 5}).url()).toBe("/posts/5")
+    })
+  })
+})
\ No newline at end of file
diff --git a/spec/javascripts/app/pages/post_new_spec.js b/spec/javascripts/app/pages/post_new_spec.js
index e0e13b39a..20055757b 100644
--- a/spec/javascripts/app/pages/post_new_spec.js
+++ b/spec/javascripts/app/pages/post_new_spec.js
@@ -6,4 +6,12 @@ describe("app.pages.PostNew", function(){
   it("renders", function(){
     this.page.render();
   })
+
+  context("when the model receives setFromForm", function(){
+    it("it calls mungeAndSave", function(){
+      spyOn(this.page.model, "mungeAndSave")
+      this.page.model.trigger("setFromForm")
+      expect(this.page.model.mungeAndSave).toHaveBeenCalled();
+    })
+  })
 });
\ No newline at end of file
diff --git a/spec/javascripts/app/views/post_form_spec.js b/spec/javascripts/app/views/post_form_spec.js
index 03bcbe233..ff6fea7ed 100644
--- a/spec/javascripts/app/views/post_form_spec.js
+++ b/spec/javascripts/app/views/post_form_spec.js
@@ -4,7 +4,27 @@ describe("app.views.PostForm", function(){
     this.view = new app.views.PostForm({model : this.post})
   })
 
-  it("renders", function(){
+  describe("rendering", function(){
+    beforeEach(function(){
       this.view.render()
+    })
+
+    describe("submitting a valid form", function(){
+      beforeEach(function(){
+        this.view.$("form .text").val("Oh My")
+      })
+
+      it("instantiates a post on form submit", function(){
+        this.view.$("form").submit()
+        expect(this.view.model.get("text")).toBe("Oh My")
+      })
+
+      it("triggers a  'setFromForm' event", function(){
+        var spy = jasmine.createSpy();
+        this.view.model.bind("setFromForm", spy);
+        this.view.$("form").submit();
+        expect(spy).toHaveBeenCalled();
+      })
+    })
   })
-})
\ No newline at end of file
+})