dorsale 3.14.4 → 3.14.5

data/spec/models/dorsale/flyboy/task_spec.rb

@@ -0,0 +1,254 @@
+require "rails_helper"
+
+describe Dorsale::Flyboy::Task do
+  it { is_expected.to belong_to(:taskable) }
+  it { is_expected.to belong_to :owner }
+  it { is_expected.to have_many(:comments).dependent(:destroy) }
+
+  it { is_expected.to validate_presence_of :name }
+  it { is_expected.to validate_presence_of :term }
+  it { is_expected.to_not validate_presence_of :reminder_type }
+
+  it { is_expected.to_not validate_presence_of :taskable }
+
+  describe "default values" do
+    before do
+      @task = Dorsale::Flyboy::Task.new
+    end
+
+    it "#done should be false" do
+      expect(@task.done).to be false
+    end
+
+    it "#term should be present" do
+      expect(@task.term).to be_present
+    end
+
+    it "#reminter should be nil" do
+      expect(@task.reminder_type).to     be nil
+      expect(@task.reminder_date).to     be nil
+      expect(@task.reminder_duration).to be nil
+      expect(@task.reminder_unit).to     be nil
+    end
+
+    it "#progress should be 0" do
+      expect(@task.progress).to eq 0
+    end
+  end
+
+  describe "#validation" do
+    it "factory should be #valid?" do
+      expect(create(:flyboy_task)).to be_valid
+    end
+  end # describe '#validation'
+
+  describe "reminders" do
+    before do
+      Timecop.freeze "2017-04-12"
+    end
+
+    it "should not validate reminder_date if reminder_type is nil" do
+      task = described_class.new(reminder_type: nil)
+      task.valid?
+      expect(task.errors).to_not have_key :reminder_date
+    end
+
+    it "should validate reminder_date if reminder_type is present" do
+      task = described_class.new(reminder_type: "custom")
+      task.valid?
+      expect(task.errors).to have_key :reminder_date
+    end
+
+    it "should auto reset reminder_date" do
+      task = described_class.new(
+        :term          => "2017-04-12",
+        :reminder_type => "custom",
+        :reminder_date => "2017-04-11",
+      )
+      task.reminder_type = nil
+      expect(task.reminder_date).to eq nil
+    end
+
+    it "should auto set reminder_date by with days" do
+      task = described_class.new(
+        :term              => "2017-04-12",
+        :reminder_type     => "duration",
+        :reminder_duration => 1,
+        :reminder_unit     => "days",
+      )
+      expect(task.reminder_date).to eq Date.parse("2017-04-11")
+    end
+
+    it "should auto set reminder_date by with weeks" do
+      task = described_class.new(
+        :term              => "2017-04-12",
+        :reminder_type     => "duration",
+        :reminder_duration => 2,
+        :reminder_unit     => "weeks",
+      )
+      expect(task.reminder_date).to eq Date.parse("2017-03-29")
+    end
+
+    it "should auto set reminder_date by with months" do
+      task = described_class.new(
+        :term              => "2017-04-12",
+        :reminder_type     => "duration",
+        :reminder_duration => 1,
+        :reminder_unit     => "months",
+      )
+      expect(task.reminder_date).to eq Date.parse("2017-03-12")
+    end
+
+    it "update term should update reminder_date" do
+      task = described_class.new(
+        :term              => "2017-04-12",
+        :reminder_type     => "duration",
+        :reminder_duration => 1,
+        :reminder_unit     => "days",
+      )
+      expect(task.reminder_date).to eq Date.parse("2017-04-11")
+      task.term = "2017-04-25"
+      expect(task.reminder_date).to eq Date.parse("2017-04-24")
+    end
+  end # describe "reminders"
+
+  describe "scopes" do
+    it "should return delayed undone tasks" do
+      task  = create(:flyboy_task, owner: @user1, term: Date.current+1)
+      task1 = create(:flyboy_task, owner: @user1, term: Date.current-1, done: true)
+      task2 = create(:flyboy_task, owner: @user1, term: Date.current-1, done: false)
+      task3 = create(:flyboy_task, owner: @user1, term: Date.current-2, done: true)
+      task4 = create(:flyboy_task, owner: @user1, term: Date.current-2, done: false)
+      tasks = ::Dorsale::Flyboy::Task.delayed
+      expect(tasks).to contain_exactly(task2, task4)
+    end
+
+    it "should return today undone tasks" do
+      task = create(:flyboy_task, owner: @user1, term: Date.current+1)
+      task1 = create(:flyboy_task, owner: @user1, term: Date.current, done: true)
+      task2 = create(:flyboy_task, owner: @user1, term: Date.current, done: false)
+      tasks = ::Dorsale::Flyboy::Task.today
+      expect(tasks).to contain_exactly(task2)
+    end
+
+    it "should return tomorrow undone tasks" do
+      task = create(:flyboy_task, owner: @user1, term: Date.current)
+      task1 = create(:flyboy_task, owner: @user1, term: Date.tomorrow, done: true)
+      task2 = create(:flyboy_task, owner: @user1, term: Date.tomorrow, done: false)
+      tasks = ::Dorsale::Flyboy::Task.tomorrow
+      expect(tasks).to contain_exactly(task2)
+    end
+
+    it "should return this week undone tasks" do
+      Timecop.freeze(2015, 5, 21, 12, 0, 0)
+      task = create(:flyboy_task, owner: @user1, term: Date.current-7, done: false)
+      task1 = create(:flyboy_task, owner: @user1, term: Date.current+2, done: true)
+      task2 = create(:flyboy_task, owner: @user1, term: Date.current+2, done: false)
+      task3 = create(:flyboy_task, owner: @user1, term: Date.current+3, done: false)
+      task4 = create(:flyboy_task, owner: @user1, term: Date.current+5, done: false)
+      tasks = ::Dorsale::Flyboy::Task.this_week
+      expect(tasks).to contain_exactly(task2, task3)
+    end
+
+    it "should return next week undone tasks" do
+      Timecop.freeze(2015, 5, 21)
+      task = create(:flyboy_task, owner: @user1, term: Date.current, done: false)
+      task1 = create(:flyboy_task, owner: @user1, term: Date.current+7, done: true)
+      task2 = create(:flyboy_task, owner: @user1, term: Date.current+7, done: false)
+      task3 = create(:flyboy_task, owner: @user1, term: Date.current+9, done: false)
+      task4 = create(:flyboy_task, owner: @user1, term: Date.current+12, done: false)
+      tasks = ::Dorsale::Flyboy::Task.next_week
+      expect(tasks).to contain_exactly(task2, task3)
+    end
+  end # describe "scopes"
+
+  describe "states" do
+    # L’affichage de la couleur de la tâche dépend de son achèvement:
+    # Si date jour < date relance alors noir
+    let(:task_ontime) {
+      create(:flyboy_task,
+        :reminder_type => "custom",
+        :reminder_date => Date.current + 1.day,
+        :term          => Date.current + 3.days,
+        :done          => false,
+      )
+    }
+
+    it "should return :ontime" do
+      expect(task_ontime.state).to eq "ontime"
+      expect(described_class.ontime).to eq [task_ontime]
+    end
+
+    # Si date relance <= date jour
+    let(:task_onwarning) {
+      create(:flyboy_task,
+        :reminder_type => "custom",
+        :reminder_date => Date.current,
+        :term          => Date.current + 3.days,
+        :done          => false,
+      )
+    }
+
+    it "should return :onwarning" do
+      expect(task_onwarning.state).to eq "onwarning"
+      expect(described_class.onwarning).to eq [task_onwarning]
+    end
+
+    # Si date butoir <= date jour alors rouge
+    let(:task_onalert) {
+      create(:flyboy_task,
+        :reminder_type => "custom",
+        :reminder_date => Date.current - 3.days,
+        :term          => Date.current,
+        :done          => false,
+      )
+    }
+
+    it "should return :onalert" do
+      expect(task_onalert.state).to eq "onalert"
+      expect(described_class.onalert).to eq [task_onalert]
+    end
+
+    # Si action faite alors vert
+    let(:task_done) {
+      create(:flyboy_task,
+        :reminder_type => "custom",
+        :reminder_date => Date.current - 3.days,
+        :term          => Date.current - 1.day,
+        :done          => true,
+      )
+    }
+
+    it "should return on_warning_or_alert" do
+      expect(described_class.on_warning_or_alert).to contain_exactly(task_onwarning, task_onalert)
+    end
+
+    it "should return :done" do
+      expect(task_done.state).to eq "done"
+      expect(described_class.done).to eq [task_done]
+    end
+  end # describe "states"
+
+  describe "#create_term_changed_comment!" do
+    let!(:task) { create(:flyboy_task, term: "2019-10-02") }
+    let!(:author) { create(:user) }
+
+    it "should create comment" do
+      expect {
+        task.create_term_changed_comment!(previous: Date.new(2019, 10, 1), author: author)
+      }.to change(Dorsale::Flyboy::TaskComment, :count).by(1)
+
+      task_comment = task.comments.last_created
+      expect(task_comment.task).to eq task
+      expect(task_comment.author).to eq author
+      expect(task_comment.description).to eq \
+        "L'échéance de la tâche a été modifiée de 01/10/2019 à 02/10/2019."
+    end
+
+    it "should not create comment if date not changed" do
+      expect {
+        task.create_term_changed_comment!(previous: Date.new(2019, 10, 2), author: author)
+      }.to_not change(Dorsale::Flyboy::TaskComment, :count)
+    end
+  end # describe "#create_term_changed_comment!"
+end

