Featured Issue #55: Added tests

app/controllers/emails_controller.rb

@@ -1,6 +1,8 @@
 class EmailsController < ApplicationController
   def send_email
+    debug(email_params)
     @email = Email.new(email_params)
+    debug(@email)
     Mailer.send_generic_email(@email.hide_recipients, @email.recipients, @email.reply_to, @email.subject, @email.content)
     redirect_to :events, notice: t('.sending_successful')
   end

app/models/email.rb

@@ -10,10 +10,14 @@ class Email
   attribute :subject, :type => String
   attribute :content, :type => String
 
-  validates_presence_of :hide_recipients, :recipients, :reply_to, :subject, :content
+  validates_presence_of :recipients, :reply_to, :subject, :content
+  validates_inclusion_of :hide_recipients, in: [true, false]
 
   def initialize(attributes = {})
     attributes.each do |name, value|
+      if name == 'content'
+        value.gsub!(/\n/, '<br/>')
+      end
       send("#{name}=", value)
     end
   end

spec/factories/emails.rb

@@ -6,4 +6,4 @@
     subject "Email-Subject"
     content "Email-Content"
   end
-end
+end

spec/models/email_spec.rb

@@ -0,0 +1,19 @@
+require 'rails_helper'
+
+describe Email do
+  let(:valid_email) {FactoryGirl.build(:email) }
+  let(:invalid_email) { FactoryGirl.build(:email, subject: nil) }
+  let(:mulitline_email) { FactoryGirl.build(:email, content: "Email-content \n Email-Content") }
+
+  it 'is buildd by factory' do
+    expect(valid_email).to be_valid
+  end
+
+  it 'should not be valid without content' do
+    expect(invalid_email).to_not be_valid
+  end
+
+  it 'should convert ruby line breaks to html <br>' do
+    expect(mulitline_email).to_not include('\n')
+  end
+end