add back test system and add tests for Diary.creatable_by?

Author: Trim

Gemfile

@@ -45,11 +45,15 @@ gem "sass-rails",            "~>5.0", require: assets
 gem "rails-sass-images",              require: assets
 gem "uglifier",                       require: assets
 
+group :development, :test do
+  # Call 'byebug' anywhere in the code to stop execution and get a debugger console
+  gem "byebug", platforms: :mri
+end
+
 group :development do
   gem "annotate"
   gem "better_errors"
   gem "binding_of_caller"
-  gem "byebug",                platform: :mri
   gem "capistrano",            "~>2.15", github: 'capistrano', branch: 'legacy-v2'
   gem "capistrano-maintenance"
   gem "letter_opener"
@@ -62,6 +66,12 @@ group :development do
   gem "web-console"
 end
 
+group :test do
+  # Adds support for Capybara system testing and selenium driver
+  gem "capybara", ">= 2.15"
+  gem "selenium-webdriver"
+end
+
 group :production, :alpha do
   gem "unicorn",               "~>5.1"
   gem "gctools",               "~>0.2"

docker-compose.yaml

@@ -13,6 +13,7 @@ services:
       - ./app:/linuxfr.org/app
       - ./db:/linuxfr.org/db
       - ./public:/linuxfr.org/public
+      - ./test:/linuxfr.org/test
       # uploads are shared with the nginx service
       - data-uploads:/linuxfr.org/uploads
     tmpfs:

test/application_system_test_case.rb

@@ -0,0 +1,5 @@
+require "test_helper"
+
+class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
+  driven_by :selenium, using: :headless_firefox, screen_size: [1400, 1400]
+end

test/fixtures/accounts.yml

@@ -0,0 +1,60 @@
+# Required Anonyme login for Account model
+anonymous:
+  user: community
+  login: Anonyme
+  email: [email protected]
+  role: inactive
+
+<% 1000.times do |n| %>
+visitor_<%= n%>:
+  user: <%= "visitor_#{n}" %>
+  login: <%= "visitor_#{n}" %>
+  email: <%= "visitor_#{n}@example.com" %>
+  role: visitor
+<% end %>
+
+visitor_negative_karma:
+  user: visitor_negative_karma
+  login: visitor_negative_karma
+  email: [email protected]
+  role: visitor
+  karma: -100
+
+visitor_zero_karma:
+  user: visitor_zero_karma
+  login: visitor_zero_karma
+  email: [email protected]
+  role: visitor
+  karma: 0
+
+<% 7.times do |n| %>
+editor_<%= n%>:
+  user: <%= "editor_#{n}" %>
+  login: <%= "editor_#{n}" %>
+  email: <%= "editor_#{n}@example.com" %>
+  role: editor
+<% end %>
+
+<% 2.times do |n| %>
+maintainer_<%= n%>:
+  user: <%= "maintainer_#{n}" %>
+  login: <%= "maintainer_#{n}" %>
+  email: <%= "maintainer_#{n}@example.com" %>
+  role: maintainer
+<% end %>
+
+<% 12.times do |n| %>
+moderator_<%= n%>:
+  user: <%= "moderator_#{n}" %>
+  login: <%= "moderator_#{n}" %>
+  email: <%= "moderator_#{n}@example.com" %>
+  role: moderator
+<% end %>
+
+<% 6.times do |n| %>
+admin_<%= n%>:
+  user: <%= "admin_#{n}" %>
+  login: <%= "admin_#{n}" %>
+  email: <%= "admin_#{n}@example.com" %>
+  role: admin
+<% end %>

test/fixtures/diaries.yml

@@ -0,0 +1,6 @@
+lorem:
+  title: Lorem ipsum
+  cached_slug: lorem-ipsum
+  owner: visitor_1
+  body: Lorem ipsum
+  wiki_body: Lorem ipsum

test/fixtures/users.yml

@@ -0,0 +1,42 @@
+# Required Collectif user for User model
+community:
+  name: Collectif
+  cached_slug: collectif
+
+<% 1000.times do |n| %>
+visitor_<%= n%>:
+  name: <%= "visitor_#{n}" %>
+  cached_slug: <%= "visitor_#{n}" %>
+<% end %>
+
+visitor_negative_karma:
+  name: visitor_negative_karma
+  cached_slug: visitor_negative_karma
+
+visitor_zero_karma:
+  name: visitor_zero_karma
+  cached_slug: visitor_zero_karma
+
+<% 7.times do |n| %>
+editor_<%= n%>:
+  name: <%= "editor_#{n}" %>
+  cached_slug: <%= "editor_#{n}" %>
+<% end %>
+
+<% 2.times do |n| %>
+maintainer_<%= n%>:
+  name: <%= "maintainer_#{n}" %>
+  cached_slug: <%= "maintainer_#{n}" %>
+<% end %>
+
+<% 12.times do |n| %>
+moderator_<%= n%>:
+  name: <%= "moderator_#{n}" %>
+  cached_slug: <%= "moderator_#{n}" %>
+<% end %>
+
+<% 6.times do |n| %>
+admin_<%= n%>:
+  name: <%= "admin_#{n}" %>
+  cached_slug: <%= "admin_#{n}" %>
+<% end %>

test/models/diary_test.rb

@@ -0,0 +1,23 @@
+require 'test_helper'
+
+class DiaryTest < ActiveSupport::TestCase
+  test "user with positive karma can create diary" do
+    diary = diaries(:lorem).dup;
+    assert diary.creatable_by?(diary.owner.account);
+    assert diary.save();
+  end
+
+  test "user with zero karma cannot create diary" do
+    diary = diaries(:lorem).dup;
+    diary.owner = users(:visitor_zero_karma);
+    assert_not diary.creatable_by?(diary.owner.account);
+    assert diary.save(), "Diary model were not saved";
+  end
+
+  test "user with negative karma cannot create diary" do
+    diary = diaries(:lorem).dup;
+    diary.owner = users(:visitor_negative_karma);
+    assert_not diary.creatable_by?(diary.owner.account);
+    assert diary.save(), "Diary model were not saved";
+  end
+end

test/test_helper.rb

@@ -0,0 +1,10 @@
+ENV['RAILS_ENV'] ||= 'test'
+require_relative '../config/environment'
+require 'rails/test_help'
+
+class ActiveSupport::TestCase
+  # Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order.
+  fixtures :all
+
+  # Add more helper methods to be used by all tests here...
+end