data/spec/models/dorsale/i18n_spec.rb

@@ -0,0 +1,30 @@
+require "rails_helper"
+
+# TODO : test FR + EN
+
+RSpec.describe "i18n" do
+  Rails.application.eager_load!
+
+  ::Dorsale::ApplicationRecord.descendants.each do |model|
+    next unless model.to_s.start_with?("Dorsale")
+
+    it "should i18n #{model} model name" do
+      i18n = model.model_name.human(default: "")
+      expect(i18n).to be_present
+    end
+
+    model.column_names.each do |column_name|
+      next if column_name.end_with?("_type")
+      next if column_name.end_with?("_bak")
+      next if column_name.start_with?("old_")
+
+      # user_id => user
+      column_name = column_name[0..-4] if column_name.end_with?("_id")
+
+      it "should i18n #{model}##{column_name}" do
+        i18n = model.human_attribute_name(column_name, default: "")
+        expect(i18n).to be_present
+      end
+    end
+  end
+end

data/spec/models/dorsale/users_spec.rb

@@ -0,0 +1,65 @@
+require "rails_helper"
+
+RSpec.describe User, type: :model do
+  it "should have a valid factores" do
+    expect(create(:user)).to be_valid
+  end
+
+  it { is_expected.to respond_to :is_active }
+
+  describe "default values" do
+    it "default active value should be true" do
+      @user = create(:user)
+      expect(@user.is_active).to eq true
+    end
+
+    it "should create a valid password upon creation" do
+      user = create(:user, password: nil, password_confirmation: nil)
+      expect(user).to be_persisted
+    end
+
+    it "should not override choosen password" do
+      user = create(:user, password: "totototo", password_confirmation: nil)
+      expect(user.password).to eq "totototo"
+    end
+
+    it "should send a welcome message upon creation" do
+      expect {
+        create(:user)
+      }.to change(ActionMailer::Base.deliveries, :count).by(1)
+    end
+  end
+
+  describe "avatar" do
+    before do
+      @user = create(:user,
+        :email  => "user@example.org",
+        :avatar => Dorsale::Engine.root.join("spec/files/avatar.png").open,
+      )
+    end
+
+    it "gravatar_url" do
+      expect(@user.gravatar_url).to include "gravatar"
+    end
+
+    it "local_avatar_url" do
+      expect(@user.local_avatar_url).to include "avatar.png"
+    end
+
+    it "avatar_url should return local_avatar_url if present" do
+      expect(@user.avatar_url).to include "avatar.png"
+    end
+
+    it "avatar_url should return gravatar_url if no local avatar" do
+      @user.remove_avatar!
+      @user.save!
+      expect(@user.avatar_url).to include "gravatar"
+    end
+
+    it "should work if email is nil" do
+      @user.remove_avatar!
+      allow(@user).to receive(:email)
+      expect(@user.avatar_url).to be_present
+    end
+  end
+end

