Merge pull request #186 from fsek/fix-news-error

Fixing news and election

Author: davidwessman

app/models/election.rb

@@ -29,7 +29,7 @@ def view_status
       return :before
     elsif start <= Time.zone.now && self.end > Time.zone.now
       return :during
-    elsif closing > Time.zone.now
+    elsif closing.nil? ||  closing > Time.zone.now
       return :after
     else
       return :closed

app/views/news/_news.html.erb

@@ -1,9 +1,10 @@
 <div class="row">
-    <div class="funny-boxes funny-boxes-left-orange">
-      <h4><%=link_to(news.title,news)%></h4>
-      <%=simple_format(truncate(news.content,length: 140, separator: '', escape: false){ link_to "Läs mer", news })%>
-      <ul class="list-unstyled">
-        <li><i class="fa fa-calendar"></i> <%=news.created_at%></li>
-      </ul>
-    </div>
+  <div class="funny-boxes funny-boxes-left-orange">
+    <h4><%=link_to(news.title,news)%></h4>
+    <%=simple_format(truncate(news.content,length: 140, separator: '', escape: false){ link_to "Läs mer", news })%>
+    <ul class="list-unstyled">
+      <li><%= fa_icon('calendar') %> <%= link_to(news.user, news.user) %></li>
+      <li><i class="fa fa-calendar"></i> <%=news.created_at%></li>
+    </ul>
+  </div>
 </div>

lib/tasks/tests_data.rake

@@ -108,5 +108,22 @@ namespace :db do
     Contact.find_or_create_by(name: 'Spindelman - David', email: '[email protected]',
                               public: true, text: 'Detta är en linte spindelman',
                               council_id: Council.first.try(:id))
+
+    # News
+    News.find_or_create_by!(title: 'Ett helt nytt användarsystem',
+                           content: 'Nu har vi en himla massa roliga funktioner som blir mycket lättare att lägga in, det är ju <br>toppen.',
+                           user: User.first)
+    News.find_or_create_by!(title: 'Andra helt nytt användarsystem',
+                           content: 'Nu har vi en himla massa roliga funktioner som blir mycket lättare att lägga in, det är ju <br>toppen.',
+                           user: User.first)
+    News.find_or_create_by!(title: 'Tredje helt nytt användarsystem',
+                           content: 'Nu har vi en himla massa roliga funktioner som blir mycket lättare att lägga in, det är ju <br>toppen.',
+                           user: User.first)
+    News.find_or_create_by!(title: 'Fjärde helt nytt användarsystem',
+                           content: 'Nu har vi en himla massa roliga funktioner som blir mycket lättare att lägga in, det är ju <br>toppen.',
+                           user: User.first)
+    News.find_or_create_by!(title: 'Femte helt nytt användarsystem',
+                           content: 'Nu har vi en himla massa roliga funktioner som blir mycket lättare att lägga in, det är ju <br>toppen.',
+                           user: User.first)
   end
 end

spec/features/home_spec.rb

@@ -1,8 +1,18 @@
 require 'spec_helper'
 
 feature 'User goes to the home page' do
+  let(:news1) { create(:news) }
+  let(:news2) { create(:news) }
+  let(:news3) { create(:news) }
+
+  before do
+    news1
+    news2
+    news3
+  end
+
   scenario 'They see "F-sektionen" on the page' do
-    visit '/'
+    visit root_path
     page.should have_content('F-sektionen')
     page.status_code.should eq(200)
   end