#28 #29 Add tests for articles

app/models/custom/article.rb

@@ -7,6 +7,8 @@ class Article < ActiveRecord::Base
 
  validates :title, presence: true
  validates :status, presence: true, inclusion: { in: VALID_STATUSES }
+ validates :author, presence: true
+ validates :content, presence: true
 
  scope :published, -> { where(status: 'published').order('id DESC') }
 

app/views/custom/articles/index.html.erb

@@ -3,7 +3,7 @@
 <main>
  <div class="row">
  <%# TODO - change id and class to smth dedicated to articles %>
- <div class="small-12 column">
+ <div id="articles" class="small-12 column">
  <% if @articles.any? %>
  <%= render partial: 'articles/article', collection: @articles %>
  <%= paginate @articles %>

app/views/custom/articles/show.html.erb

@@ -14,11 +14,11 @@
  <%= render '/shared/author_info', resource: @article %>
  <span class="bullet">&nbsp;&bull;&nbsp;</span>
  <%= l @article.created_at.to_date %>
- <!--span class="bullet">&nbsp;&bull;&nbsp;</span-->
- <%# geozone_name(@article) %>
  </div>
 
- <%= image_tag @article.image_url(:large), alt: @article.image.title, class: 'article-image' %>
+ <% if @article.image.present? %>
+ <%= image_tag @article.image_url(:large), alt: @article.image.title, class: 'article-image' %>
+ <% end %>
 
  <%= safe_html_with_links @article.content.html_safe %>
 

spec/custom/factories.rb

@@ -0,0 +1,17 @@
+FactoryBot.define do
+ factory :article do
+ sequence(:title) { |n| "Titre #{n}" }
+ association :author, factory: :user
+ content "Article body"
+
+ trait :published do
+ status 'published'
+ end
+
+ trait :draft do
+ status 'draft'
+ end
+
+ end
+
+end

spec/custom/features/articles_spec.rb

@@ -0,0 +1,38 @@
+# coding: utf-8
+require 'rails_helper'
+
+feature 'Articles' do
+
+ scenario 'Index' do
+ published_articles = [create(:article, :published), create(:article, :published)]
+ create(:article, :draft)
+
+ visit articles_path
+
+ expect(page).to have_selector('#articles .article', count: 2)
+ published_articles.each do |article|
+ within('#articles') do
+ expect(page).to have_content article.title
+ expect(page).to have_css("a[href='#{article_path(article)}']", text: article.title)
+ end
+ end
+ end
+
+ scenario 'Show' do
+ article = create(:article, :published)
+
+ visit article_path(article)
+
+ expect(page).to have_content article.title
+ expect(page).to have_content "Article body"
+ expect(page).to have_content article.author.name
+ expect(page).to have_content I18n.l(article.created_at.to_date)
+ expect(page).to have_selector(avatar(article.author.name))
+ expect(page.html).to include "<title>#{article.title}</title>"
+
+ within('.social-share-button') do
+ expect(page.all('a').count).to be(4) # Twitter, Facebook, Google+, Telegram
+ end
+ end
+
+end

spec/custom/models/article_spec.rb

@@ -0,0 +1,10 @@
+# coding: utf-8
+require 'rails_helper'
+
+describe Article do
+ let(:article) { build(:article) }
+
+ it "is valid" do
+ expect(article).to be_valid
+ end
+end

spec/spec_helper.rb

@@ -8,6 +8,7 @@
 Dir["./spec/support/**/*.rb"].sort.each { |f| require f }
 Dir["./spec/shared/**/*.rb"].sort.each { |f| require f }
 Dir["./spec/custom/support/**/*.rb"].sort.each { |f| require f }
+Dir["./spec/custom/factories.rb"].sort.each { |f| require f }
 
 RSpec.configure do |config|
  config.use_transactional_fixtures = false