data/spec/pdfs/dorsale/billing_machine/invoice_multiple_vat_pdf_spec.rb

@@ -0,0 +1,42 @@
+require "rails_helper"
+
+describe ::Dorsale::BillingMachine::InvoiceMultipleVatPdf, pdfs: true do
+  before :each do
+    ::Dorsale::BillingMachine.vat_mode = :multiple
+  end
+
+  let(:invoice) {
+    i = create(:billing_machine_invoice)
+
+    create(:billing_machine_invoice_line,
+      :invoice  => i,
+      :vat_rate => 19.6,
+    )
+
+    i
+  }
+
+  let(:generate!) {
+    Dorsale::BillingMachine::PdfFileGenerator.(invoice)
+    invoice.reload
+  }
+
+  let(:content) {
+    generate!
+    Yomu.new(invoice.pdf_file.path).text
+  }
+
+  it "should not display global vat rate" do
+    expect(content).to_not include "TVA 19,60 %"
+    expect(content).to include "MONTANT TVA"
+    expect(content).to include "TVA %"
+  end
+
+  it "should work with empty invoice" do
+    invoice = ::Dorsale::BillingMachine::Invoice.new
+
+    expect {
+      described_class.new(invoice).tap(&:build).render_with_attachments
+    }.to_not raise_error
+  end
+end

