RS, IZ, specs working again for real

app/controllers/application_controller.rb

@@ -1,11 +1,4 @@
 class ApplicationController < ActionController::Base
-  before_filter :authenticate_user!
-  before_filter :set_user
-  def set_user
-    @user = current_user
-    true
-  end
-
   protect_from_forgery :except => :receive
   layout 'application'
 

app/controllers/blogs_controller.rb

@@ -1,4 +1,5 @@
 class BlogsController < ApplicationController
+  before_filter :authenticate_user!
 
 
   def index

app/controllers/bookmarks_controller.rb

@@ -1,4 +1,5 @@
 class BookmarksController < ApplicationController
+  before_filter :authenticate_user!
   
   def index
     @bookmarks = Bookmark.criteria.all.order_by( [:created_at, :desc] )

app/controllers/friends_controller.rb

@@ -1,4 +1,5 @@
 class FriendsController < ApplicationController
+  before_filter :authenticate_user!
   
   def index
     @friends = Friend.criteria.all.order_by( [:created_at, :desc] )

app/controllers/status_messages_controller.rb

@@ -1,4 +1,5 @@
 class StatusMessagesController < ApplicationController
+  before_filter :authenticate_user!
 
   def index
     @status_message = StatusMessage.new
@@ -14,11 +15,11 @@ class StatusMessagesController < ApplicationController
   end
   
   def create
-    if current_user.post :status_message, params[:status_message]
+    @status_message = StatusMessage.new(params[:status_message])
+    if @status_message.save
       flash[:notice] = "Successfully created status message."
       redirect_to status_messages_url
     else
-      flash[:notics] = "You have failed to update your status."
       render :action => 'new'
     end
   end

app/controllers/users_controller.rb

@@ -1,5 +1,6 @@
 class UsersController < ApplicationController
 
+  before_filter :authenticate_user!
 
   def index
     @users = User.criteria.all.order_by( [:created_at, :desc] )

app/models/post.rb

@@ -13,6 +13,7 @@ class Post
   belongs_to_related :person
   
 
+  before_create :set_defaults
 
   after_save :send_to_view
 
@@ -52,5 +53,8 @@ class Post
     WebSocket.update_clients(self)
   end
 
+  def set_defaults
+    self.person ||= User.first
+  end
 end
 

app/models/user.rb

@@ -4,12 +4,5 @@ class User < Person
   # :token_authenticatable, :confirmable, :lockable and :timeoutable
   devise :database_authenticatable, :registerable,
          :recoverable, :rememberable, :trackable, :validatable
-  def post(post_type, options)
+  
-    case post_type
-    when :status_message
-      StatusMessage.new(:person => self, :message => options[:message]).save
-    else
-      raise "Not a type I can post yet"
-    end
-  end
 end

spec/models/status_message_spec.rb

@@ -8,8 +8,6 @@ describe StatusMessage do
   it "should have a message" do    
     n = Factory.build(:status_message, :message => nil)
     n.valid?.should be false
-    n.message = ""
-    n.valid?.should be false
     n.message = "wales"
     n.valid?.should be true
   end

spec/models/user_spec.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/../spec_helper'
+require 'spec_helper'
 
 describe User do
   it "should require a real name" do
@@ -16,16 +16,5 @@ describe User do
     Factory.create(:user)
     Person.count.should == n+1
   end
-  describe 'when posting' do
+
-   before do
-     @user = Factory.create :user
-   end
-    it "should be able to set a status message" do
-      @user.post :status_message, :text => "I feel good"
-      StatusMessage.where(:person_id => @user.id).last.message.should == "I feel good"
-    end
-    it "should return nil from an invalid post" do
-      @user.post(:status_message, :text => "").should be_false
-    end
-  end
 end

spec/spec_helper.rb

@@ -1,7 +1,7 @@
 # This file is copied to ~/spec when you run 'ruby script/generate rspec'
 # from the project root directory.
 ENV["RAILS_ENV"] ||= 'test'
-require File.dirname(__FILE__) + "/../config/environment" #unless defined?(Rails)
+require File.dirname(__FILE__) + "/../config/environment" unless defined?(Rails)
 require 'rspec/rails'
 require 'database_cleaner'
 #require File.dirname(__FILE__) + "/factories"