data/spec/pdfs/dorsale/billing_machine/invoice_single_vat_pdf_spec.rb

@@ -0,0 +1,42 @@
+require "rails_helper"
+
+describe ::Dorsale::BillingMachine::InvoiceSingleVatPdf, pdfs: true do
+  before :each do
+    ::Dorsale::BillingMachine.vat_mode = :single
+  end
+
+  let(:invoice) {
+    i = create(:billing_machine_invoice)
+
+    create(:billing_machine_invoice_line,
+      :invoice  => i,
+      :vat_rate => 19.6,
+    )
+
+    i
+  }
+
+  let(:generate!) {
+    Dorsale::BillingMachine::PdfFileGenerator.(invoice)
+    invoice.reload
+  }
+
+  let(:content) {
+    generate!
+    Yomu.new(invoice.pdf_file.path).text
+  }
+
+  it "should display global vat rate" do
+    expect(content).to include "TVA 19,60"
+    expect(content).to_not include "MONTANT TVA"
+    expect(content).to_not include "TVA %"
+  end
+
+  it "should work with empty invoice" do
+    invoice = ::Dorsale::BillingMachine::Invoice.new
+
+    expect {
+      described_class.new(invoice).tap(&:build).render_with_attachments
+    }.to_not raise_error
+  end
+end

data/spec/pdfs/dorsale/billing_machine/quotation_multiple_vat_pdf_spec.rb

@@ -0,0 +1,42 @@
+require "rails_helper"
+
+describe ::Dorsale::BillingMachine::QuotationMultipleVatPdf, pdfs: true do
+  before :each do
+    ::Dorsale::BillingMachine.vat_mode = :multiple
+  end
+
+  let(:quotation) {
+    q = create(:billing_machine_quotation)
+
+    create(:billing_machine_quotation_line,
+      :quotation => q,
+      :vat_rate  => 19.6,
+    )
+
+    q
+  }
+
+  let(:generate!) {
+    Dorsale::BillingMachine::PdfFileGenerator.(quotation)
+    quotation.reload
+  }
+
+  let(:content) {
+    generate!
+    Yomu.new(quotation.pdf_file.path).text
+  }
+
+  it "should not display global vat rate" do
+    expect(content).to_not include "TVA 19,60 %"
+    expect(content).to include "MONTANT TVA"
+    expect(content).to include "TVA %"
+  end
+
+  it "should work with empty quotation" do
+    quotation = ::Dorsale::BillingMachine::Quotation.new
+
+    expect {
+      described_class.new(quotation).tap(&:build).render_with_attachments
+    }.to_not raise_error
+  end
+end