From 955dc106930409c24b07e185e3a6bb9843efe446 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 20 Nov 2017 08:10:59 +0200 Subject: [PATCH 01/87] Extract partial #623 --- app/views/admin/registrars/_billing.html.erb | 17 +++++++++++++++++ app/views/admin/registrars/show.html.erb | 19 +------------------ config/locales/admin/registrars.en.yml | 3 +++ config/locales/en.yml | 4 +++- 4 files changed, 24 insertions(+), 19 deletions(-) create mode 100644 app/views/admin/registrars/_billing.html.erb diff --git a/app/views/admin/registrars/_billing.html.erb b/app/views/admin/registrars/_billing.html.erb new file mode 100644 index 000000000..f3960233e --- /dev/null +++ b/app/views/admin/registrars/_billing.html.erb @@ -0,0 +1,17 @@ +
+
+ <%= t '.header' %> +
+
+
+
<%= Registrar.human_attribute_name :vat_no %>
+
<%= registrar.vat_no %>
+ +
<%= Registrar.human_attribute_name :accounting_customer_code %>
+
<%= registrar.accounting_customer_code %>
+ +
<%= Registrar.human_attribute_name :billing_email %>
+
<%= registrar.billing_email %>
+
+
+
diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index c09602985..b42d1b9f3 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -41,12 +41,6 @@
<%= @registrar.reg_no %>
-
- <%= t(:vat_no) %> -
-
- <%= @registrar.vat_no %> -
<%= t(:reference_no) %>
@@ -71,12 +65,6 @@
<%= @registrar.website %>
-
- <%= Registrar.human_attribute_name :accounting_customer_code %> -
-
- <%= @registrar.accounting_customer_code %> -
@@ -120,15 +108,10 @@
<%= @registrar.billing_address %>
-
- <%= t(:billing_email) %> -
-
- <%= @registrar.billing_email %> -
+ <%= render 'billing', registrar: @registrar %> <%= render 'admin/registrars/show/preferences', registrar: registrar %> diff --git a/config/locales/admin/registrars.en.yml b/config/locales/admin/registrars.en.yml index 3d269f64b..e805b19c6 100644 --- a/config/locales/admin/registrars.en.yml +++ b/config/locales/admin/registrars.en.yml @@ -12,6 +12,9 @@ en: preferences: header: Preferences + billing: + header: Billing + create: created: Registrar has been successfully created not_created: Unable to create registrar diff --git a/config/locales/en.yml b/config/locales/en.yml index 6b1dc7e31..89e7b0aad 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -199,7 +199,6 @@ en: alg: 'Algorithm' public_key: 'Public key' registrar: - billing_email: 'Billing e-mail' phone: 'Contact phone' email: 'Contact e-mail' state: 'State / Province' @@ -782,3 +781,6 @@ en: delimiter: " " precision: 2 unit: € + + attributes: + vat_no: VAT number From d2e84d28c6f9a82df583e80007259b5fc04f16ef Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 20 Nov 2017 08:12:40 +0200 Subject: [PATCH 02/87] Remove unused HTML #623 --- app/views/admin/registrars/show.html.erb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index b42d1b9f3..c0a596b27 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -117,7 +117,7 @@
-
+
<%= t('.api_users') %> @@ -157,7 +157,7 @@
-
+
<%= t(:white_ips) %> From 4a41984a5c8da8287b5e93e57d510e4c28588474 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 21 Nov 2017 12:04:41 +0200 Subject: [PATCH 03/87] Use human_attribute_name #623 --- app/views/registrant/registrars/show.haml | 2 +- app/views/registrar/invoices/partials/_seller.haml | 2 +- app/views/registrar/invoices/pdf.haml | 2 +- config/locales/en.yml | 1 - 4 files changed, 3 insertions(+), 4 deletions(-) diff --git a/app/views/registrant/registrars/show.haml b/app/views/registrant/registrars/show.haml index e0da73ee3..6d4aa9e79 100644 --- a/app/views/registrant/registrars/show.haml +++ b/app/views/registrant/registrars/show.haml @@ -19,7 +19,7 @@ %dt= t(:reg_no) %dd= @registrar.reg_no - %dt= t(:vat_no) + %dt= Registrar.human_attribute_name :vat_no %dd= @registrar.vat_no %dt= t(:id) diff --git a/app/views/registrar/invoices/partials/_seller.haml b/app/views/registrar/invoices/partials/_seller.haml index 0008c2f13..9c5387e16 100644 --- a/app/views/registrar/invoices/partials/_seller.haml +++ b/app/views/registrar/invoices/partials/_seller.haml @@ -16,7 +16,7 @@ %dt= t(:swift) %dd= @invoice.seller_swift - %dt= t(:vat_no) + %dt= Registrar.human_attribute_name :vat_no %dd= @invoice.seller_vat_no %dt= t(:address) diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index a74254a2a..61b4e5789 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -261,7 +261,7 @@ %br = "#{t('reg_no')} #{@invoice.seller_reg_no}" %br - = "#{t('vat_no')} #{@invoice.seller_vat_no}" + = "#{Registrar.human_attribute_name :vat_no} #{@invoice.seller_vat_no}" .col-md-3.left = @invoice.seller_phone diff --git a/config/locales/en.yml b/config/locales/en.yml index 89e7b0aad..4951f3b26 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -316,7 +316,6 @@ en: new_registrar: 'New registrar' registrar_details: 'Registrar details' - vat_no: 'VAT no' edit_registrar: 'Edit registrar' back_to_registrar: 'Back to registrar' registrar_deleted: 'Registrar deleted' From f86ab085acf5c1f637a300a76952cd08871cca07 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 08:38:26 +0200 Subject: [PATCH 04/87] Add `registrars.vat_rate` DB column #623 --- db/migrate/20180228055259_add_registrars_vat_rate.rb | 5 +++++ db/structure.sql | 5 ++++- 2 files changed, 9 insertions(+), 1 deletion(-) create mode 100644 db/migrate/20180228055259_add_registrars_vat_rate.rb diff --git a/db/migrate/20180228055259_add_registrars_vat_rate.rb b/db/migrate/20180228055259_add_registrars_vat_rate.rb new file mode 100644 index 000000000..5965b65cf --- /dev/null +++ b/db/migrate/20180228055259_add_registrars_vat_rate.rb @@ -0,0 +1,5 @@ +class AddRegistrarsVatRate < ActiveRecord::Migration + def change + add_column :registrars, :vat_rate, :decimal, precision: 4, scale: 3 + end +end diff --git a/db/structure.sql b/db/structure.sql index 237c82e66..722aa4111 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -2436,7 +2436,8 @@ CREATE TABLE registrars ( legacy_id integer, reference_no character varying, test_registrar boolean DEFAULT false, - language character varying NOT NULL + language character varying NOT NULL, + vat_rate numeric(4,3) ); @@ -5112,3 +5113,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180214213743'); INSERT INTO schema_migrations (version) VALUES ('20180218004148'); +INSERT INTO schema_migrations (version) VALUES ('20180228055259'); + From 7dee10c6835835769f8e92b43d5d1a59a2f5164d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 08:46:36 +0200 Subject: [PATCH 05/87] Rename `invoices.var_prc` to `vat_rate` #623 --- app/models/directo.rb | 2 +- app/models/invoice.rb | 6 +++--- app/models/registrar.rb | 2 +- app/views/registrar/invoices/partials/_items.haml | 2 +- app/views/registrar/invoices/pdf.haml | 2 +- config/locales/en.yml | 2 +- ...20180228064342_rename_invoices_var_prc_to_vat_rate.rb | 5 +++++ db/structure.sql | 4 +++- doc/models_complete.svg | 2 +- spec/factories/invoice.rb | 2 +- spec/models/invoice_spec.rb | 2 +- test/fixtures/invoices.yml | 2 +- test/models/invoice_test.rb | 9 +++++++++ 13 files changed, 29 insertions(+), 13 deletions(-) create mode 100644 db/migrate/20180228064342_rename_invoices_var_prc_to_vat_rate.rb create mode 100644 test/models/invoice_test.rb diff --git a/app/models/directo.rb b/app/models/directo.rb index 62cf43804..4a5f00dd9 100644 --- a/app/models/directo.rb +++ b/app/models/directo.rb @@ -34,7 +34,7 @@ class Directo < ActiveRecord::Base xml.line( "ProductID" => Setting.directo_receipt_product_name, "Quantity" => 1, - "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.sum_cache/(1+invoice.vat_prc), precision: 2, separator: "."), + "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.sum_cache/(1+invoice.vat_rate), precision: 2, separator: "."), "ProductName" => invoice.order ) } diff --git a/app/models/invoice.rb b/app/models/invoice.rb index ad478443d..37eb8a72c 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -28,7 +28,7 @@ class Invoice < ActiveRecord::Base validates :billing_email, email_format: { message: :invalid }, allow_blank: true validates :invoice_type, :due_date, :currency, :seller_name, - :seller_iban, :buyer_name, :invoice_items, :vat_prc, presence: true + :seller_iban, :buyer_name, :invoice_items, :vat_rate, presence: true before_create :set_invoice_number, :check_vat @@ -52,7 +52,7 @@ class Invoice < ActiveRecord::Base def check_vat if buyer.country_code != 'EE' && buyer.vat_no.present? - self.vat_prc = 0 + self.vat_rate = 0 end end @@ -157,7 +157,7 @@ class Invoice < ActiveRecord::Base end def vat - (sum_without_vat * vat_prc).round(2) + (sum_without_vat * vat_rate).round(2) end def sum diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 46022808f..5fb68fea2 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -81,7 +81,7 @@ class Registrar < ActiveRecord::Base payment_term: 'prepayment', description: description, currency: 'EUR', - vat_prc: Setting.registry_vat_prc, + vat_rate: Setting.registry_vat_prc, seller_name: Setting.registry_juridical_name, seller_reg_no: Setting.registry_reg_no, seller_iban: Setting.registry_iban, diff --git a/app/views/registrar/invoices/partials/_items.haml b/app/views/registrar/invoices/partials/_items.haml index 6ed5144a8..64fc71786 100644 --- a/app/views/registrar/invoices/partials/_items.haml +++ b/app/views/registrar/invoices/partials/_items.haml @@ -24,7 +24,7 @@ %td= currency(@invoice.sum_without_vat) %tr %th.no-border{colspan: 3} - %th= t('vat', vat_prc: (@invoice.vat_prc * 100).round) + %th= t('vat', rate: (@invoice.vat_rate * 100).round) %td= currency(@invoice.vat) %tr %th.no-border{colspan: 3} diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index 61b4e5789..6f92c4a8a 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -242,7 +242,7 @@ %td= "#{currency(@invoice.sum_without_vat)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} - %th= t('vat', vat_prc: (@invoice.vat_prc * 100).round) + %th= t('vat', rate: (@invoice.vat_rate * 100).round) %td= "#{currency(@invoice.vat)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} diff --git a/config/locales/en.yml b/config/locales/en.yml index 4951f3b26..58c4e4c69 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -553,7 +553,7 @@ en: invoice_number: Invoice no. seller: 'Seller' prepayment: 'Prepayment' - vat: 'VAT (%{vat_prc}%)' + vat: 'VAT (%{rate}%)' unpaid: 'Unpaid' your_current_account_balance_is: 'Your current account balance is %{balance} %{currency}' billing: 'Billing' diff --git a/db/migrate/20180228064342_rename_invoices_var_prc_to_vat_rate.rb b/db/migrate/20180228064342_rename_invoices_var_prc_to_vat_rate.rb new file mode 100644 index 000000000..b1d13de1d --- /dev/null +++ b/db/migrate/20180228064342_rename_invoices_var_prc_to_vat_rate.rb @@ -0,0 +1,5 @@ +class RenameInvoicesVarPrcToVatRate < ActiveRecord::Migration + def change + rename_column :invoices, :vat_prc, :vat_rate + end +end diff --git a/db/structure.sql b/db/structure.sql index 722aa4111..cb47a50c0 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -1127,7 +1127,7 @@ CREATE TABLE invoices ( currency character varying NOT NULL, description character varying, reference_no character varying, - vat_prc numeric(10,2) NOT NULL, + vat_rate numeric(10,2) NOT NULL, paid_at timestamp without time zone, seller_id integer, seller_name character varying NOT NULL, @@ -5115,3 +5115,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180218004148'); INSERT INTO schema_migrations (version) VALUES ('20180228055259'); +INSERT INTO schema_migrations (version) VALUES ('20180228064342'); + diff --git a/doc/models_complete.svg b/doc/models_complete.svg index 148b12942..12fb078a4 100644 --- a/doc/models_complete.svg +++ b/doc/models_complete.svg @@ -1687,7 +1687,7 @@ currency :string description :string reference_no :string -vat_prc :decimal +vat_rate :decimal paid_at :datetime seller_id :integer seller_name :string diff --git a/spec/factories/invoice.rb b/spec/factories/invoice.rb index 7eeed32b9..0ea81d5e4 100644 --- a/spec/factories/invoice.rb +++ b/spec/factories/invoice.rb @@ -8,7 +8,7 @@ FactoryBot.define do seller_name { 'EIS' } seller_city { 'Tallinn' } seller_street { 'Paldiski mnt. 123' } - vat_prc 0.2 + vat_rate 0.2 buyer { FactoryBot.create(:registrar) } after :build do |invoice| diff --git a/spec/models/invoice_spec.rb b/spec/models/invoice_spec.rb index 49d15310f..eb8866d07 100644 --- a/spec/models/invoice_spec.rb +++ b/spec/models/invoice_spec.rb @@ -54,7 +54,7 @@ describe Invoice do it 'should calculate sums correctly' do @invoice = create(:invoice) - @invoice.vat_prc.should == BigDecimal.new('0.2') + @invoice.vat_rate.should == BigDecimal.new('0.2') @invoice.sum_without_vat.should == BigDecimal.new('300.0') @invoice.vat.should == BigDecimal.new('60.0') @invoice.sum.should == BigDecimal.new('360.0') diff --git a/test/fixtures/invoices.yml b/test/fixtures/invoices.yml index 7a1c85dd1..04cedc153 100644 --- a/test/fixtures/invoices.yml +++ b/test/fixtures/invoices.yml @@ -6,7 +6,7 @@ DEFAULTS: &DEFAULTS seller_name: John Doe seller_iban: 1234 buyer_name: Jane Doe - vat_prc: 0.2 + vat_rate: 0.2 valid: <<: *DEFAULTS diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb new file mode 100644 index 000000000..c89c37626 --- /dev/null +++ b/test/models/invoice_test.rb @@ -0,0 +1,9 @@ +require 'test_helper' + +class InvoiceTest < ActiveSupport::TestCase + def test_invalid_without_vat_rate + invoice = Invoice.new(vat_rate: nil) + invoice.validate + assert invoice.errors.added?(:vat_rate, :blank) + end +end From dc620e7780d3f2e70a6959a8df3195e4c5e2d9c7 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 08:47:54 +0200 Subject: [PATCH 06/87] Fix typo #623 --- ...rb => 20180228064342_rename_invoices_vat_prc_to_vat_rate.rb} | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename db/migrate/{20180228064342_rename_invoices_var_prc_to_vat_rate.rb => 20180228064342_rename_invoices_vat_prc_to_vat_rate.rb} (53%) diff --git a/db/migrate/20180228064342_rename_invoices_var_prc_to_vat_rate.rb b/db/migrate/20180228064342_rename_invoices_vat_prc_to_vat_rate.rb similarity index 53% rename from db/migrate/20180228064342_rename_invoices_var_prc_to_vat_rate.rb rename to db/migrate/20180228064342_rename_invoices_vat_prc_to_vat_rate.rb index b1d13de1d..89b4de724 100644 --- a/db/migrate/20180228064342_rename_invoices_var_prc_to_vat_rate.rb +++ b/db/migrate/20180228064342_rename_invoices_vat_prc_to_vat_rate.rb @@ -1,4 +1,4 @@ -class RenameInvoicesVarPrcToVatRate < ActiveRecord::Migration +class RenameInvoicesVatPrcToVatRate < ActiveRecord::Migration def change rename_column :invoices, :vat_prc, :vat_rate end From fccc6eb9f0581a3ae299f4a11e5a38fac8cc573e Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 09:02:23 +0200 Subject: [PATCH 07/87] Change `invoices.vat_rate` DB column type #623 --- db/migrate/20180228070102_change_invoices_vat_rate_type.rb | 5 +++++ db/structure.sql | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 db/migrate/20180228070102_change_invoices_vat_rate_type.rb diff --git a/db/migrate/20180228070102_change_invoices_vat_rate_type.rb b/db/migrate/20180228070102_change_invoices_vat_rate_type.rb new file mode 100644 index 000000000..2c0790a18 --- /dev/null +++ b/db/migrate/20180228070102_change_invoices_vat_rate_type.rb @@ -0,0 +1,5 @@ +class ChangeInvoicesVatRateType < ActiveRecord::Migration + def change + change_column :invoices, :vat_rate, :decimal, precision: 4, scale: 3 + end +end diff --git a/db/structure.sql b/db/structure.sql index cb47a50c0..3f890fcd9 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -1127,7 +1127,7 @@ CREATE TABLE invoices ( currency character varying NOT NULL, description character varying, reference_no character varying, - vat_rate numeric(10,2) NOT NULL, + vat_rate numeric(4,3) NOT NULL, paid_at timestamp without time zone, seller_id integer, seller_name character varying NOT NULL, @@ -5117,3 +5117,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180228055259'); INSERT INTO schema_migrations (version) VALUES ('20180228064342'); +INSERT INTO schema_migrations (version) VALUES ('20180228070102'); + From 6f9ea85a95dd9b55efd2ad5da4e676c4c9d7ea7c Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 09:05:41 +0200 Subject: [PATCH 08/87] Change `invoices.vat_rate` DB column to NULL #623 --- db/migrate/20180228070431_change_invoice_vat_rate_to_null.rb | 5 +++++ db/structure.sql | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 db/migrate/20180228070431_change_invoice_vat_rate_to_null.rb diff --git a/db/migrate/20180228070431_change_invoice_vat_rate_to_null.rb b/db/migrate/20180228070431_change_invoice_vat_rate_to_null.rb new file mode 100644 index 000000000..37b8394b5 --- /dev/null +++ b/db/migrate/20180228070431_change_invoice_vat_rate_to_null.rb @@ -0,0 +1,5 @@ +class ChangeInvoiceVatRateToNull < ActiveRecord::Migration + def change + change_column_null :invoices, :vat_rate, true + end +end diff --git a/db/structure.sql b/db/structure.sql index 3f890fcd9..ba040e7cd 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -1127,7 +1127,7 @@ CREATE TABLE invoices ( currency character varying NOT NULL, description character varying, reference_no character varying, - vat_rate numeric(4,3) NOT NULL, + vat_rate numeric(4,3), paid_at timestamp without time zone, seller_id integer, seller_name character varying NOT NULL, @@ -5119,3 +5119,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180228064342'); INSERT INTO schema_migrations (version) VALUES ('20180228070102'); +INSERT INTO schema_migrations (version) VALUES ('20180228070431'); + From c47ca77ca655c3c1c35faecbf89ba2f3c3ac0944 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 23 Nov 2017 23:27:06 +0200 Subject: [PATCH 09/87] Add registrar VAT rate #623 --- .../admin/registrars_controller.rb | 3 +- app/models/registrar.rb | 20 ++++++++ app/views/admin/registrars/_billing.html.erb | 3 ++ .../admin/registrars/form/_billing.html.erb | 12 +++++ config/locales/en.yml | 1 + test/fixtures/registrars.yml | 7 +++ .../admin/registrars/show_registrar_test.rb | 8 +++ test/models/registrar_test.rb | 50 +++++++++++++++++++ test/test_helper.rb | 2 + 9 files changed, 105 insertions(+), 1 deletion(-) diff --git a/app/controllers/admin/registrars_controller.rb b/app/controllers/admin/registrars_controller.rb index 119abf85b..eb9ae9bd7 100644 --- a/app/controllers/admin/registrars_controller.rb +++ b/app/controllers/admin/registrars_controller.rb @@ -79,7 +79,8 @@ module Admin :code, :test_registrar, :accounting_customer_code, - :language) + :language, + :vat_rate,) end end end diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 5fb68fea2..393bd56f2 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -17,10 +17,22 @@ class Registrar < ActiveRecord::Base validates :name, :reg_no, :reference_no, :code, uniqueness: true validates :accounting_customer_code, presence: true validates :language, presence: true + validates :vat_rate, :vat_no, absence: true, if: :local_vat_payer? + validates :vat_rate, presence: true, if: 'foreign_vat_payer? && vat_no.blank?' + validates :vat_rate, absence: true, if: 'foreign_vat_payer? && vat_no?' + validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 99 }, allow_nil: true validate :forbidden_codes after_initialize :set_defaults + before_save do + self.vat_rate = vat_rate / 100.0 if vat_rate + end + + after_find do |registrar| + registrar.vat_rate = registrar.vat_rate * 100 if registrar.vat_rate + end + def forbidden_codes return true unless ['CID'].include? code errors.add(:code, I18n.t(:forbidden_code)) @@ -163,4 +175,12 @@ class Registrar < ActiveRecord::Base def set_defaults self.language = Setting.default_language unless language end + + def local_vat_payer? + country == Registry.instance.legal_address_country + end + + def foreign_vat_payer? + !local_vat_payer? + end end diff --git a/app/views/admin/registrars/_billing.html.erb b/app/views/admin/registrars/_billing.html.erb index f3960233e..7ba778d08 100644 --- a/app/views/admin/registrars/_billing.html.erb +++ b/app/views/admin/registrars/_billing.html.erb @@ -7,6 +7,9 @@
<%= Registrar.human_attribute_name :vat_no %>
<%= registrar.vat_no %>
+
<%= Registrar.human_attribute_name :vat_rate %>
+
<%= number_to_percentage registrar.vat_rate, precision: 1 %>
+
<%= Registrar.human_attribute_name :accounting_customer_code %>
<%= registrar.accounting_customer_code %>
diff --git a/app/views/admin/registrars/form/_billing.html.erb b/app/views/admin/registrars/form/_billing.html.erb index a3adf1312..5bc87a93d 100644 --- a/app/views/admin/registrars/form/_billing.html.erb +++ b/app/views/admin/registrars/form/_billing.html.erb @@ -17,6 +17,18 @@
+
+
+ <%= f.label :vat_rate %> +
+
+
+ <%= f.number_field :vat_rate, class: 'form-control' %> +
%
+
+
+
+
<%= f.label :accounting_customer_code %> diff --git a/config/locales/en.yml b/config/locales/en.yml index 58c4e4c69..e9b8316d6 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -783,3 +783,4 @@ en: attributes: vat_no: VAT number + vat_rate: VAT rate diff --git a/test/fixtures/registrars.yml b/test/fixtures/registrars.yml index 0026f323b..b5715a0a5 100644 --- a/test/fixtures/registrars.yml +++ b/test/fixtures/registrars.yml @@ -16,3 +16,10 @@ goodnames: country_code: US accounting_customer_code: goodnames language: en + +foreign_vat_payer_with_rate: + <<: *DEFAULTS + name: 3 + reg_no: 3 + code: 3 + vat_rate: 0.2 diff --git a/test/integration/admin/registrars/show_registrar_test.rb b/test/integration/admin/registrars/show_registrar_test.rb index cb310317c..f13748191 100644 --- a/test/integration/admin/registrars/show_registrar_test.rb +++ b/test/integration/admin/registrars/show_registrar_test.rb @@ -16,4 +16,12 @@ class ShowRegistrarTest < ActionDispatch::IntegrationTest def test_language assert_text 'Language English' end + + def test_vat_number + assert_text 'US12345' + end + + def test_vat_rate + assert_text number_to_percentage(@registrar.vat_rate, precision: 1) + end end diff --git a/test/models/registrar_test.rb b/test/models/registrar_test.rb index 58a06b0ae..3ace52ba7 100644 --- a/test/models/registrar_test.rb +++ b/test/models/registrar_test.rb @@ -38,4 +38,54 @@ class RegistrarTest < ActiveSupport::TestCase registrar = Registrar.new(language: 'de') assert_equal 'de', registrar.language end + + def test_rejects_vat_number_when_local_vat_payer + Registry.instance.stub(:legal_address_country, Country.new('US')) do + @registrar.vat_no = 'US1' + @registrar.validate + assert @registrar.invalid? + end + end + + def test_rejects_vat_rate_when_local_vat_payer + Registry.instance.stub(:legal_address_country, Country.new('US')) do + @registrar.vat_rate = 20 + @registrar.validate + assert @registrar.invalid? + end + end + + def test_rejects_negative_vat_rate + @registrar.vat_rate = -1 + @registrar.validate + assert @registrar.invalid? + end + + def test_rejects_vat_rate_greater_than_max + @registrar.vat_rate = 100 + @registrar.validate + assert @registrar.invalid? + end + + def test_converts_integer_vat_rate_to_fractional + @registrar = registrars(:foreign_vat_payer_with_rate) + assert_equal 20, @registrar.vat_rate + end + + def test_requires_vat_rate_when_foreign_vat_payer_without_number + Registry.instance.stub(:legal_address_country, Country.new('GB')) do + @registrar.vat_no = nil + @registrar.validate + assert @registrar.invalid? + end + end + + def test_rejects_vat_rate_when_foreign_vat_payer_with_number + Registry.instance.stub(:legal_address_country, Country.new('GB')) do + @registrar.vat_no = 'US1' + @registrar.vat_rate = 1 + @registrar.validate + assert @registrar.invalid? + end + end end diff --git a/test/test_helper.rb b/test/test_helper.rb index 7bde0991d..64cef7e0f 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -14,6 +14,8 @@ require 'webmock/minitest' Setting.address_processing = false +Setting.registry_country_code = 'US' + class ActiveSupport::TestCase include FactoryBot::Syntax::Methods From 6a0122613854abeee5f25c579a8abb292e60262d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 27 Nov 2017 11:11:06 +0200 Subject: [PATCH 10/87] Update VAT #623 --- app/models/invoice.rb | 5 ++- app/models/registrar.rb | 38 +++++++++++++++---- test/fixtures/invoices.yml | 8 ++++ .../admin/invoices/invoice_details_test.rb | 19 ++++++++++ .../admin/registrars/show_registrar_test.rb | 2 +- .../invoices/invoice_details_test.rb | 19 ++++++++++ test/models/invoice_test.rb | 31 +++++++++++++-- test/models/registrar_test.rb | 24 +++++++++--- 8 files changed, 126 insertions(+), 20 deletions(-) create mode 100644 test/integration/admin/invoices/invoice_details_test.rb create mode 100644 test/integration/registrar/invoices/invoice_details_test.rb diff --git a/app/models/invoice.rb b/app/models/invoice.rb index 37eb8a72c..90fa380e1 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -29,6 +29,7 @@ class Invoice < ActiveRecord::Base validates :invoice_type, :due_date, :currency, :seller_name, :seller_iban, :buyer_name, :invoice_items, :vat_rate, presence: true + validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 99 }, allow_nil: true before_create :set_invoice_number, :check_vat @@ -106,12 +107,12 @@ class Invoice < ActiveRecord::Base def buyer_country Country.new(buyer_country_code) end - + # order is used for directo/banklink description def order "Order nr. #{number}" end - + def pdf(html) kit = PDFKit.new(html) kit.to_pdf diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 393bd56f2..dd7a7a9f5 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -1,3 +1,21 @@ +class VATRateType < ActiveRecord::Type::Value + def type_cast_from_user(value) + if value.blank? + nil + else + super + end + end + + def type_cast_from_database(value) + BigDecimal.new(value) * 100 if value + end + + def type_cast_for_database(value) + BigDecimal.new(value) / 100 if value + end +end + class Registrar < ActiveRecord::Base include Versions # version/registrar_version.rb @@ -23,16 +41,9 @@ class Registrar < ActiveRecord::Base validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 99 }, allow_nil: true validate :forbidden_codes + attribute :vat_rate, VATRateType.new after_initialize :set_defaults - before_save do - self.vat_rate = vat_rate / 100.0 if vat_rate - end - - after_find do |registrar| - registrar.vat_rate = registrar.vat_rate * 100 if registrar.vat_rate - end - def forbidden_codes return true unless ['CID'].include? code errors.add(:code, I18n.t(:forbidden_code)) @@ -87,6 +98,16 @@ class Registrar < ActiveRecord::Base # rubocop:disable Metrics/MethodLength # rubocop:disable Metrics/AbcSize def issue_prepayment_invoice(amount, description = nil) + vat_rate = if local_vat_payer? + Setting.registry_vat_prc + else + if vat_no.blank? + self.vat_rate + else + nil + end + end + invoices.create( invoice_type: 'DEB', due_date: (Time.zone.now.to_date + Setting.days_to_keep_invoices_active.days).end_of_day, @@ -94,6 +115,7 @@ class Registrar < ActiveRecord::Base description: description, currency: 'EUR', vat_rate: Setting.registry_vat_prc, + vat_rate: vat_rate, seller_name: Setting.registry_juridical_name, seller_reg_no: Setting.registry_reg_no, seller_iban: Setting.registry_iban, diff --git a/test/fixtures/invoices.yml b/test/fixtures/invoices.yml index 04cedc153..bdd8ea033 100644 --- a/test/fixtures/invoices.yml +++ b/test/fixtures/invoices.yml @@ -30,3 +30,11 @@ outstanding: overdue: <<: *DEFAULTS due_date: <%= Date.parse '2010-07-03' %> + +with_vat: + <<: *DEFAULTS + vat_rate: 0.2 + +without_vat: + <<: *DEFAULTS + vat_rate: null diff --git a/test/integration/admin/invoices/invoice_details_test.rb b/test/integration/admin/invoices/invoice_details_test.rb new file mode 100644 index 000000000..8a67771ea --- /dev/null +++ b/test/integration/admin/invoices/invoice_details_test.rb @@ -0,0 +1,19 @@ +require 'test_helper' + +class InvoiceDetailsTest < ActionDispatch::IntegrationTest + def setup + login_as users(:admin) + end + + def test_with_vat + invoice = invoices(:with_vat) + visit admin_invoice_path(invoice) + assert_selector '.total', text: 'VAT' + end + + def test_without_vat + invoice = invoices(:without_vat) + visit admin_invoice_path(invoice) + assert_no_selector '.total', text: 'VAT' + end +end diff --git a/test/integration/admin/registrars/show_registrar_test.rb b/test/integration/admin/registrars/show_registrar_test.rb index f13748191..d4ed927bd 100644 --- a/test/integration/admin/registrars/show_registrar_test.rb +++ b/test/integration/admin/registrars/show_registrar_test.rb @@ -17,7 +17,7 @@ class ShowRegistrarTest < ActionDispatch::IntegrationTest assert_text 'Language English' end - def test_vat_number + def test_vat_no assert_text 'US12345' end diff --git a/test/integration/registrar/invoices/invoice_details_test.rb b/test/integration/registrar/invoices/invoice_details_test.rb new file mode 100644 index 000000000..d3961faf3 --- /dev/null +++ b/test/integration/registrar/invoices/invoice_details_test.rb @@ -0,0 +1,19 @@ +require 'test_helper' + +class InvoiceDetailsTest < ActionDispatch::IntegrationTest + def setup + login_as users(:api) + end + + def test_with_vat + invoice = invoices(:with_vat) + visit registrar_invoice_path(invoice) + assert_selector '.total', text: 'VAT' + end + + def test_without_vat + invoice = invoices(:without_vat) + visit registrar_invoice_path(invoice) + assert_no_selector '.total', text: 'VAT' + end +end diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index c89c37626..770a88590 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -1,9 +1,34 @@ require 'test_helper' class InvoiceTest < ActiveSupport::TestCase + def setup + @invoice = invoices(:valid) + end + + def test_valid + assert @invoice.valid? + end + def test_invalid_without_vat_rate - invoice = Invoice.new(vat_rate: nil) - invoice.validate - assert invoice.errors.added?(:vat_rate, :blank) + @invoice.vat_rate = nil + assert @invoice.invalid? + end + + def test_allows_absent_vat_rate + @invoice.vat_rate = nil + @invoice.validate + assert @invoice.valid? + end + + def test_rejects_negative_vat_rate + @invoice.vat_rate = -1 + @invoice.validate + assert @invoice.invalid? + end + + def test_rejects_vat_rate_greater_than_max + @invoice.vat_rate = 100 + @invoice.validate + assert @invoice.invalid? end end diff --git a/test/models/registrar_test.rb b/test/models/registrar_test.rb index 3ace52ba7..96328d38c 100644 --- a/test/models/registrar_test.rb +++ b/test/models/registrar_test.rb @@ -39,7 +39,7 @@ class RegistrarTest < ActiveSupport::TestCase assert_equal 'de', registrar.language end - def test_rejects_vat_number_when_local_vat_payer + def test_rejects_vat_no_when_local_vat_payer Registry.instance.stub(:legal_address_country, Country.new('US')) do @registrar.vat_no = 'US1' @registrar.validate @@ -67,11 +67,6 @@ class RegistrarTest < ActiveSupport::TestCase assert @registrar.invalid? end - def test_converts_integer_vat_rate_to_fractional - @registrar = registrars(:foreign_vat_payer_with_rate) - assert_equal 20, @registrar.vat_rate - end - def test_requires_vat_rate_when_foreign_vat_payer_without_number Registry.instance.stub(:legal_address_country, Country.new('GB')) do @registrar.vat_no = nil @@ -88,4 +83,21 @@ class RegistrarTest < ActiveSupport::TestCase assert @registrar.invalid? end end + + def test_serializes_and_deserializes_vat_rate + valid_attributes = registrars(:valid).attributes.except('id').merge({ name: 'uniq1', + reg_no: 'uniq1', + code: 'uniq1', + country_code: 'GB' }) + registrar = Registrar.new(valid_attributes) + registrar.vat_rate = 55 + registrar.save! + registrar.reload + assert_equal 55, registrar.vat_rate + end + + def test_treats_empty_vat_rate_as_absent + @registrar.vat_rate = '' + assert_nil @registrar.vat_rate + end end From 37cc05e44c3a10c49d710c86a53a7df24cdf4676 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 09:46:17 +0200 Subject: [PATCH 11/87] Remove `registrars.vat` DB column #623 --- db/migrate/20180228074442_remove_registrars_vat.rb | 5 +++++ db/structure.sql | 3 ++- doc/models_complete.svg | 1 - lib/tasks/import.rake | 1 - 4 files changed, 7 insertions(+), 3 deletions(-) create mode 100644 db/migrate/20180228074442_remove_registrars_vat.rb diff --git a/db/migrate/20180228074442_remove_registrars_vat.rb b/db/migrate/20180228074442_remove_registrars_vat.rb new file mode 100644 index 000000000..eb2bd44f1 --- /dev/null +++ b/db/migrate/20180228074442_remove_registrars_vat.rb @@ -0,0 +1,5 @@ +class RemoveRegistrarsVat < ActiveRecord::Migration + def change + remove_column :registrars, :vat, :boolean + end +end diff --git a/db/structure.sql b/db/structure.sql index ba040e7cd..46b433aa7 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -2432,7 +2432,6 @@ CREATE TABLE registrars ( code character varying, website character varying, accounting_customer_code character varying NOT NULL, - vat boolean, legacy_id integer, reference_no character varying, test_registrar boolean DEFAULT false, @@ -5121,3 +5120,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180228070102'); INSERT INTO schema_migrations (version) VALUES ('20180228070431'); +INSERT INTO schema_migrations (version) VALUES ('20180228074442'); + diff --git a/doc/models_complete.svg b/doc/models_complete.svg index 12fb078a4..3985846b3 100644 --- a/doc/models_complete.svg +++ b/doc/models_complete.svg @@ -1556,7 +1556,6 @@ zip :string code :string url :string -vat :boolean legacy_id :integer reference_no :string diff --git a/lib/tasks/import.rake b/lib/tasks/import.rake index c3b67ba88..6c4be1e53 100644 --- a/lib/tasks/import.rake +++ b/lib/tasks/import.rake @@ -88,7 +88,6 @@ namespace :import do zip: x.postalcode.try(:strip), url: x.url.try(:strip), accounting_customer_code: x.directo_handle.try(:strip), - vat: x.vat, legacy_id: x.id, creator_str: user, updator_str: user, From ea305618a57bd503f1378f6a445964a946342d46 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 09:53:53 +0200 Subject: [PATCH 12/87] Add Registry model #623 --- app/models/registrar.rb | 3 +-- app/models/registry.rb | 11 +++++++++++ config/application.rb | 2 +- test/models/registry_test.rb | 16 ++++++++++++++++ 4 files changed, 29 insertions(+), 3 deletions(-) create mode 100644 app/models/registry.rb create mode 100644 test/models/registry_test.rb diff --git a/app/models/registrar.rb b/app/models/registrar.rb index dd7a7a9f5..487bc513a 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -99,7 +99,7 @@ class Registrar < ActiveRecord::Base # rubocop:disable Metrics/AbcSize def issue_prepayment_invoice(amount, description = nil) vat_rate = if local_vat_payer? - Setting.registry_vat_prc + Registry.instance.vat_rate else if vat_no.blank? self.vat_rate @@ -114,7 +114,6 @@ class Registrar < ActiveRecord::Base payment_term: 'prepayment', description: description, currency: 'EUR', - vat_rate: Setting.registry_vat_prc, vat_rate: vat_rate, seller_name: Setting.registry_juridical_name, seller_reg_no: Setting.registry_reg_no, diff --git a/app/models/registry.rb b/app/models/registry.rb new file mode 100644 index 000000000..162ab4d71 --- /dev/null +++ b/app/models/registry.rb @@ -0,0 +1,11 @@ +class Registry + include Singleton + + def vat_rate + Setting.registry_vat_prc + end + + def legal_address_country + Country.new(Setting.registry_country_code) + end +end diff --git a/config/application.rb b/config/application.rb index 57106c8ea..04d697f55 100644 --- a/config/application.rb +++ b/config/application.rb @@ -15,7 +15,7 @@ require 'rails/all' # you've limited to :test, :development, or :production. Bundler.require(*Rails.groups) -module Registry +module DomainNameRegistry class Application < Rails::Application # Settings in config/environments/* take precedence over those specified here. # Application configuration should go into files in config/initializers diff --git a/test/models/registry_test.rb b/test/models/registry_test.rb new file mode 100644 index 000000000..2f25686c6 --- /dev/null +++ b/test/models/registry_test.rb @@ -0,0 +1,16 @@ +require 'test_helper' + +class RegistryTest < ActiveSupport::TestCase + def setup + @registry = Registry.send(:new) + end + + def test_implements_singleton + assert_equal Registry.instance.object_id, Registry.instance.object_id + end + + def test_vat_rate + Setting.registry_vat_prc = 0.2 + assert 0.2, @registry.vat_rate + end +end From 6393e4ba6fe8ea5e05abba95c6c8f8f48b626aee Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 10:42:41 +0200 Subject: [PATCH 13/87] Use standard page header #623 --- app/views/admin/registrars/edit.html.erb | 13 +++++--- app/views/admin/registrars/index.html.erb | 16 +++++++--- app/views/admin/registrars/new.html.erb | 9 +++++- app/views/admin/registrars/show.html.erb | 38 ++++++++++++++++------- config/locales/admin/registrars.en.yml | 10 ++++++ 5 files changed, 66 insertions(+), 20 deletions(-) diff --git a/app/views/admin/registrars/edit.html.erb b/app/views/admin/registrars/edit.html.erb index 71d6bd882..205ba6b28 100644 --- a/app/views/admin/registrars/edit.html.erb +++ b/app/views/admin/registrars/edit.html.erb @@ -1,5 +1,10 @@ -<% content_for :actions do %> - <%= link_to(t(:back_to_registrar), [:admin, @registrar], class: 'btn btn-default') %> -<% end %> -<%= render 'shared/title', name: "#{t(:edit)}: #{@registrar.name}" %> +
    +
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. +
  2. <%= link_to @registrar.name, admin_registrar_path(@registrar) %>
    3. +
+ +
+

<%= t '.header' %>

+
+ <%= render 'form' %> diff --git a/app/views/admin/registrars/index.html.erb b/app/views/admin/registrars/index.html.erb index 5533a6f07..49bf7eae8 100644 --- a/app/views/admin/registrars/index.html.erb +++ b/app/views/admin/registrars/index.html.erb @@ -1,7 +1,15 @@ -<% content_for :actions do %> - <%= link_to(t('.new_btn'), new_admin_registrar_path, class: 'btn btn-primary') %> -<% end %> -<%= render 'shared/title', name: t(:registrars) %> +
+
+
+

<%= t '.header' %>

+
+ +
+ <%= link_to t('.new_btn'), new_admin_registrar_path, class: 'btn btn-primary' %> +
+
+
+
diff --git a/app/views/admin/registrars/new.html.erb b/app/views/admin/registrars/new.html.erb index 70154d217..e813bdd57 100644 --- a/app/views/admin/registrars/new.html.erb +++ b/app/views/admin/registrars/new.html.erb @@ -1,2 +1,9 @@ -<%= render 'shared/title', name: t(:new_registrar) %> +
    +
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. +
+ +
+

<%= t '.header' %>

+
+ <%= render 'form' %> diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index c0a596b27..cff8ded47 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -1,15 +1,31 @@ <% registrar = RegistrarPresenter.new(registrar: @registrar, view: self) %> -<% content_for :actions do %> - <%= link_to(t(:edit), edit_admin_registrar_path(@registrar), class: 'btn btn-primary') %> - <%= link_to(t(:delete), admin_registrar_path(@registrar), method: :delete, data: { confirm: t(:are_you_sure) }, class: 'btn btn-danger') %> -<% end %> -<% content_for :page_name do %> - <%= @registrar.name %> - <% if @registrar.test_registrar? %> - (test) - <% end %> -<% end %> -<%= render 'shared/title', name: @registrar.name %> + +
    +
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. +
+ +
+
+
+

+ <%= @registrar.name %> + <% if @registrar.test_registrar? %> + (test) + <% end %> +

+
+ +
+ <%= link_to t('.edit_btn'), edit_admin_registrar_path(@registrar), + class: 'btn btn-primary' %> + <%= link_to t('.delete_btn'), admin_registrar_path(@registrar), + method: :delete, + data: { confirm: t('.delete_btn_confirm') }, + class: 'btn btn-default' %> +
+
+
+ <% if @registrar.errors.any? %> <% @registrar.errors.each do |attr, err| %> <%= err %> diff --git a/config/locales/admin/registrars.en.yml b/config/locales/admin/registrars.en.yml index e805b19c6..ec8d39b63 100644 --- a/config/locales/admin/registrars.en.yml +++ b/config/locales/admin/registrars.en.yml @@ -2,9 +2,16 @@ en: admin: registrars: index: + header: Registrars new_btn: New registrar + new: + header: New registrar + show: + edit_btn: Edit + delete_btn: Delete + delete_btn_confirm: Are you sure you want delete registrar? new_api_use_btn: New API user active: Active api_users: API users @@ -12,6 +19,9 @@ en: preferences: header: Preferences + edit: + header: Edit registrar + billing: header: Billing From 64325d7535a5b272c31d49eb3f4b1627c3556c75 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 10:43:15 +0200 Subject: [PATCH 14/87] Remove inline CSS #623 --- app/views/admin/registrars/show.html.erb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index cff8ded47..ef75e4a64 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -10,7 +10,7 @@

<%= @registrar.name %> <% if @registrar.test_registrar? %> - (test) + (test) <% end %>

From 8dcac194e27c1431109a2842f122737a951f39a4 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 10:43:30 +0200 Subject: [PATCH 15/87] Remove unused translations #623 --- config/locales/en.yml | 3 --- 1 file changed, 3 deletions(-) diff --git a/config/locales/en.yml b/config/locales/en.yml index e9b8316d6..f92c525ce 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -314,9 +314,6 @@ en: edit_statuses: 'Edit statuses' history: 'History' - new_registrar: 'New registrar' - registrar_details: 'Registrar details' - edit_registrar: 'Edit registrar' back_to_registrar: 'Back to registrar' registrar_deleted: 'Registrar deleted' failed_to_delete_registrar: 'Failed to delete registrar' From 70fb91c812196dc46a01bcf5f54e4a3698d3a6cf Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 10:44:59 +0200 Subject: [PATCH 16/87] Add HTML validation #623 --- app/views/admin/registrars/form/_billing.html.erb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/views/admin/registrars/form/_billing.html.erb b/app/views/admin/registrars/form/_billing.html.erb index 5bc87a93d..ff67232de 100644 --- a/app/views/admin/registrars/form/_billing.html.erb +++ b/app/views/admin/registrars/form/_billing.html.erb @@ -23,7 +23,7 @@
- <%= f.number_field :vat_rate, class: 'form-control' %> + <%= f.number_field :vat_rate, min: 0, max: 99, class: 'form-control' %>
%
From f449f1c107746018041c8d25b9e12dfbe89cf062 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 10:49:20 +0200 Subject: [PATCH 17/87] Change `registrars.country_code` DB column to NOT NULL #623 --- ...80228084818_change_registrars_country_code_to_not_null.rb | 5 +++++ db/structure.sql | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb diff --git a/db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb b/db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb new file mode 100644 index 000000000..16a66e989 --- /dev/null +++ b/db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb @@ -0,0 +1,5 @@ +class ChangeRegistrarsCountryCodeToNotNull < ActiveRecord::Migration + def change + change_column_null :registrars, :country_code, false + end +end diff --git a/db/structure.sql b/db/structure.sql index 46b433aa7..00eac536d 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -2424,7 +2424,7 @@ CREATE TABLE registrars ( phone character varying, email character varying, billing_email character varying, - country_code character varying, + country_code character varying NOT NULL, state character varying, city character varying, street character varying, @@ -5122,3 +5122,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180228070431'); INSERT INTO schema_migrations (version) VALUES ('20180228074442'); +INSERT INTO schema_migrations (version) VALUES ('20180228084818'); + From 9e4fd48c70225b2229ccdba596f3a32b44b268a6 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 11:10:08 +0200 Subject: [PATCH 18/87] Fix style #623 --- app/controllers/admin/registrars_controller.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/admin/registrars_controller.rb b/app/controllers/admin/registrars_controller.rb index eb9ae9bd7..df2a43e1b 100644 --- a/app/controllers/admin/registrars_controller.rb +++ b/app/controllers/admin/registrars_controller.rb @@ -80,7 +80,7 @@ module Admin :test_registrar, :accounting_customer_code, :language, - :vat_rate,) + :vat_rate) end end end From b2a38ae4e9e5decd9a215bd36a3be694f5044317 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 11:36:49 +0200 Subject: [PATCH 19/87] Re-order arguments #623 --- app/controllers/admin/registrars_controller.rb | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/controllers/admin/registrars_controller.rb b/app/controllers/admin/registrars_controller.rb index df2a43e1b..34eb49598 100644 --- a/app/controllers/admin/registrars_controller.rb +++ b/app/controllers/admin/registrars_controller.rb @@ -65,7 +65,6 @@ module Admin def registrar_params params.require(:registrar).permit(:name, :reg_no, - :vat_no, :street, :city, :state, @@ -78,9 +77,11 @@ module Admin :billing_email, :code, :test_registrar, + :vat_no, + :vat_rate, :accounting_customer_code, - :language, - :vat_rate) + :billing_email, + :language) end end end From b81263ee3d54c0f2e356a34b1461c033a19fc5b6 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 11:48:19 +0200 Subject: [PATCH 20/87] Remove unused fixture #623 --- test/fixtures/registrars.yml | 7 ------- 1 file changed, 7 deletions(-) diff --git a/test/fixtures/registrars.yml b/test/fixtures/registrars.yml index b5715a0a5..0026f323b 100644 --- a/test/fixtures/registrars.yml +++ b/test/fixtures/registrars.yml @@ -16,10 +16,3 @@ goodnames: country_code: US accounting_customer_code: goodnames language: en - -foreign_vat_payer_with_rate: - <<: *DEFAULTS - name: 3 - reg_no: 3 - code: 3 - vat_rate: 0.2 From e4b730caf715235e5536f59492bf5d88b7f4914d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 11:50:52 +0200 Subject: [PATCH 21/87] Fix invalid fixture #623 --- test/models/registrar_test.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/models/registrar_test.rb b/test/models/registrar_test.rb index 96328d38c..5d0894e0e 100644 --- a/test/models/registrar_test.rb +++ b/test/models/registrar_test.rb @@ -85,7 +85,7 @@ class RegistrarTest < ActiveSupport::TestCase end def test_serializes_and_deserializes_vat_rate - valid_attributes = registrars(:valid).attributes.except('id').merge({ name: 'uniq1', + valid_attributes = registrars(:bestnames).attributes.except('id').merge({ name: 'uniq1', reg_no: 'uniq1', code: 'uniq1', country_code: 'GB' }) From 07d58d5b9a5fd6dc49872c466e343c633258cb8e Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 28 Feb 2018 11:52:20 +0200 Subject: [PATCH 22/87] Convert spec to test #623 --- spec/models/registrar_spec.rb | 1 - test/models/registrar_test.rb | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/spec/models/registrar_spec.rb b/spec/models/registrar_spec.rb index 4a8110d45..301d59bcb 100644 --- a/spec/models/registrar_spec.rb +++ b/spec/models/registrar_spec.rb @@ -10,7 +10,6 @@ describe Registrar do @registrar.valid? @registrar.errors.full_messages.should include(*[ 'Contact e-mail is missing', - 'Country code is missing', 'Name is missing', 'Reg no is missing', 'Code is missing' diff --git a/test/models/registrar_test.rb b/test/models/registrar_test.rb index 5d0894e0e..057c34f62 100644 --- a/test/models/registrar_test.rb +++ b/test/models/registrar_test.rb @@ -15,9 +15,8 @@ class RegistrarTest < ActiveSupport::TestCase assert @registrar.invalid? end - def test_requires_country_code + def test_invalid_without_country_code @registrar.country_code = nil - @registrar.validate assert @registrar.invalid? end From d53a7f9c47587e5592483aea3a9b1d9b78cb998d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 8 Mar 2018 14:22:28 +0200 Subject: [PATCH 23/87] Revert "Extract partial" This reverts commit 955dc10 --- app/views/admin/registrars/_billing.html.erb | 20 -------------------- app/views/admin/registrars/show.html.erb | 19 ++++++++++++++++++- config/locales/admin/registrars.en.yml | 3 --- config/locales/en.yml | 2 +- 4 files changed, 19 insertions(+), 25 deletions(-) delete mode 100644 app/views/admin/registrars/_billing.html.erb diff --git a/app/views/admin/registrars/_billing.html.erb b/app/views/admin/registrars/_billing.html.erb deleted file mode 100644 index 7ba778d08..000000000 --- a/app/views/admin/registrars/_billing.html.erb +++ /dev/null @@ -1,20 +0,0 @@ -
-
- <%= t '.header' %> -
-
-
-
<%= Registrar.human_attribute_name :vat_no %>
-
<%= registrar.vat_no %>
- -
<%= Registrar.human_attribute_name :vat_rate %>
-
<%= number_to_percentage registrar.vat_rate, precision: 1 %>
- -
<%= Registrar.human_attribute_name :accounting_customer_code %>
-
<%= registrar.accounting_customer_code %>
- -
<%= Registrar.human_attribute_name :billing_email %>
-
<%= registrar.billing_email %>
-
-
-
diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index ef75e4a64..2b1515fa0 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -57,6 +57,12 @@
<%= @registrar.reg_no %>
+
+ <%= t(:vat_no) %> +
+
+ <%= @registrar.vat_no %> +
<%= t(:reference_no) %>
@@ -81,6 +87,12 @@
<%= @registrar.website %>
+
+ <%= Registrar.human_attribute_name :accounting_customer_code %> +
+
+ <%= @registrar.accounting_customer_code %> +
@@ -124,10 +136,15 @@
<%= @registrar.billing_address %>
+
+ <%= t(:billing_email) %> +
+
+ <%= @registrar.billing_email %> +
- <%= render 'billing', registrar: @registrar %> <%= render 'admin/registrars/show/preferences', registrar: registrar %>
diff --git a/config/locales/admin/registrars.en.yml b/config/locales/admin/registrars.en.yml index ec8d39b63..249a3b70a 100644 --- a/config/locales/admin/registrars.en.yml +++ b/config/locales/admin/registrars.en.yml @@ -22,9 +22,6 @@ en: edit: header: Edit registrar - billing: - header: Billing - create: created: Registrar has been successfully created not_created: Unable to create registrar diff --git a/config/locales/en.yml b/config/locales/en.yml index 4845d7309..65c3d658d 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -199,6 +199,7 @@ en: alg: 'Algorithm' public_key: 'Public key' registrar: + billing_email: 'Billing e-mail' phone: 'Contact phone' email: 'Contact e-mail' state: 'State / Province' @@ -777,5 +778,4 @@ en: unit: € attributes: - vat_no: VAT number vat_rate: VAT rate From 6949b684eb524640e193e9b84458b64f4b0b3a0b Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 8 Mar 2018 14:22:39 +0200 Subject: [PATCH 24/87] Revert "Remove unused HTML" This reverts commit d2e84d2 --- app/views/admin/registrars/show.html.erb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index 2b1515fa0..755143ff4 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -150,7 +150,7 @@
-
+
<%= t('.api_users') %> @@ -190,7 +190,7 @@
-
+
<%= t(:white_ips) %> From fcc89769cf3fbdcf8c949e8fd39eac094d629073 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 8 Mar 2018 14:23:26 +0200 Subject: [PATCH 25/87] Revert "Use human_attribute_name" This reverts commit 4a41984 --- app/views/registrant/registrars/show.haml | 2 +- app/views/registrar/invoices/partials/_seller.haml | 2 +- app/views/registrar/invoices/pdf.haml | 2 +- config/locales/en.yml | 1 + 4 files changed, 4 insertions(+), 3 deletions(-) diff --git a/app/views/registrant/registrars/show.haml b/app/views/registrant/registrars/show.haml index 6d4aa9e79..e0da73ee3 100644 --- a/app/views/registrant/registrars/show.haml +++ b/app/views/registrant/registrars/show.haml @@ -19,7 +19,7 @@ %dt= t(:reg_no) %dd= @registrar.reg_no - %dt= Registrar.human_attribute_name :vat_no + %dt= t(:vat_no) %dd= @registrar.vat_no %dt= t(:id) diff --git a/app/views/registrar/invoices/partials/_seller.haml b/app/views/registrar/invoices/partials/_seller.haml index 9c5387e16..0008c2f13 100644 --- a/app/views/registrar/invoices/partials/_seller.haml +++ b/app/views/registrar/invoices/partials/_seller.haml @@ -16,7 +16,7 @@ %dt= t(:swift) %dd= @invoice.seller_swift - %dt= Registrar.human_attribute_name :vat_no + %dt= t(:vat_no) %dd= @invoice.seller_vat_no %dt= t(:address) diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index 6f92c4a8a..1622613f5 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -261,7 +261,7 @@ %br = "#{t('reg_no')} #{@invoice.seller_reg_no}" %br - = "#{Registrar.human_attribute_name :vat_no} #{@invoice.seller_vat_no}" + = "#{t('vat_no')} #{@invoice.seller_vat_no}" .col-md-3.left = @invoice.seller_phone diff --git a/config/locales/en.yml b/config/locales/en.yml index 65c3d658d..099a9fbbf 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -315,6 +315,7 @@ en: edit_statuses: 'Edit statuses' history: 'History' + vat_no: 'VAT no' back_to_registrar: 'Back to registrar' registrar_deleted: 'Registrar deleted' failed_to_delete_registrar: 'Failed to delete registrar' From 6406e6116e421334a59f0fd4a5ed4516e12cf9f6 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 8 Mar 2018 14:25:54 +0200 Subject: [PATCH 26/87] Revert "Use standard page header" This reverts commit 6393e4b --- app/views/admin/registrars/edit.html.erb | 13 +++----- app/views/admin/registrars/index.html.erb | 16 +++------- app/views/admin/registrars/new.html.erb | 9 +----- app/views/admin/registrars/show.html.erb | 38 +++++++---------------- config/locales/admin/registrars.en.yml | 10 ------ 5 files changed, 20 insertions(+), 66 deletions(-) diff --git a/app/views/admin/registrars/edit.html.erb b/app/views/admin/registrars/edit.html.erb index 205ba6b28..71d6bd882 100644 --- a/app/views/admin/registrars/edit.html.erb +++ b/app/views/admin/registrars/edit.html.erb @@ -1,10 +1,5 @@ -
    -
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. -
  2. <%= link_to @registrar.name, admin_registrar_path(@registrar) %>
    3. -
- -
-

<%= t '.header' %>

-
- +<% content_for :actions do %> + <%= link_to(t(:back_to_registrar), [:admin, @registrar], class: 'btn btn-default') %> +<% end %> +<%= render 'shared/title', name: "#{t(:edit)}: #{@registrar.name}" %> <%= render 'form' %> diff --git a/app/views/admin/registrars/index.html.erb b/app/views/admin/registrars/index.html.erb index 49bf7eae8..5533a6f07 100644 --- a/app/views/admin/registrars/index.html.erb +++ b/app/views/admin/registrars/index.html.erb @@ -1,15 +1,7 @@ -
-
-
-

<%= t '.header' %>

-
- -
- <%= link_to t('.new_btn'), new_admin_registrar_path, class: 'btn btn-primary' %> -
-
-
- +<% content_for :actions do %> + <%= link_to(t('.new_btn'), new_admin_registrar_path, class: 'btn btn-primary') %> +<% end %> +<%= render 'shared/title', name: t(:registrars) %>
diff --git a/app/views/admin/registrars/new.html.erb b/app/views/admin/registrars/new.html.erb index e813bdd57..70154d217 100644 --- a/app/views/admin/registrars/new.html.erb +++ b/app/views/admin/registrars/new.html.erb @@ -1,9 +1,2 @@ -
    -
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. -
- -
-

<%= t '.header' %>

-
- +<%= render 'shared/title', name: t(:new_registrar) %> <%= render 'form' %> diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index 755143ff4..c09602985 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -1,31 +1,15 @@ <% registrar = RegistrarPresenter.new(registrar: @registrar, view: self) %> - -
    -
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. -
- -
-
-
-

- <%= @registrar.name %> - <% if @registrar.test_registrar? %> - (test) - <% end %> -

-
- -
- <%= link_to t('.edit_btn'), edit_admin_registrar_path(@registrar), - class: 'btn btn-primary' %> - <%= link_to t('.delete_btn'), admin_registrar_path(@registrar), - method: :delete, - data: { confirm: t('.delete_btn_confirm') }, - class: 'btn btn-default' %> -
-
-
- +<% content_for :actions do %> + <%= link_to(t(:edit), edit_admin_registrar_path(@registrar), class: 'btn btn-primary') %> + <%= link_to(t(:delete), admin_registrar_path(@registrar), method: :delete, data: { confirm: t(:are_you_sure) }, class: 'btn btn-danger') %> +<% end %> +<% content_for :page_name do %> + <%= @registrar.name %> + <% if @registrar.test_registrar? %> + (test) + <% end %> +<% end %> +<%= render 'shared/title', name: @registrar.name %> <% if @registrar.errors.any? %> <% @registrar.errors.each do |attr, err| %> <%= err %> diff --git a/config/locales/admin/registrars.en.yml b/config/locales/admin/registrars.en.yml index 249a3b70a..3d269f64b 100644 --- a/config/locales/admin/registrars.en.yml +++ b/config/locales/admin/registrars.en.yml @@ -2,16 +2,9 @@ en: admin: registrars: index: - header: Registrars new_btn: New registrar - new: - header: New registrar - show: - edit_btn: Edit - delete_btn: Delete - delete_btn_confirm: Are you sure you want delete registrar? new_api_use_btn: New API user active: Active api_users: API users @@ -19,9 +12,6 @@ en: preferences: header: Preferences - edit: - header: Edit registrar - create: created: Registrar has been successfully created not_created: Unable to create registrar From bdeb266d279ba3021f725c92c3d81dd1d06d467f Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 8 Mar 2018 14:28:21 +0200 Subject: [PATCH 27/87] Revert "Remove unused translations" This reverts commit 8dcac19 --- config/locales/en.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/config/locales/en.yml b/config/locales/en.yml index 099a9fbbf..6feb4c353 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -315,7 +315,10 @@ en: edit_statuses: 'Edit statuses' history: 'History' + new_registrar: 'New registrar' + registrar_details: 'Registrar details' vat_no: 'VAT no' + edit_registrar: 'Edit registrar' back_to_registrar: 'Back to registrar' registrar_deleted: 'Registrar deleted' failed_to_delete_registrar: 'Failed to delete registrar' From 810c34ffaf020f01400de7ce157ff255a070e889 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Fri, 9 Mar 2018 08:24:24 +0200 Subject: [PATCH 28/87] Revert "Change `registrars.country_code` DB column to NOT NULL" This reverts commit f449f1c --- ...80228084818_change_registrars_country_code_to_not_null.rb | 5 ----- db/structure.sql | 4 +--- 2 files changed, 1 insertion(+), 8 deletions(-) delete mode 100644 db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb diff --git a/db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb b/db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb deleted file mode 100644 index 16a66e989..000000000 --- a/db/migrate/20180228084818_change_registrars_country_code_to_not_null.rb +++ /dev/null @@ -1,5 +0,0 @@ -class ChangeRegistrarsCountryCodeToNotNull < ActiveRecord::Migration - def change - change_column_null :registrars, :country_code, false - end -end diff --git a/db/structure.sql b/db/structure.sql index 15f92fe2b..fbbb4a7d0 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -2147,7 +2147,7 @@ CREATE TABLE registrars ( phone character varying, email character varying, billing_email character varying, - country_code character varying NOT NULL, + country_code character varying, state character varying, city character varying, street character varying, @@ -4670,8 +4670,6 @@ INSERT INTO schema_migrations (version) VALUES ('20180228070431'); INSERT INTO schema_migrations (version) VALUES ('20180228074442'); -INSERT INTO schema_migrations (version) VALUES ('20180228084818'); - INSERT INTO schema_migrations (version) VALUES ('20180306180401'); INSERT INTO schema_migrations (version) VALUES ('20180306181538'); From a5869e0ecad1927b47d4e12188cc57508870029e Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 20 Nov 2017 08:10:59 +0200 Subject: [PATCH 29/87] Refactor registrars #765 --- .../admin/registrars_controller.rb | 25 ++-- app/models/registrar.rb | 56 ++++----- app/views/admin/registrars/_billing.html.erb | 17 +++ app/views/admin/registrars/_form.html.erb | 7 +- app/views/admin/registrars/edit.html.erb | 13 +- app/views/admin/registrars/index.html.erb | 16 ++- app/views/admin/registrars/new.html.erb | 9 +- app/views/admin/registrars/show.html.erb | 61 +++++---- app/views/registrant/registrars/show.haml | 2 +- .../registrar/invoices/partials/_seller.haml | 2 +- app/views/registrar/invoices/pdf.haml | 2 +- config/locales/admin/registrars.en.yml | 18 ++- config/locales/en.yml | 11 +- config/locales/registrars.en.yml | 8 ++ ...53424_add_registrars_unique_constraints.rb | 19 +++ ...0180309053921_remove_registrars_indexes.rb | 6 + ...510_add_registrars_not_null_constraints.rb | 9 ++ db/structure.sql | 62 ++++++--- lib/tasks/import.rake | 2 +- spec/models/registrar_spec.rb | 118 ------------------ spec/requests/admin/registrars/update_spec.rb | 42 ------- spec/routing/admin/registrars_routing_spec.rb | 33 ----- .../admin/registrars/_form.html.erb_spec.rb | 19 --- spec/views/admin/registrars/show.haml_spec.rb | 19 --- .../admin/registrars/update_test.rb | 30 ----- test/fixtures/accounts.yml | 6 + test/fixtures/registrars.yml | 13 ++ .../admin/registrars/delete_test.rb | 18 +++ .../admin/registrars/edit_registrar_test.rb | 17 --- .../integration/admin/registrars/edit_test.rb | 69 ++++++++++ .../admin/registrars/new_registrar_test.rb | 23 ---- test/integration/admin/registrars/new_test.rb | 49 ++++++++ test/models/registrar/code_test.rb | 29 +++++ test/models/registrar/delete_test.rb | 37 ++++++ test/models/registrar_test.rb | 31 ++++- 35 files changed, 471 insertions(+), 427 deletions(-) create mode 100644 app/views/admin/registrars/_billing.html.erb create mode 100644 config/locales/registrars.en.yml create mode 100644 db/migrate/20180309053424_add_registrars_unique_constraints.rb create mode 100644 db/migrate/20180309053921_remove_registrars_indexes.rb create mode 100644 db/migrate/20180309054510_add_registrars_not_null_constraints.rb delete mode 100644 spec/models/registrar_spec.rb delete mode 100644 spec/requests/admin/registrars/update_spec.rb delete mode 100644 spec/routing/admin/registrars_routing_spec.rb delete mode 100644 spec/views/admin/registrars/_form.html.erb_spec.rb delete mode 100644 spec/views/admin/registrars/show.haml_spec.rb delete mode 100644 test/controllers/admin/registrars/update_test.rb create mode 100644 test/integration/admin/registrars/delete_test.rb delete mode 100644 test/integration/admin/registrars/edit_registrar_test.rb create mode 100644 test/integration/admin/registrars/edit_test.rb delete mode 100644 test/integration/admin/registrars/new_registrar_test.rb create mode 100644 test/integration/admin/registrars/new_test.rb create mode 100644 test/models/registrar/code_test.rb create mode 100644 test/models/registrar/delete_test.rb diff --git a/app/controllers/admin/registrars_controller.rb b/app/controllers/admin/registrars_controller.rb index 119abf85b..dd31e1c3d 100644 --- a/app/controllers/admin/registrars_controller.rb +++ b/app/controllers/admin/registrars_controller.rb @@ -19,17 +19,15 @@ module Admin def create @registrar = Registrar.new(registrar_params) - begin + if @registrar.valid? @registrar.transaction do @registrar.save! @registrar.accounts.create!(account_type: Account::CASH, currency: 'EUR') end - flash[:notice] = t('.created') - redirect_to [:admin, @registrar] - rescue ActiveRecord::RecordInvalid - flash.now[:alert] = t('.not_created') - render 'new' + redirect_to [:admin, @registrar], notice: t('.created') + else + render :new end end @@ -38,22 +36,15 @@ module Admin def update if @registrar.update(registrar_params) - flash[:notice] = t('.updated') - redirect_to [:admin, @registrar] + redirect_to [:admin, @registrar], notice: t('.updated') else - flash.now[:alert] = t('.not_updated') - render 'edit' + render :edit end end def destroy - if @registrar.destroy - flash[:notice] = I18n.t('registrar_deleted') - redirect_to admin_registrars_path - else - flash.now[:alert] = I18n.t('failed_to_delete_registrar') - render 'show' - end + @registrar.destroy! + redirect_to admin_registrars_url, notice: t('.deleted') end private diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 6a10983d2..de030b81b 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -6,7 +6,7 @@ class Registrar < ActiveRecord::Base has_many :api_users, dependent: :restrict_with_error has_many :messages has_many :invoices, foreign_key: 'buyer_id' - has_many :accounts + has_many :accounts, dependent: :destroy has_many :nameservers, through: :domains has_many :whois_records has_many :white_ips, dependent: :destroy @@ -17,37 +17,10 @@ class Registrar < ActiveRecord::Base validates :name, :reg_no, :reference_no, :code, uniqueness: true validates :accounting_customer_code, presence: true validates :language, presence: true - validate :forbidden_codes + validate :forbid_special_code after_initialize :set_defaults - - def forbidden_codes - return true unless ['CID'].include? code - errors.add(:code, I18n.t(:forbidden_code)) - false - end - before_validation :generate_iso_11649_reference_no - def generate_iso_11649_reference_no - return if reference_no.present? - - loop do - base = nil - loop do - base = SecureRandom.random_number.to_s.last(8) - break if base.to_i != 0 && base.length == 8 - end - - control_base = (base + '2715' + '00').to_i - reminder = control_base % 97 - check_digits = 98 - reminder - - check_digits = check_digits < 10 ? "0#{check_digits}" : check_digits.to_s - - self.reference_no = "RF#{check_digits}#{base}" - break unless self.class.exists?(reference_no: reference_no) - end - end validates :email, :billing_email, email_format: { message: :invalid }, @@ -177,4 +150,29 @@ class Registrar < ActiveRecord::Base def set_defaults self.language = Setting.default_language unless language end + + def forbid_special_code + errors.add(:code, :forbidden) if code == 'CID' + end + + def generate_iso_11649_reference_no + return if reference_no.present? + + loop do + base = nil + loop do + base = SecureRandom.random_number.to_s.last(8) + break if base.to_i != 0 && base.length == 8 + end + + control_base = (base + '2715' + '00').to_i + reminder = control_base % 97 + check_digits = 98 - reminder + + check_digits = check_digits < 10 ? "0#{check_digits}" : check_digits.to_s + + self.reference_no = "RF#{check_digits}#{base}" + break unless self.class.exists?(reference_no: reference_no) + end + end end diff --git a/app/views/admin/registrars/_billing.html.erb b/app/views/admin/registrars/_billing.html.erb new file mode 100644 index 000000000..f3960233e --- /dev/null +++ b/app/views/admin/registrars/_billing.html.erb @@ -0,0 +1,17 @@ +
+
+ <%= t '.header' %> +
+
+
+
<%= Registrar.human_attribute_name :vat_no %>
+
<%= registrar.vat_no %>
+ +
<%= Registrar.human_attribute_name :accounting_customer_code %>
+
<%= registrar.accounting_customer_code %>
+ +
<%= Registrar.human_attribute_name :billing_email %>
+
<%= registrar.billing_email %>
+
+
+
diff --git a/app/views/admin/registrars/_form.html.erb b/app/views/admin/registrars/_form.html.erb index 65a5351ad..71c4fb68f 100644 --- a/app/views/admin/registrars/_form.html.erb +++ b/app/views/admin/registrars/_form.html.erb @@ -1,5 +1,6 @@ <%= form_for([:admin, @registrar], html: { class: 'form-horizontal' }) do |f| %> - <%= render 'shared/full_errors', object: @registrar %> + <%= render 'form_errors', target: @registrar %> +
@@ -133,7 +134,9 @@ <%= f.label :code %>
- <%= f.text_field :code, required: f.object.new_record?, class: 'form-control', disabled: !f.object.new_record? %> + <%= f.text_field :code, required: f.object.new_record?, + disabled: f.object.persisted?, + class: 'form-control' %>
diff --git a/app/views/admin/registrars/edit.html.erb b/app/views/admin/registrars/edit.html.erb index 71d6bd882..205ba6b28 100644 --- a/app/views/admin/registrars/edit.html.erb +++ b/app/views/admin/registrars/edit.html.erb @@ -1,5 +1,10 @@ -<% content_for :actions do %> - <%= link_to(t(:back_to_registrar), [:admin, @registrar], class: 'btn btn-default') %> -<% end %> -<%= render 'shared/title', name: "#{t(:edit)}: #{@registrar.name}" %> +
    +
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. +
  2. <%= link_to @registrar.name, admin_registrar_path(@registrar) %>
    3. +
+ +
+

<%= t '.header' %>

+
+ <%= render 'form' %> diff --git a/app/views/admin/registrars/index.html.erb b/app/views/admin/registrars/index.html.erb index 5533a6f07..49bf7eae8 100644 --- a/app/views/admin/registrars/index.html.erb +++ b/app/views/admin/registrars/index.html.erb @@ -1,7 +1,15 @@ -<% content_for :actions do %> - <%= link_to(t('.new_btn'), new_admin_registrar_path, class: 'btn btn-primary') %> -<% end %> -<%= render 'shared/title', name: t(:registrars) %> +
+
+
+

<%= t '.header' %>

+
+ +
+ <%= link_to t('.new_btn'), new_admin_registrar_path, class: 'btn btn-primary' %> +
+
+
+
diff --git a/app/views/admin/registrars/new.html.erb b/app/views/admin/registrars/new.html.erb index 70154d217..e813bdd57 100644 --- a/app/views/admin/registrars/new.html.erb +++ b/app/views/admin/registrars/new.html.erb @@ -1,2 +1,9 @@ -<%= render 'shared/title', name: t(:new_registrar) %> +
    +
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. +
+ +
+

<%= t '.header' %>

+
+ <%= render 'form' %> diff --git a/app/views/admin/registrars/show.html.erb b/app/views/admin/registrars/show.html.erb index c09602985..ef75e4a64 100644 --- a/app/views/admin/registrars/show.html.erb +++ b/app/views/admin/registrars/show.html.erb @@ -1,15 +1,31 @@ <% registrar = RegistrarPresenter.new(registrar: @registrar, view: self) %> -<% content_for :actions do %> - <%= link_to(t(:edit), edit_admin_registrar_path(@registrar), class: 'btn btn-primary') %> - <%= link_to(t(:delete), admin_registrar_path(@registrar), method: :delete, data: { confirm: t(:are_you_sure) }, class: 'btn btn-danger') %> -<% end %> -<% content_for :page_name do %> - <%= @registrar.name %> - <% if @registrar.test_registrar? %> - (test) - <% end %> -<% end %> -<%= render 'shared/title', name: @registrar.name %> + +
    +
  1. <%= link_to t('admin.registrars.index.header'), admin_registrars_path %>
    2. +
+ +
+
+
+

+ <%= @registrar.name %> + <% if @registrar.test_registrar? %> + (test) + <% end %> +

+
+ +
+ <%= link_to t('.edit_btn'), edit_admin_registrar_path(@registrar), + class: 'btn btn-primary' %> + <%= link_to t('.delete_btn'), admin_registrar_path(@registrar), + method: :delete, + data: { confirm: t('.delete_btn_confirm') }, + class: 'btn btn-default' %> +
+
+
+ <% if @registrar.errors.any? %> <% @registrar.errors.each do |attr, err| %> <%= err %> @@ -41,12 +57,6 @@
<%= @registrar.reg_no %>
-
- <%= t(:vat_no) %> -
-
- <%= @registrar.vat_no %> -
<%= t(:reference_no) %>
@@ -71,12 +81,6 @@
<%= @registrar.website %>
-
- <%= Registrar.human_attribute_name :accounting_customer_code %> -
-
- <%= @registrar.accounting_customer_code %> -
@@ -120,21 +124,16 @@
<%= @registrar.billing_address %>
-
- <%= t(:billing_email) %> -
-
- <%= @registrar.billing_email %> -
+ <%= render 'billing', registrar: @registrar %> <%= render 'admin/registrars/show/preferences', registrar: registrar %>
-
+
<%= t('.api_users') %> @@ -174,7 +173,7 @@
-
+
<%= t(:white_ips) %> diff --git a/app/views/registrant/registrars/show.haml b/app/views/registrant/registrars/show.haml index e0da73ee3..6d4aa9e79 100644 --- a/app/views/registrant/registrars/show.haml +++ b/app/views/registrant/registrars/show.haml @@ -19,7 +19,7 @@ %dt= t(:reg_no) %dd= @registrar.reg_no - %dt= t(:vat_no) + %dt= Registrar.human_attribute_name :vat_no %dd= @registrar.vat_no %dt= t(:id) diff --git a/app/views/registrar/invoices/partials/_seller.haml b/app/views/registrar/invoices/partials/_seller.haml index 0008c2f13..9c5387e16 100644 --- a/app/views/registrar/invoices/partials/_seller.haml +++ b/app/views/registrar/invoices/partials/_seller.haml @@ -16,7 +16,7 @@ %dt= t(:swift) %dd= @invoice.seller_swift - %dt= t(:vat_no) + %dt= Registrar.human_attribute_name :vat_no %dd= @invoice.seller_vat_no %dt= t(:address) diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index a74254a2a..61b4e5789 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -261,7 +261,7 @@ %br = "#{t('reg_no')} #{@invoice.seller_reg_no}" %br - = "#{t('vat_no')} #{@invoice.seller_vat_no}" + = "#{Registrar.human_attribute_name :vat_no} #{@invoice.seller_vat_no}" .col-md-3.left = @invoice.seller_phone diff --git a/config/locales/admin/registrars.en.yml b/config/locales/admin/registrars.en.yml index 3d269f64b..5d212f219 100644 --- a/config/locales/admin/registrars.en.yml +++ b/config/locales/admin/registrars.en.yml @@ -2,9 +2,16 @@ en: admin: registrars: index: + header: Registrars new_btn: New registrar + new: + header: New registrar + show: + edit_btn: Edit + delete_btn: Delete + delete_btn_confirm: Are you sure you want delete registrar? new_api_use_btn: New API user active: Active api_users: API users @@ -12,13 +19,20 @@ en: preferences: header: Preferences + edit: + header: Edit registrar + + billing: + header: Billing + create: created: Registrar has been successfully created - not_created: Unable to create registrar update: updated: Registrar has been successfully updated - not_updated: Unable to update registrar + + destroy: + deleted: Registrar has been successfully deleted form: misc: Miscellaneous diff --git a/config/locales/en.yml b/config/locales/en.yml index 00923faf2..a093243b2 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -199,7 +199,6 @@ en: alg: 'Algorithm' public_key: 'Public key' registrar: - billing_email: 'Billing e-mail' phone: 'Contact phone' email: 'Contact e-mail' state: 'State / Province' @@ -315,13 +314,7 @@ en: edit_statuses: 'Edit statuses' history: 'History' - new_registrar: 'New registrar' - registrar_details: 'Registrar details' - vat_no: 'VAT no' - edit_registrar: 'Edit registrar' back_to_registrar: 'Back to registrar' - registrar_deleted: 'Registrar deleted' - failed_to_delete_registrar: 'Failed to delete registrar' users: 'Users' user_details: 'User details' @@ -652,7 +645,6 @@ en: due_date_until: 'Due date until' minimum_total: 'Minimum total' maximum_total: 'Maximum total' - forbidden_code: 'is forbidden to use' unimplemented_object_service: 'Unimplemented object service' contact_email_update_subject: 'Teie domeenide kontakt epostiaadress on muutunud / Contact e-mail addresses of your domains have changed' object_status_prohibits_operation: 'Object status prohibits operation' @@ -780,3 +772,6 @@ en: delimiter: " " precision: 2 unit: € + + attributes: + vat_no: VAT number diff --git a/config/locales/registrars.en.yml b/config/locales/registrars.en.yml new file mode 100644 index 000000000..60cb9d5c5 --- /dev/null +++ b/config/locales/registrars.en.yml @@ -0,0 +1,8 @@ +en: + activerecord: + errors: + models: + registrar: + attributes: + code: + forbidden: is forbidden diff --git a/db/migrate/20180309053424_add_registrars_unique_constraints.rb b/db/migrate/20180309053424_add_registrars_unique_constraints.rb new file mode 100644 index 000000000..1c2d3c1b8 --- /dev/null +++ b/db/migrate/20180309053424_add_registrars_unique_constraints.rb @@ -0,0 +1,19 @@ +class AddRegistrarsUniqueConstraints < ActiveRecord::Migration + def up + execute <<-SQL + ALTER TABLE registrars ADD CONSTRAINT unique_name UNIQUE (name); + ALTER TABLE registrars ADD CONSTRAINT unique_reg_no UNIQUE (reg_no); + ALTER TABLE registrars ADD CONSTRAINT unique_reference_no UNIQUE (reference_no); + ALTER TABLE registrars ADD CONSTRAINT unique_code UNIQUE (code); + SQL + end + + def down + execute <<-SQL + ALTER TABLE registrars DROP CONSTRAINT unique_name; + ALTER TABLE registrars DROP CONSTRAINT unique_reg_no; + ALTER TABLE registrars DROP CONSTRAINT unique_reference_no; + ALTER TABLE registrars DROP CONSTRAINT unique_code; + SQL + end +end diff --git a/db/migrate/20180309053921_remove_registrars_indexes.rb b/db/migrate/20180309053921_remove_registrars_indexes.rb new file mode 100644 index 000000000..0613d9fc6 --- /dev/null +++ b/db/migrate/20180309053921_remove_registrars_indexes.rb @@ -0,0 +1,6 @@ +class RemoveRegistrarsIndexes < ActiveRecord::Migration + def change + remove_index :registrars, name: :index_registrars_on_code + remove_index :registrars, name: :index_registrars_on_legacy_id + end +end diff --git a/db/migrate/20180309054510_add_registrars_not_null_constraints.rb b/db/migrate/20180309054510_add_registrars_not_null_constraints.rb new file mode 100644 index 000000000..33d464356 --- /dev/null +++ b/db/migrate/20180309054510_add_registrars_not_null_constraints.rb @@ -0,0 +1,9 @@ +class AddRegistrarsNotNullConstraints < ActiveRecord::Migration + def change + change_column_null :registrars, :name, false + change_column_null :registrars, :reg_no, false + change_column_null :registrars, :country_code, false + change_column_null :registrars, :email, false + change_column_null :registrars, :code, false + end +end diff --git a/db/structure.sql b/db/structure.sql index b303afbe7..194bc8111 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -2136,8 +2136,8 @@ ALTER SEQUENCE registrant_verifications_id_seq OWNED BY registrant_verifications CREATE TABLE registrars ( id integer NOT NULL, - name character varying, - reg_no character varying, + name character varying NOT NULL, + reg_no character varying NOT NULL, vat_no character varying, billing_address character varying, created_at timestamp without time zone, @@ -2145,14 +2145,14 @@ CREATE TABLE registrars ( creator_str character varying, updator_str character varying, phone character varying, - email character varying, + email character varying NOT NULL, billing_email character varying, - country_code character varying, + country_code character varying NOT NULL, state character varying, city character varying, street character varying, zip character varying, - code character varying, + code character varying NOT NULL, website character varying, accounting_customer_code character varying NOT NULL, vat boolean, @@ -3244,6 +3244,14 @@ ALTER TABLE ONLY settings ADD CONSTRAINT settings_pkey PRIMARY KEY (id); +-- +-- Name: unique_code; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- + +ALTER TABLE ONLY registrars + ADD CONSTRAINT unique_code UNIQUE (code); + + -- -- Name: unique_contact_code; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: -- @@ -3252,6 +3260,30 @@ ALTER TABLE ONLY contacts ADD CONSTRAINT unique_contact_code UNIQUE (code); +-- +-- Name: unique_name; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- + +ALTER TABLE ONLY registrars + ADD CONSTRAINT unique_name UNIQUE (name); + + +-- +-- Name: unique_reference_no; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- + +ALTER TABLE ONLY registrars + ADD CONSTRAINT unique_reference_no UNIQUE (reference_no); + + +-- +-- Name: unique_reg_no; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- + +ALTER TABLE ONLY registrars + ADD CONSTRAINT unique_reg_no UNIQUE (reg_no); + + -- -- Name: unique_session_id; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: -- @@ -3854,20 +3886,6 @@ CREATE INDEX index_registrant_verifications_on_created_at ON registrant_verifica CREATE INDEX index_registrant_verifications_on_domain_id ON registrant_verifications USING btree (domain_id); --- --- Name: index_registrars_on_code; Type: INDEX; Schema: public; Owner: -; Tablespace: --- - -CREATE INDEX index_registrars_on_code ON registrars USING btree (code); - - --- --- Name: index_registrars_on_legacy_id; Type: INDEX; Schema: public; Owner: -; Tablespace: --- - -CREATE INDEX index_registrars_on_legacy_id ON registrars USING btree (legacy_id); - - -- -- Name: index_settings_on_thing_type_and_thing_id_and_var; Type: INDEX; Schema: public; Owner: -; Tablespace: -- @@ -4678,3 +4696,9 @@ INSERT INTO schema_migrations (version) VALUES ('20180306183540'); INSERT INTO schema_migrations (version) VALUES ('20180306183549'); +INSERT INTO schema_migrations (version) VALUES ('20180309053424'); + +INSERT INTO schema_migrations (version) VALUES ('20180309053921'); + +INSERT INTO schema_migrations (version) VALUES ('20180309054510'); + diff --git a/lib/tasks/import.rake b/lib/tasks/import.rake index c3b67ba88..2b9909f68 100644 --- a/lib/tasks/import.rake +++ b/lib/tasks/import.rake @@ -101,7 +101,7 @@ namespace :import do puts "-----> Generating reference numbers" Registrar.all.each do |x| - x.generate_iso_11649_reference_no + x.send(:generate_iso_11649_reference_no) x.save(validate: false) end diff --git a/spec/models/registrar_spec.rb b/spec/models/registrar_spec.rb deleted file mode 100644 index 4a8110d45..000000000 --- a/spec/models/registrar_spec.rb +++ /dev/null @@ -1,118 +0,0 @@ -require 'rails_helper' - -describe Registrar do - context 'with invalid attribute' do - before :all do - @registrar = Registrar.new - end - - it 'is not valid' do - @registrar.valid? - @registrar.errors.full_messages.should include(*[ - 'Contact e-mail is missing', - 'Country code is missing', - 'Name is missing', - 'Reg no is missing', - 'Code is missing' - ]) - end - - it 'returns an error with invalid email' do - @registrar.email = 'bla' - @registrar.billing_email = 'bla' - - @registrar.valid? - @registrar.errors[:email].should == ['is invalid'] - @registrar.errors[:billing_email].should == ['is invalid'] - end - - it 'should not have valid code' do - @registrar.code.should == nil - end - - it 'should generate reference number' do - @registrar.generate_iso_11649_reference_no - @registrar.reference_no.should_not be_blank - @registrar.reference_no.last(10).to_i.should_not == 0 - end - end - - context 'with valid attributes' do - before :all do - @registrar = create(:registrar) - end - - it 'should be valid' do - @registrar.valid? - @registrar.errors.full_messages.should match_array([]) - end - - it 'should be valid twice' do - @registrar = create(:registrar) - @registrar.valid? - @registrar.errors.full_messages.should match_array([]) - end - - it 'should remove blank from code' do - registrar = build(:registrar, code: 'with blank') - registrar.valid? - registrar.errors.full_messages.should match_array([ - ]) - registrar.code.should == 'WITHBLANK' - end - - it 'should remove colon from code' do - registrar = build(:registrar, code: 'with colon:and:blank') - registrar.valid? - registrar.errors.full_messages.should match_array([ - ]) - registrar.code.should == 'WITHCOLONANDBLANK' - end - - it 'should allow dot in code' do - registrar = build(:registrar, code: 'with.dot') - registrar.valid? - registrar.errors.full_messages.should match_array([ - ]) - registrar.code.should == 'WITH.DOT' - end - - it 'should have one version' do - with_versioning do - @registrar.versions.should == [] - @registrar.name = 'New name' - @registrar.save - @registrar.errors.full_messages.should match_array([]) - @registrar.versions.size.should == 1 - end - end - - it 'should return full address' do - registrar = described_class.new(street: 'Street 999', city: 'Town', state: 'County', zip: 'Postal') - registrar.address.should == 'Street 999, Town, County, Postal' - end - - it 'should not be able to change code' do - registrar = create(:registrar, code: 'TEST') - registrar.code = 'new-code' - expect(registrar.code).to eq('TEST') - end - - it 'should be able to issue a prepayment invoice' do - Setting.days_to_keep_invoices_active = 30 - create(:registrar, name: 'EIS', reg_no: '90010019') - @registrar.issue_prepayment_invoice(200, 'add some money') - @registrar.invoices.count.should == 1 - i = @registrar.invoices.first - i.sum.should == BigDecimal.new('240.0') - i.due_date.should be_within(0.1).of((Time.zone.now + 30.days).end_of_day) - i.description.should == 'add some money' - end - - it 'should not allaw to use CID as code for leagcy reasons' do - registrar = build(:registrar, code: 'CID') - registrar.valid? - registrar.errors.full_messages.should == ['Code is forbidden to use'] - end - end -end diff --git a/spec/requests/admin/registrars/update_spec.rb b/spec/requests/admin/registrars/update_spec.rb deleted file mode 100644 index 7bffb99d0..000000000 --- a/spec/requests/admin/registrars/update_spec.rb +++ /dev/null @@ -1,42 +0,0 @@ -require 'rails_helper' - -RSpec.describe 'admin registrar update' do - before :example do - sign_in_to_admin_area - end - - it 'updates website' do - registrar = create(:registrar, website: 'test') - - patch admin_registrar_path(registrar), registrar: attributes_for(:registrar, website: 'new-website') - registrar.reload - - expect(registrar.website).to eq('new-website') - end - - it 'updates email' do - registrar = create(:registrar, email: 'test@test.com') - - patch admin_registrar_path(registrar), registrar: attributes_for(:registrar, email: 'new-test@test.com') - registrar.reload - - expect(registrar.email).to eq('new-test@test.com') - end - - it 'updates billing email' do - registrar = create(:registrar, billing_email: 'test@test.com') - - patch admin_registrar_path(registrar), registrar: attributes_for(:registrar, billing_email: 'new-test@test.com') - registrar.reload - - expect(registrar.billing_email).to eq('new-test@test.com') - end - - it 'redirects to :show' do - registrar = create(:registrar) - - patch admin_registrar_path(registrar), { registrar: attributes_for(:registrar) } - - expect(response).to redirect_to admin_registrar_path(registrar) - end -end diff --git a/spec/routing/admin/registrars_routing_spec.rb b/spec/routing/admin/registrars_routing_spec.rb deleted file mode 100644 index 14ec765e9..000000000 --- a/spec/routing/admin/registrars_routing_spec.rb +++ /dev/null @@ -1,33 +0,0 @@ -require 'rails_helper' - -RSpec.describe Admin::RegistrarsController do - describe 'routing' do - it 'routes to #index' do - expect(get: '/admin/registrars').to route_to('admin/registrars#index') - end - - it 'routes to #new' do - expect(get: '/admin/registrars/new').to route_to('admin/registrars#new') - end - - it 'routes to #show' do - expect(get: '/admin/registrars/1').to route_to('admin/registrars#show', id: '1') - end - - it 'routes to #edit' do - expect(get: '/admin/registrars/1/edit').to route_to('admin/registrars#edit', id: '1') - end - - it 'routes to #create' do - expect(post: '/admin/registrars').to route_to('admin/registrars#create') - end - - it 'routes to #update' do - expect(patch: '/admin/registrars/1').to route_to('admin/registrars#update', id: '1') - end - - it 'routes to #destroy' do - expect(delete: '/admin/registrars/1').to route_to('admin/registrars#destroy', id: '1') - end - end -end diff --git a/spec/views/admin/registrars/_form.html.erb_spec.rb b/spec/views/admin/registrars/_form.html.erb_spec.rb deleted file mode 100644 index 83b727a30..000000000 --- a/spec/views/admin/registrars/_form.html.erb_spec.rb +++ /dev/null @@ -1,19 +0,0 @@ -require 'rails_helper' - -RSpec.describe 'admin/registrars/_form' do - let(:registrar) { build_stubbed(:registrar) } - - before :example do - assign(:registrar, registrar) - stub_template 'shared/_full_errors' => '' - - without_partial_double_verification do - allow(view).to receive(:available_languages).and_return({}) - end - end - - it 'has website' do - render - expect(rendered).to have_css('[name="registrar[website]"]') - end -end diff --git a/spec/views/admin/registrars/show.haml_spec.rb b/spec/views/admin/registrars/show.haml_spec.rb deleted file mode 100644 index 4def10e8c..000000000 --- a/spec/views/admin/registrars/show.haml_spec.rb +++ /dev/null @@ -1,19 +0,0 @@ -require 'rails_helper' - -RSpec.describe 'admin/registrars/show' do - let(:registrar) { build_stubbed(:registrar, website: 'test website') } - - before :example do - assign(:registrar, registrar) - stub_template 'shared/_title' => '' - - without_partial_double_verification do - allow(view).to receive(:available_languages).and_return({}) - end - end - - it 'has website' do - render - expect(rendered).to have_text('test website') - end -end diff --git a/test/controllers/admin/registrars/update_test.rb b/test/controllers/admin/registrars/update_test.rb deleted file mode 100644 index 2c996ef44..000000000 --- a/test/controllers/admin/registrars/update_test.rb +++ /dev/null @@ -1,30 +0,0 @@ -require 'test_helper' - -class RegistrarsControllerTest < ActionDispatch::IntegrationTest - def setup - login_as users(:admin) - @registrar = registrars(:bestnames) - end - - def test_updates_website - patch admin_registrar_path(@registrar), registrar: @registrar.attributes.merge(website: 'new.example.com') - @registrar.reload - - assert_equal 'new.example.com', @registrar.website - end - - def test_updates_email - patch admin_registrar_path(@registrar), registrar: @registrar.attributes.merge(email: 'new@example.com') - @registrar.reload - - assert_equal 'new@example.com', @registrar.email - end - - def test_updates_billing_email - patch admin_registrar_path(@registrar), - registrar: @registrar.attributes.merge(billing_email: 'new-billing@example.com') - @registrar.reload - - assert_equal 'new-billing@example.com', @registrar.billing_email - end -end diff --git a/test/fixtures/accounts.yml b/test/fixtures/accounts.yml index ea51ff7eb..a7060c249 100644 --- a/test/fixtures/accounts.yml +++ b/test/fixtures/accounts.yml @@ -3,3 +3,9 @@ cash: balance: 100 currency: EUR registrar: bestnames + +not_in_use_cash: + account_type: cash + balance: 0 + currency: EUR + registrar: not_in_use diff --git a/test/fixtures/registrars.yml b/test/fixtures/registrars.yml index 0026f323b..b0cc371a7 100644 --- a/test/fixtures/registrars.yml +++ b/test/fixtures/registrars.yml @@ -3,6 +3,10 @@ bestnames: reg_no: 1234 code: bestnames email: info@bestnames.test + street: Main Street + zip: 12345 + city: New York + state: New York country_code: US accounting_customer_code: bestnames language: en @@ -16,3 +20,12 @@ goodnames: country_code: US accounting_customer_code: goodnames language: en + +not_in_use: + name: any + reg_no: any + code: any + email: any@example.com + country_code: US + accounting_customer_code: any + language: en diff --git a/test/integration/admin/registrars/delete_test.rb b/test/integration/admin/registrars/delete_test.rb new file mode 100644 index 000000000..22e561664 --- /dev/null +++ b/test/integration/admin/registrars/delete_test.rb @@ -0,0 +1,18 @@ +require 'test_helper' + +class AdminAreaDeleteRegistrarTest < ActionDispatch::IntegrationTest + def setup + login_as users(:admin) + end + + def test_can_be_deleted_if_not_in_use + visit admin_registrar_url(registrars(:not_in_use)) + + assert_difference 'Registrar.count', -1 do + click_link_or_button 'Delete' + end + + assert_current_path admin_registrars_path + assert_text 'Registrar has been successfully deleted' + end +end diff --git a/test/integration/admin/registrars/edit_registrar_test.rb b/test/integration/admin/registrars/edit_registrar_test.rb deleted file mode 100644 index ada794032..000000000 --- a/test/integration/admin/registrars/edit_registrar_test.rb +++ /dev/null @@ -1,17 +0,0 @@ -require 'test_helper' - -class EditRegistrarTest < ActionDispatch::IntegrationTest - def setup - login_as users(:admin) - @registrar = registrars(:bestnames) - end - - def test_updates_registrar - visit admin_registrar_path(@registrar) - click_link_or_button 'Edit' - click_link_or_button 'Update registrar' - - assert_current_path admin_registrar_path(@registrar) - assert_text 'Registrar has been successfully updated' - end -end diff --git a/test/integration/admin/registrars/edit_test.rb b/test/integration/admin/registrars/edit_test.rb new file mode 100644 index 000000000..af420b179 --- /dev/null +++ b/test/integration/admin/registrars/edit_test.rb @@ -0,0 +1,69 @@ +require 'test_helper' + +class AdminAreaEditRegistrarTest < ActionDispatch::IntegrationTest + def setup + login_as users(:admin) + @registrar = registrars(:bestnames) + end + + def test_attributes_update + visit admin_registrar_path(@registrar) + click_link_or_button 'Edit' + + fill_in 'Name', with: 'new name' + fill_in 'Reg no', with: '4727673' + fill_in 'Contact phone', with: '2570937' + fill_in 'Website', with: 'http://new.example.com' + fill_in 'Contact e-mail', with: 'new@example.com' + + fill_in 'Street', with: 'new street' + fill_in 'Zip', with: 'new zip' + fill_in 'City', with: 'new city' + fill_in 'State / Province', with: 'new state' + select 'Germany', from: 'Country' + + fill_in 'VAT number', with: '2386449' + fill_in 'Accounting customer code', with: '866477' + fill_in 'Billing email', with: 'new-billing@example.com' + + select 'Estonian', from: 'Language' + click_link_or_button 'Update registrar' + + @registrar.reload + assert_equal 'new name', @registrar.name + assert_equal '4727673', @registrar.reg_no + assert_equal '2570937', @registrar.phone + assert_equal 'http://new.example.com', @registrar.website + assert_equal 'new@example.com', @registrar.email + + assert_equal 'new street', @registrar.street + assert_equal 'new zip', @registrar.zip + assert_equal 'new city', @registrar.city + assert_equal 'new state', @registrar.state + assert_equal Country.new('DE'), @registrar.country + + assert_equal '2386449', @registrar.vat_no + assert_equal '866477', @registrar.accounting_customer_code + assert_equal 'new-billing@example.com', @registrar.billing_email + + assert_equal 'et', @registrar.language + assert_current_path admin_registrar_path(@registrar) + assert_text 'Registrar has been successfully updated' + end + + def test_code_cannot_be_changed + visit admin_registrar_path(@registrar) + click_link_or_button 'Edit' + assert_no_field 'Code' + end + + def test_fails_gracefully + visit admin_registrar_path(@registrar) + click_link_or_button 'Edit' + fill_in 'Name', with: 'Good Names' + click_link_or_button 'Update registrar' + + assert_field 'Name', with: 'Good Names' + assert_text 'Name has already been taken' + end +end diff --git a/test/integration/admin/registrars/new_registrar_test.rb b/test/integration/admin/registrars/new_registrar_test.rb deleted file mode 100644 index 33deea9e8..000000000 --- a/test/integration/admin/registrars/new_registrar_test.rb +++ /dev/null @@ -1,23 +0,0 @@ -require 'test_helper' - -class NewRegistrarTest < ActionDispatch::IntegrationTest - def setup - login_as users(:admin) - end - - def test_creates_registrar - visit admin_registrars_path - click_link_or_button 'New registrar' - - fill_in 'registrar[name]', with: 'John Doe' - fill_in 'registrar[reg_no]', with: '1234567' - fill_in 'registrar[email]', with: 'test@test.com' - fill_in 'registrar[code]', with: 'test' - fill_in 'registrar[accounting_customer_code]', with: 'test' - click_link_or_button 'Create registrar' - - assert_current_path admin_registrar_path(Registrar.last) - assert_text 'Registrar has been successfully created' - assert_text 'John Doe' - end -end diff --git a/test/integration/admin/registrars/new_test.rb b/test/integration/admin/registrars/new_test.rb new file mode 100644 index 000000000..61a7a43be --- /dev/null +++ b/test/integration/admin/registrars/new_test.rb @@ -0,0 +1,49 @@ +require 'test_helper' + +class AdminAreaNewRegistrarTest < ActionDispatch::IntegrationTest + def setup + login_as users(:admin) + end + + def test_new_registrar_creation_with_required_params + visit admin_registrars_url + click_link_or_button 'New registrar' + + fill_in 'Name', with: 'Brand new names' + fill_in 'Reg no', with: '55555555' + fill_in 'Contact e-mail', with: 'test@example.com' + fill_in 'Accounting customer code', with: 'test' + fill_in 'Code', with: 'test' + + assert_difference 'Registrar.count' do + click_link_or_button 'Create registrar' + end + + assert_current_path admin_registrar_path(Registrar.last) + assert_text 'Registrar has been successfully created' + end + + def test_fails_gracefully + visit admin_registrars_url + click_link_or_button 'New registrar' + + fill_in 'Name', with: 'Best Names' + fill_in 'Reg no', with: '55555555' + fill_in 'Contact e-mail', with: 'test@example.com' + fill_in 'Accounting customer code', with: 'test' + fill_in 'Code', with: 'test' + + assert_no_difference 'Registrar.count' do + click_link_or_button 'Create registrar' + end + assert_field 'Name', with: 'Best Names' + assert_text 'Name has already been taken' + end + + def test_pre_populated_default_language + Setting.default_language = 'en' + visit admin_registrars_url + click_link_or_button 'New registrar' + assert_field 'Language', with: 'en' + end +end diff --git a/test/models/registrar/code_test.rb b/test/models/registrar/code_test.rb new file mode 100644 index 000000000..7b5188878 --- /dev/null +++ b/test/models/registrar/code_test.rb @@ -0,0 +1,29 @@ +require 'test_helper' + +class RegistrarCodeTest < ActiveSupport::TestCase + def setup + @registrar = registrars(:bestnames).dup + end + + def test_registrar_is_invalid_without_code + @registrar.code = '' + assert @registrar.invalid? + end + + def test_special_code_validation + @registrar.code = 'CID' + assert @registrar.invalid? + assert_includes @registrar.errors.full_messages, 'Code is forbidden' + end + + def test_cannot_be_changed_once_registrar_is_created + registrar = registrars(:bestnames) + registrar.update!(code: 'new-code') + refute_equal 'new-code', registrar.code + end + + def test_normalization + @registrar.code = 'with spaces:and:colon.' + assert_equal 'WITHSPACESANDCOLON.', @registrar.code + end +end diff --git a/test/models/registrar/delete_test.rb b/test/models/registrar/delete_test.rb new file mode 100644 index 000000000..2020c76dc --- /dev/null +++ b/test/models/registrar/delete_test.rb @@ -0,0 +1,37 @@ +require 'test_helper' + +class DeleteRegistrarTest < ActiveSupport::TestCase + def setup + @registrar = registrars(:not_in_use) + end + + def test_can_be_deleted_if_not_in_use + assert_difference 'Registrar.count', -1 do + @registrar.destroy + end + end + + def test_cannot_be_deleted_if_has_at_least_one_user + users(:api_bestnames).update!(registrar: @registrar) + + assert_no_difference 'Registrar.count' do + @registrar.destroy + end + end + + def test_cannot_be_deleted_if_has_at_least_one_contact + contacts(:john).update!(registrar: @registrar) + + assert_no_difference 'Registrar.count' do + @registrar.destroy + end + end + + def test_cannot_be_deleted_if_has_at_least_one_domain + domains(:shop).update!(registrar: @registrar) + + assert_no_difference 'Registrar.count' do + @registrar.destroy + end + end +end diff --git a/test/models/registrar_test.rb b/test/models/registrar_test.rb index 58a06b0ae..0901e83f0 100644 --- a/test/models/registrar_test.rb +++ b/test/models/registrar_test.rb @@ -9,21 +9,33 @@ class RegistrarTest < ActiveSupport::TestCase assert @registrar.valid? end + def test_invalid_without_name + @registrar.name = '' + assert @registrar.invalid? + end + + def test_invalid_without_reg_no + @registrar.reg_no = '' + assert @registrar.invalid? + end + + def test_invalid_without_email + @registrar.email = '' + assert @registrar.invalid? + end + def test_rejects_absent_accounting_customer_code @registrar.accounting_customer_code = nil - @registrar.validate assert @registrar.invalid? end def test_requires_country_code @registrar.country_code = nil - @registrar.validate assert @registrar.invalid? end - def test_requires_language - @registrar.language = nil - @registrar.validate + def test_invalid_without_language + @registrar.language = '' assert @registrar.invalid? end @@ -38,4 +50,13 @@ class RegistrarTest < ActiveSupport::TestCase registrar = Registrar.new(language: 'de') assert_equal 'de', registrar.language end + + def test_full_address + assert_equal 'Main Street, New York, New York, 12345', @registrar.address + end + + def test_reference_number_generation + @registrar.validate + refute_empty @registrar.reference_no + end end From d6d20a81a95e1810989b20f5e9ab9b1766542163 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Fri, 9 Mar 2018 13:08:36 +0200 Subject: [PATCH 30/87] Add registrar vat rate #623 --- app/models/registrar.rb | 33 +++------ app/models/type/vat_rate.rb | 19 +++++ app/views/admin/registrars/_billing.html.erb | 3 + .../admin/registrars/form/_billing.html.erb | 3 +- test/models/registrar/vat_test.rb | 73 +++++++++++++++++++ 5 files changed, 108 insertions(+), 23 deletions(-) create mode 100644 app/models/type/vat_rate.rb create mode 100644 test/models/registrar/vat_test.rb diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 1d5c4eba3..87915e4ba 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -1,21 +1,3 @@ -class VATRateType < ActiveRecord::Type::Value - def type_cast_from_user(value) - if value.blank? - nil - else - super - end - end - - def type_cast_from_database(value) - BigDecimal.new(value) * 100 if value - end - - def type_cast_for_database(value) - BigDecimal.new(value) / 100 if value - end -end - class Registrar < ActiveRecord::Base include Versions # version/registrar_version.rb @@ -35,13 +17,16 @@ class Registrar < ActiveRecord::Base validates :name, :reg_no, :reference_no, :code, uniqueness: true validates :accounting_customer_code, presence: true validates :language, presence: true - validates :vat_rate, :vat_no, absence: true, if: :local_vat_payer? - validates :vat_rate, presence: true, if: 'foreign_vat_payer? && vat_no.blank?' + + validates :vat_rate, presence: true, if: :vat_rate_required? + validates :vat_rate, absence: true, if: :local_vat_payer? validates :vat_rate, absence: true, if: 'foreign_vat_payer? && vat_no?' - validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 99 }, allow_nil: true + validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 99.9 }, + allow_nil: true + validate :forbid_special_code - attribute :vat_rate, VATRateType.new + attribute :vat_rate, ::Type::VATRate.new after_initialize :set_defaults before_validation :generate_iso_11649_reference_no @@ -216,4 +201,8 @@ class Registrar < ActiveRecord::Base def foreign_vat_payer? !local_vat_payer? end + + def vat_rate_required? + foreign_vat_payer? && vat_no.blank? + end end diff --git a/app/models/type/vat_rate.rb b/app/models/type/vat_rate.rb new file mode 100644 index 000000000..466b076f8 --- /dev/null +++ b/app/models/type/vat_rate.rb @@ -0,0 +1,19 @@ +module Type + class VATRate < ActiveRecord::Type::Value + def type_cast_from_user(value) + if value.blank? + nil + else + super + end + end + + def type_cast_from_database(value) + BigDecimal(value) * 100 if value + end + + def type_cast_for_database(value) + BigDecimal(value) / 100.0 if value + end + end +end diff --git a/app/views/admin/registrars/_billing.html.erb b/app/views/admin/registrars/_billing.html.erb index f3960233e..7ba778d08 100644 --- a/app/views/admin/registrars/_billing.html.erb +++ b/app/views/admin/registrars/_billing.html.erb @@ -7,6 +7,9 @@
<%= Registrar.human_attribute_name :vat_no %>
<%= registrar.vat_no %>
+
<%= Registrar.human_attribute_name :vat_rate %>
+
<%= number_to_percentage registrar.vat_rate, precision: 1 %>
+
<%= Registrar.human_attribute_name :accounting_customer_code %>
<%= registrar.accounting_customer_code %>
diff --git a/app/views/admin/registrars/form/_billing.html.erb b/app/views/admin/registrars/form/_billing.html.erb index ff67232de..e644085cf 100644 --- a/app/views/admin/registrars/form/_billing.html.erb +++ b/app/views/admin/registrars/form/_billing.html.erb @@ -23,7 +23,8 @@
- <%= f.number_field :vat_rate, min: 0, max: 99, class: 'form-control' %> + <%= f.number_field :vat_rate, min: 0, max: 99, step: 0.1, + class: 'form-control' %>
%
diff --git a/test/models/registrar/vat_test.rb b/test/models/registrar/vat_test.rb new file mode 100644 index 000000000..cf36b6266 --- /dev/null +++ b/test/models/registrar/vat_test.rb @@ -0,0 +1,73 @@ +require 'test_helper' + +class RegistrarVATTest < ActiveSupport::TestCase + def setup + @registrar = registrars(:bestnames) + end + + def test_optional_vat_no + @registrar.vat_no = '' + assert @registrar.valid? + + @registrar.vat_no = 'any' + assert @registrar.valid? + end + + def test_requires_vat_rate_when_registrar_is_foreign_vat_payer_and_vat_no_is_absent + @registrar.vat_no = '' + + Registry.instance.stub(:legal_address_country, Country.new('GB')) do + @registrar.vat_rate = '' + assert @registrar.invalid? + assert @registrar.errors.added?(:vat_rate, :blank) + + @registrar.vat_rate = -1 + assert @registrar.invalid? + + @registrar.vat_rate = 1 + assert @registrar.valid? + + @registrar.vat_rate = 99.9 + assert @registrar.valid? + + @registrar.vat_rate = 100 + assert @registrar.invalid? + end + end + + def test_vat_is_not_applied_when_registrar_is_local_vat_payer + @registrar.vat_rate = 1 + assert @registrar.invalid? + + @registrar.vat_rate = nil + assert @registrar.valid? + end + + def test_vat_is_not_applied_when_registrar_is_foreign_vat_payer_and_vat_no_is_present + @registrar.vat_no = 'valid' + + Registry.instance.stub(:legal_address_country, Country.new('GB')) do + @registrar.vat_rate = 1 + assert @registrar.invalid? + + @registrar.vat_rate = nil + assert @registrar.valid? + end + end + + def test_serializes_and_deserializes_vat_rate + @registrar.vat_rate = '25.5' + + Registry.instance.stub(:legal_address_country, Country.new('GB')) do + @registrar.save! + end + + @registrar.reload + assert_equal 25.5, @registrar.vat_rate + end + + def test_treats_empty_vat_rate_as_absent + @registrar.vat_rate = '' + assert_nil @registrar.vat_rate + end +end From 9ae08a1e6f2215db847805ce497de794514480b5 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sat, 4 Nov 2017 21:33:01 +0200 Subject: [PATCH 31/87] Remove dead code #623 (cherry picked from commit 49af8cc) --- app/models/registrar.rb | 5 ----- 1 file changed, 5 deletions(-) diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 87915e4ba..1f1bfaea9 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -123,11 +123,6 @@ class Registrar < ActiveRecord::Base cash_account.account_activities.create!(args) end - def credit!(args) - args[:currency] = 'EUR' - cash_account.account_activities.create!(args) - end - def address [street, city, state, zip].reject(&:blank?).compact.join(', ') end From efaac16ceeec2ead3349522d15d90ba76b629290 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sat, 10 Mar 2018 16:33:35 +0200 Subject: [PATCH 32/87] Remove unused attribute #772 --- app/models/directo.rb | 2 +- app/models/invoice.rb | 2 +- app/models/registrar.rb | 1 - ...0180310142630_remove_invoices_invoice_type.rb | 5 +++++ db/structure.sql | 3 ++- doc/models_complete.svg | 1 - spec/factories/invoice.rb | 1 - spec/models/invoice_spec.rb | 1 - test/fixtures/invoices.yml | 1 - .../registrar/billing/balance_top_up_test.rb | 16 ++++++++++++++++ 10 files changed, 25 insertions(+), 8 deletions(-) create mode 100644 db/migrate/20180310142630_remove_invoices_invoice_type.rb create mode 100644 test/integration/registrar/billing/balance_top_up_test.rb diff --git a/app/models/directo.rb b/app/models/directo.rb index 62cf43804..e0301e3dc 100644 --- a/app/models/directo.rb +++ b/app/models/directo.rb @@ -3,7 +3,7 @@ class Directo < ActiveRecord::Base belongs_to :item, polymorphic: true def self.send_receipts - new_trans = Invoice.where(invoice_type: "DEB", in_directo: false).where(cancelled_at: nil) + new_trans = Invoice.where(in_directo: false).where(cancelled_at: nil) total = new_trans.count counter = 0 Rails.logger.info("[DIRECTO] Will try to send #{total} invoices") diff --git a/app/models/invoice.rb b/app/models/invoice.rb index ad478443d..5dcf9ba78 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -27,7 +27,7 @@ class Invoice < ActiveRecord::Base attr_accessor :billing_email validates :billing_email, email_format: { message: :invalid }, allow_blank: true - validates :invoice_type, :due_date, :currency, :seller_name, + validates :due_date, :currency, :seller_name, :seller_iban, :buyer_name, :invoice_items, :vat_prc, presence: true before_create :set_invoice_number, :check_vat diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 6a10983d2..3c8300eb5 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -76,7 +76,6 @@ class Registrar < ActiveRecord::Base # rubocop:disable Metrics/AbcSize def issue_prepayment_invoice(amount, description = nil) invoices.create( - invoice_type: 'DEB', due_date: (Time.zone.now.to_date + Setting.days_to_keep_invoices_active.days).end_of_day, payment_term: 'prepayment', description: description, diff --git a/db/migrate/20180310142630_remove_invoices_invoice_type.rb b/db/migrate/20180310142630_remove_invoices_invoice_type.rb new file mode 100644 index 000000000..f8cddadf9 --- /dev/null +++ b/db/migrate/20180310142630_remove_invoices_invoice_type.rb @@ -0,0 +1,5 @@ +class RemoveInvoicesInvoiceType < ActiveRecord::Migration + def change + remove_column :invoices, :invoice_type, :string + end +end diff --git a/db/structure.sql b/db/structure.sql index a9257ffa3..65a686a97 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -998,7 +998,6 @@ CREATE TABLE invoices ( id integer NOT NULL, created_at timestamp without time zone NOT NULL, updated_at timestamp without time zone NOT NULL, - invoice_type character varying NOT NULL, due_date timestamp without time zone NOT NULL, payment_term character varying, currency character varying NOT NULL, @@ -4679,3 +4678,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180306183549'); INSERT INTO schema_migrations (version) VALUES ('20180308123240'); +INSERT INTO schema_migrations (version) VALUES ('20180310142630'); + diff --git a/doc/models_complete.svg b/doc/models_complete.svg index 296d20c67..1cbefea90 100644 --- a/doc/models_complete.svg +++ b/doc/models_complete.svg @@ -1614,7 +1614,6 @@ id :integer created_at :datetime updated_at :datetime -invoice_type :string due_date :datetime payment_term :string currency :string diff --git a/spec/factories/invoice.rb b/spec/factories/invoice.rb index 7eeed32b9..3bd0c5f08 100644 --- a/spec/factories/invoice.rb +++ b/spec/factories/invoice.rb @@ -3,7 +3,6 @@ FactoryBot.define do buyer_name 'Registrar 1' currency { 'EUR' } due_date { Time.zone.now.to_date + 1.day } - invoice_type 'DEB' seller_iban { '123' } seller_name { 'EIS' } seller_city { 'Tallinn' } diff --git a/spec/models/invoice_spec.rb b/spec/models/invoice_spec.rb index 49d15310f..95b603f85 100644 --- a/spec/models/invoice_spec.rb +++ b/spec/models/invoice_spec.rb @@ -13,7 +13,6 @@ describe Invoice do "Currency is missing", "Due date is missing", "Invoice items is missing", - "Invoice type is missing", "Seller iban is missing", "Seller name is missing", "Vat prc is missing" diff --git a/test/fixtures/invoices.yml b/test/fixtures/invoices.yml index 7a1c85dd1..cc9f829d0 100644 --- a/test/fixtures/invoices.yml +++ b/test/fixtures/invoices.yml @@ -1,7 +1,6 @@ DEFAULTS: &DEFAULTS created_at: <%= Date.parse '2010-07-05' %> due_date: <%= Date.parse '2010-07-06' %> - invoice_type: DEB currency: EUR seller_name: John Doe seller_iban: 1234 diff --git a/test/integration/registrar/billing/balance_top_up_test.rb b/test/integration/registrar/billing/balance_top_up_test.rb new file mode 100644 index 000000000..c4dd1cf5d --- /dev/null +++ b/test/integration/registrar/billing/balance_top_up_test.rb @@ -0,0 +1,16 @@ +require 'test_helper' + +class BalanceTopUpTest < ActionDispatch::IntegrationTest + def setup + login_as users(:api_bestnames) + end + + def test_registrar_balance_top_up + visit registrar_invoices_url + click_link_or_button 'Add deposit' + fill_in 'Amount', with: 100 + click_link_or_button 'Add' + + assert_text 'Please pay the following invoice' + end +end From c129598d2b443a03b473bf3ae3acfe825e08774b Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sat, 10 Mar 2018 16:51:11 +0200 Subject: [PATCH 33/87] Improve VAT rate validation #623 --- app/models/invoice.rb | 5 +++-- app/models/registrar.rb | 2 +- test/models/invoice_test.rb | 20 ++++++++------------ 3 files changed, 12 insertions(+), 15 deletions(-) diff --git a/app/models/invoice.rb b/app/models/invoice.rb index dfdafee6d..165b7f614 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -28,8 +28,9 @@ class Invoice < ActiveRecord::Base validates :billing_email, email_format: { message: :invalid }, allow_blank: true validates :due_date, :currency, :seller_name, - :seller_iban, :buyer_name, :invoice_items, :vat_rate, presence: true - validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 99 }, allow_nil: true + :seller_iban, :buyer_name, :invoice_items, presence: true + validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than: 100 }, + allow_nil: true before_create :set_invoice_number, :check_vat diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 0c7559ce8..0f9cdae06 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -21,7 +21,7 @@ class Registrar < ActiveRecord::Base validates :vat_rate, presence: true, if: :vat_rate_required? validates :vat_rate, absence: true, if: :local_vat_payer? validates :vat_rate, absence: true, if: 'foreign_vat_payer? && vat_no?' - validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 99.9 }, + validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than: 100 }, allow_nil: true validate :forbid_special_code diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index 770a88590..4a2457be3 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -9,26 +9,22 @@ class InvoiceTest < ActiveSupport::TestCase assert @invoice.valid? end - def test_invalid_without_vat_rate + def test_valid_without_vat_rate @invoice.vat_rate = nil - assert @invoice.invalid? - end - - def test_allows_absent_vat_rate - @invoice.vat_rate = nil - @invoice.validate assert @invoice.valid? end - def test_rejects_negative_vat_rate + def test_vat_rate_validation @invoice.vat_rate = -1 - @invoice.validate assert @invoice.invalid? - end - def test_rejects_vat_rate_greater_than_max + @invoice.vat_rate = 1 + assert @invoice.valid? + + @invoice.vat_rate = 99.9 + assert @invoice.valid? + @invoice.vat_rate = 100 - @invoice.validate assert @invoice.invalid? end end From cf2aa76cc62b3d14ef46241822a5eb3dbe4a1c87 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sat, 10 Mar 2018 16:57:24 +0200 Subject: [PATCH 34/87] Regenerate structure.sql #623 --- db/structure.sql | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/db/structure.sql b/db/structure.sql index 70cb7e5b2..d54caef91 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -3243,7 +3243,7 @@ ALTER TABLE ONLY settings -- --- Name: unique_code; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- Name: unique_code; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: -- ALTER TABLE ONLY registrars @@ -3259,7 +3259,7 @@ ALTER TABLE ONLY contacts -- --- Name: unique_name; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- Name: unique_name; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: -- ALTER TABLE ONLY registrars @@ -3267,7 +3267,7 @@ ALTER TABLE ONLY registrars -- --- Name: unique_reference_no; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- Name: unique_reference_no; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: -- ALTER TABLE ONLY registrars @@ -3275,7 +3275,7 @@ ALTER TABLE ONLY registrars -- --- Name: unique_reg_no; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: +-- Name: unique_reg_no; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: -- ALTER TABLE ONLY registrars From 026c36d066f2d50216dc4414655263dab0e4fde7 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sun, 11 Mar 2018 13:35:33 +0200 Subject: [PATCH 35/87] Fix vat rate form field constraints #623 --- app/views/admin/registrars/form/_billing.html.erb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/views/admin/registrars/form/_billing.html.erb b/app/views/admin/registrars/form/_billing.html.erb index e644085cf..c87da8b6c 100644 --- a/app/views/admin/registrars/form/_billing.html.erb +++ b/app/views/admin/registrars/form/_billing.html.erb @@ -23,7 +23,7 @@
- <%= f.number_field :vat_rate, min: 0, max: 99, step: 0.1, + <%= f.number_field :vat_rate, min: 0, max: 99.9, step: 0.1, class: 'form-control' %>
%
From 7d72d9cb34fc05dbf95dded17837e3a187f7f3b8 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sun, 11 Mar 2018 19:33:21 +0200 Subject: [PATCH 36/87] Fix vat rate #623 --- app/models/invoice.rb | 19 ++++--- app/models/registrar.rb | 31 +++++------ app/models/type/vat_rate.rb | 14 ++--- test/fixtures/invoices.yml | 1 + test/fixtures/registrars.yml | 3 +- test/models/invoice/vat_rate_test.rb | 51 ++++++++++++++++++ test/models/invoice_test.rb | 19 ------- test/models/registrar/vat_test.rb | 77 +++++++++++++++------------- 8 files changed, 121 insertions(+), 94 deletions(-) create mode 100644 test/models/invoice/vat_rate_test.rb diff --git a/app/models/invoice.rb b/app/models/invoice.rb index 165b7f614..d78c24d07 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -32,9 +32,11 @@ class Invoice < ActiveRecord::Base validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than: 100 }, allow_nil: true - before_create :set_invoice_number, :check_vat + after_initialize :apply_defaults + before_create :set_invoice_number - before_save :check_vat + attribute :vat_rate, ::Type::VATRate.new + attr_readonly :vat_rate def set_invoice_number last_no = Invoice.order(number: :desc).where('number IS NOT NULL').limit(1).pluck(:number).first @@ -52,12 +54,6 @@ class Invoice < ActiveRecord::Base false end - def check_vat - if buyer.country_code != 'EE' && buyer.vat_no.present? - self.vat_rate = 0 - end - end - before_save -> { self.sum_cache = sum } class << self @@ -159,10 +155,17 @@ class Invoice < ActiveRecord::Base end def vat + return 0 unless vat_rate (sum_without_vat * vat_rate).round(2) end def sum (sum_without_vat + vat).round(2) end + + private + + def apply_defaults + self.vat_rate = buyer.effective_vat_rate unless vat_rate + end end diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 0f9cdae06..9e7d84a7b 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -18,8 +18,8 @@ class Registrar < ActiveRecord::Base validates :accounting_customer_code, presence: true validates :language, presence: true - validates :vat_rate, presence: true, if: :vat_rate_required? - validates :vat_rate, absence: true, if: :local_vat_payer? + validates :vat_rate, presence: true, if: 'foreign_vat_payer? && vat_no.blank?' + validates :vat_rate, absence: true, if: :home_vat_payer? validates :vat_rate, absence: true, if: 'foreign_vat_payer? && vat_no?' validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than: 100 }, allow_nil: true @@ -56,22 +56,11 @@ class Registrar < ActiveRecord::Base # rubocop:disable Metrics/MethodLength # rubocop:disable Metrics/AbcSize def issue_prepayment_invoice(amount, description = nil) - vat_rate = if local_vat_payer? - Registry.instance.vat_rate - else - if vat_no.blank? - self.vat_rate - else - nil - end - end - invoices.create( due_date: (Time.zone.now.to_date + Setting.days_to_keep_invoices_active.days).end_of_day, payment_term: 'prepayment', description: description, currency: 'EUR', - vat_rate: vat_rate, seller_name: Setting.registry_juridical_name, seller_reg_no: Setting.registry_reg_no, seller_iban: Setting.registry_iban, @@ -157,6 +146,14 @@ class Registrar < ActiveRecord::Base end end + def effective_vat_rate + if home_vat_payer? + Registry.instance.vat_rate + else + vat_rate + end + end + private def set_defaults @@ -188,15 +185,11 @@ class Registrar < ActiveRecord::Base end end - def local_vat_payer? + def home_vat_payer? country == Registry.instance.legal_address_country end def foreign_vat_payer? - !local_vat_payer? - end - - def vat_rate_required? - foreign_vat_payer? && vat_no.blank? + !home_vat_payer? end end diff --git a/app/models/type/vat_rate.rb b/app/models/type/vat_rate.rb index 466b076f8..5ee993211 100644 --- a/app/models/type/vat_rate.rb +++ b/app/models/type/vat_rate.rb @@ -1,19 +1,11 @@ module Type - class VATRate < ActiveRecord::Type::Value - def type_cast_from_user(value) - if value.blank? - nil - else - super - end - end - + class VATRate < ActiveRecord::Type::Decimal def type_cast_from_database(value) - BigDecimal(value) * 100 if value + super * 100 if value end def type_cast_for_database(value) - BigDecimal(value) / 100.0 if value + super / 100.0 if value end end end diff --git a/test/fixtures/invoices.yml b/test/fixtures/invoices.yml index 6fbfa282b..ffb53f938 100644 --- a/test/fixtures/invoices.yml +++ b/test/fixtures/invoices.yml @@ -4,6 +4,7 @@ DEFAULTS: &DEFAULTS currency: EUR seller_name: John Doe seller_iban: 1234 + buyer: bestnames buyer_name: Jane Doe vat_rate: 0.2 diff --git a/test/fixtures/registrars.yml b/test/fixtures/registrars.yml index b0cc371a7..a0faee8b4 100644 --- a/test/fixtures/registrars.yml +++ b/test/fixtures/registrars.yml @@ -17,7 +17,8 @@ goodnames: reg_no: 12345 code: goodnames email: info@goodnames.test - country_code: US + country_code: DE + vat_no: DE123456789 accounting_customer_code: goodnames language: en diff --git a/test/models/invoice/vat_rate_test.rb b/test/models/invoice/vat_rate_test.rb new file mode 100644 index 000000000..cb2bd9f19 --- /dev/null +++ b/test/models/invoice/vat_rate_test.rb @@ -0,0 +1,51 @@ +require 'test_helper' + +class InvoiceVATRateTest < ActiveSupport::TestCase + def setup + @invoice = invoices(:valid) + end + + def test_valid_without_vat_rate + @invoice.vat_rate = nil + assert @invoice.valid? + end + + def test_vat_rate_validation + @invoice.vat_rate = -1 + assert @invoice.invalid? + + @invoice.vat_rate = 0 + assert @invoice.valid? + + @invoice.vat_rate = 99.9 + assert @invoice.valid? + + @invoice.vat_rate = 100 + assert @invoice.invalid? + end + + def test_serializes_and_deserializes_vat_rate + invoice = @invoice.dup + invoice.invoice_items = @invoice.invoice_items + invoice.vat_rate = BigDecimal('25.5') + invoice.save! + invoice.reload + assert_equal BigDecimal('25.5'), invoice.vat_rate + end + + def test_vat_rate_defaults_to_effective_vat_rate_of_a_registrar + registrar = registrars(:bestnames) + + registrar.stub(:effective_vat_rate, 55) do + invoice = Invoice.new(buyer: registrar) + assert_equal 55, invoice.vat_rate + end + end + + def test_vat_rate_cannot_be_updated + @invoice.vat_rate = 21 + @invoice.save! + @invoice.reload + refute_equal 21, @invoice.vat_rate + end +end diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index 4a2457be3..29af6fab8 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -8,23 +8,4 @@ class InvoiceTest < ActiveSupport::TestCase def test_valid assert @invoice.valid? end - - def test_valid_without_vat_rate - @invoice.vat_rate = nil - assert @invoice.valid? - end - - def test_vat_rate_validation - @invoice.vat_rate = -1 - assert @invoice.invalid? - - @invoice.vat_rate = 1 - assert @invoice.valid? - - @invoice.vat_rate = 99.9 - assert @invoice.valid? - - @invoice.vat_rate = 100 - assert @invoice.invalid? - end end diff --git a/test/models/registrar/vat_test.rb b/test/models/registrar/vat_test.rb index cf36b6266..802a0d987 100644 --- a/test/models/registrar/vat_test.rb +++ b/test/models/registrar/vat_test.rb @@ -13,28 +13,6 @@ class RegistrarVATTest < ActiveSupport::TestCase assert @registrar.valid? end - def test_requires_vat_rate_when_registrar_is_foreign_vat_payer_and_vat_no_is_absent - @registrar.vat_no = '' - - Registry.instance.stub(:legal_address_country, Country.new('GB')) do - @registrar.vat_rate = '' - assert @registrar.invalid? - assert @registrar.errors.added?(:vat_rate, :blank) - - @registrar.vat_rate = -1 - assert @registrar.invalid? - - @registrar.vat_rate = 1 - assert @registrar.valid? - - @registrar.vat_rate = 99.9 - assert @registrar.valid? - - @registrar.vat_rate = 100 - assert @registrar.invalid? - end - end - def test_vat_is_not_applied_when_registrar_is_local_vat_payer @registrar.vat_rate = 1 assert @registrar.invalid? @@ -43,30 +21,57 @@ class RegistrarVATTest < ActiveSupport::TestCase assert @registrar.valid? end + def test_requires_vat_rate_when_registrar_is_foreign_vat_payer_and_vat_no_is_absent + @registrar.country_code = 'DE' + @registrar.vat_no = '' + + @registrar.vat_rate = '' + assert @registrar.invalid? + assert @registrar.errors.added?(:vat_rate, :blank) + end + def test_vat_is_not_applied_when_registrar_is_foreign_vat_payer_and_vat_no_is_present + @registrar.country_code = 'DE' @registrar.vat_no = 'valid' - Registry.instance.stub(:legal_address_country, Country.new('GB')) do - @registrar.vat_rate = 1 - assert @registrar.invalid? + @registrar.vat_rate = 1 + assert @registrar.invalid? - @registrar.vat_rate = nil - assert @registrar.valid? - end + @registrar.vat_rate = nil + assert @registrar.valid? + end + + def test_vat_rate_validation + @registrar.country_code = 'DE' + @registrar.vat_no = '' + + @registrar.vat_rate = -1 + assert @registrar.invalid? + + @registrar.vat_rate = 0 + assert @registrar.valid? + + @registrar.vat_rate = 99.9 + assert @registrar.valid? + + @registrar.vat_rate = 100 + assert @registrar.invalid? end def test_serializes_and_deserializes_vat_rate - @registrar.vat_rate = '25.5' - - Registry.instance.stub(:legal_address_country, Country.new('GB')) do - @registrar.save! - end - + @registrar.country_code = 'DE' + @registrar.vat_rate = BigDecimal('25.5') + @registrar.save! @registrar.reload - assert_equal 25.5, @registrar.vat_rate + assert_equal BigDecimal('25.5'), @registrar.vat_rate end - def test_treats_empty_vat_rate_as_absent + def test_parses_vat_rate_as_a_string + @registrar.vat_rate = '25.5' + assert_equal BigDecimal('25.5'), @registrar.vat_rate + end + + def test_treats_empty_vat_rate_as_nil @registrar.vat_rate = '' assert_nil @registrar.vat_rate end From 4a63a60433360bbe7eb8c065d31795ca23b81586 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 12 Mar 2018 10:17:15 +0200 Subject: [PATCH 37/87] Return VAT rate as BigDecimal #623 --- app/models/registry.rb | 2 +- test/models/registry_test.rb | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/models/registry.rb b/app/models/registry.rb index 162ab4d71..5e5c8cd93 100644 --- a/app/models/registry.rb +++ b/app/models/registry.rb @@ -2,7 +2,7 @@ class Registry include Singleton def vat_rate - Setting.registry_vat_prc + Setting.registry_vat_prc.to_d * 100 end def legal_address_country diff --git a/test/models/registry_test.rb b/test/models/registry_test.rb index 2f25686c6..70bc037c6 100644 --- a/test/models/registry_test.rb +++ b/test/models/registry_test.rb @@ -10,7 +10,7 @@ class RegistryTest < ActiveSupport::TestCase end def test_vat_rate - Setting.registry_vat_prc = 0.2 - assert 0.2, @registry.vat_rate + Setting.registry_vat_prc = 0.25 + assert_equal BigDecimal(25), @registry.vat_rate end end From 57ab19d86adc88c74bd04f84c834754e4228abbe Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 12 Mar 2018 11:34:41 +0200 Subject: [PATCH 38/87] Improve readability #623 --- test/models/invoice/vat_rate_test.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/models/invoice/vat_rate_test.rb b/test/models/invoice/vat_rate_test.rb index cb2bd9f19..27bc4b705 100644 --- a/test/models/invoice/vat_rate_test.rb +++ b/test/models/invoice/vat_rate_test.rb @@ -5,7 +5,7 @@ class InvoiceVATRateTest < ActiveSupport::TestCase @invoice = invoices(:valid) end - def test_valid_without_vat_rate + def test_optional_vat_rate @invoice.vat_rate = nil assert @invoice.valid? end From 7cc479c3fe34b4a9a3ccaaa3e9f73852ae7a95bf Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 12 Mar 2018 11:35:34 +0200 Subject: [PATCH 39/87] Update fixtures #623 --- test/fixtures/invoice_items.yml | 18 +++++++++--------- test/fixtures/invoices.yml | 2 +- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/test/fixtures/invoice_items.yml b/test/fixtures/invoice_items.yml index 417203a51..9c37ece9a 100644 --- a/test/fixtures/invoice_items.yml +++ b/test/fixtures/invoice_items.yml @@ -1,13 +1,13 @@ -DEFAULTS: &DEFAULTS +one: description: Acme services + price: 5 + amount: 1 + unit: pc + invoice: valid + +two: + description: Acme services + price: 5 amount: 2 unit: pc - price: 5 - -valid: - <<: *DEFAULTS - invoice: valid - -another: - <<: *DEFAULTS invoice: valid diff --git a/test/fixtures/invoices.yml b/test/fixtures/invoices.yml index ffb53f938..7d532cd87 100644 --- a/test/fixtures/invoices.yml +++ b/test/fixtures/invoices.yml @@ -6,7 +6,7 @@ DEFAULTS: &DEFAULTS seller_iban: 1234 buyer: bestnames buyer_name: Jane Doe - vat_rate: 0.2 + vat_rate: 0.1 valid: <<: *DEFAULTS From 8d4b2d902b4878376425c51cb6d5833ddc4034b4 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 12 Mar 2018 11:36:07 +0200 Subject: [PATCH 40/87] Convert specs to tests #623 --- spec/models/invoice_spec.rb | 14 -------------- test/models/invoice_test.rb | 12 ++++++++++++ 2 files changed, 12 insertions(+), 14 deletions(-) diff --git a/spec/models/invoice_spec.rb b/spec/models/invoice_spec.rb index d6cdc2725..23f91f741 100644 --- a/spec/models/invoice_spec.rb +++ b/spec/models/invoice_spec.rb @@ -51,20 +51,6 @@ describe Invoice do @invoice.seller_address.should == 'Paldiski mnt. 123, Tallinn' end - it 'should calculate sums correctly' do - @invoice = create(:invoice) - @invoice.vat_rate.should == BigDecimal.new('0.2') - @invoice.sum_without_vat.should == BigDecimal.new('300.0') - @invoice.vat.should == BigDecimal.new('60.0') - @invoice.sum.should == BigDecimal.new('360.0') - - ii = @invoice.items.first - ii.item_sum_without_vat.should == BigDecimal.new('150.0') - - ii = @invoice.items.last - ii.item_sum_without_vat.should == BigDecimal.new('150.0') - end - it 'should cancel overdue invoices' do create(:invoice, created_at: Time.zone.now - 35.days, due_date: Time.zone.now - 30.days) Invoice.cancel_overdue_invoices diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index 29af6fab8..f7f2900aa 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -8,4 +8,16 @@ class InvoiceTest < ActiveSupport::TestCase def test_valid assert @invoice.valid? end + + def test_calculates_subtotal + assert_equal BigDecimal('15'), @invoice.sum_without_vat + end + + def test_calculates_vat_amount + assert_equal BigDecimal('1.5'), @invoice.vat + end + + def test_calculates_total + assert_equal BigDecimal('16.5'), @invoice.sum + end end From 8de70735f628f2075d3216f5cd763026ff98ac9f Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 12 Mar 2018 12:19:06 +0200 Subject: [PATCH 41/87] Fix spec #623 --- spec/models/invoice_spec.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/spec/models/invoice_spec.rb b/spec/models/invoice_spec.rb index 23f91f741..14747c9c1 100644 --- a/spec/models/invoice_spec.rb +++ b/spec/models/invoice_spec.rb @@ -15,7 +15,6 @@ describe Invoice do "Invoice items is missing", "Seller iban is missing", "Seller name is missing", - "Vat prc is missing" ]) end From 6b54dd520d9c7cbc93fbdd3f12e522d23e2f629f Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 12 Mar 2018 12:19:20 +0200 Subject: [PATCH 42/87] Enhance test #623 --- .../registrar/billing/balance_top_up_test.rb | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/test/integration/registrar/billing/balance_top_up_test.rb b/test/integration/registrar/billing/balance_top_up_test.rb index c4dd1cf5d..36109e4c4 100644 --- a/test/integration/registrar/billing/balance_top_up_test.rb +++ b/test/integration/registrar/billing/balance_top_up_test.rb @@ -5,12 +5,20 @@ class BalanceTopUpTest < ActionDispatch::IntegrationTest login_as users(:api_bestnames) end - def test_registrar_balance_top_up + def test_creates_new_invoice visit registrar_invoices_url click_link_or_button 'Add deposit' - fill_in 'Amount', with: 100 - click_link_or_button 'Add' + fill_in 'Amount', with: '25.5' + Registry.instance.stub(:vat_rate, 10) do + assert_difference 'Invoice.count' do + click_link_or_button 'Add' + end + end + + invoice = Invoice.last + + assert_equal BigDecimal('28.05'), invoice.sum_cache assert_text 'Please pay the following invoice' end end From 6dfd57ee2e979b4b42deddd4a74ff6dbd8a22bb6 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 08:47:08 +0200 Subject: [PATCH 43/87] Remove tests #623 --- test/fixtures/invoices.yml | 8 -------- .../admin/invoices/invoice_details_test.rb | 19 ------------------- .../invoices/invoice_details_test.rb | 19 ------------------- 3 files changed, 46 deletions(-) delete mode 100644 test/integration/admin/invoices/invoice_details_test.rb delete mode 100644 test/integration/registrar/invoices/invoice_details_test.rb diff --git a/test/fixtures/invoices.yml b/test/fixtures/invoices.yml index 7d532cd87..6d1e0fc73 100644 --- a/test/fixtures/invoices.yml +++ b/test/fixtures/invoices.yml @@ -30,11 +30,3 @@ outstanding: overdue: <<: *DEFAULTS due_date: <%= Date.parse '2010-07-03' %> - -with_vat: - <<: *DEFAULTS - vat_rate: 0.2 - -without_vat: - <<: *DEFAULTS - vat_rate: null diff --git a/test/integration/admin/invoices/invoice_details_test.rb b/test/integration/admin/invoices/invoice_details_test.rb deleted file mode 100644 index 8a67771ea..000000000 --- a/test/integration/admin/invoices/invoice_details_test.rb +++ /dev/null @@ -1,19 +0,0 @@ -require 'test_helper' - -class InvoiceDetailsTest < ActionDispatch::IntegrationTest - def setup - login_as users(:admin) - end - - def test_with_vat - invoice = invoices(:with_vat) - visit admin_invoice_path(invoice) - assert_selector '.total', text: 'VAT' - end - - def test_without_vat - invoice = invoices(:without_vat) - visit admin_invoice_path(invoice) - assert_no_selector '.total', text: 'VAT' - end -end diff --git a/test/integration/registrar/invoices/invoice_details_test.rb b/test/integration/registrar/invoices/invoice_details_test.rb deleted file mode 100644 index d3961faf3..000000000 --- a/test/integration/registrar/invoices/invoice_details_test.rb +++ /dev/null @@ -1,19 +0,0 @@ -require 'test_helper' - -class InvoiceDetailsTest < ActionDispatch::IntegrationTest - def setup - login_as users(:api) - end - - def test_with_vat - invoice = invoices(:with_vat) - visit registrar_invoice_path(invoice) - assert_selector '.total', text: 'VAT' - end - - def test_without_vat - invoice = invoices(:without_vat) - visit registrar_invoice_path(invoice) - assert_no_selector '.total', text: 'VAT' - end -end From a73a8a0b89116579f91be50913b77cf31a1de91a Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 09:02:08 +0200 Subject: [PATCH 44/87] Fix VAT rate #623 --- app/models/invoice.rb | 8 ++++---- app/views/registrar/invoices/partials/_items.haml | 2 +- app/views/registrar/invoices/pdf.haml | 2 +- config/locales/en.yml | 2 +- test/models/invoice/vat_rate_test.rb | 15 ++++++++++----- 5 files changed, 17 insertions(+), 12 deletions(-) diff --git a/app/models/invoice.rb b/app/models/invoice.rb index d78c24d07..7dd8bbe67 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -32,8 +32,8 @@ class Invoice < ActiveRecord::Base validates :vat_rate, numericality: { greater_than_or_equal_to: 0, less_than: 100 }, allow_nil: true - after_initialize :apply_defaults before_create :set_invoice_number + before_create :apply_default_vat_rate, unless: :vat_rate? attribute :vat_rate, ::Type::VATRate.new attr_readonly :vat_rate @@ -156,7 +156,7 @@ class Invoice < ActiveRecord::Base def vat return 0 unless vat_rate - (sum_without_vat * vat_rate).round(2) + sum_without_vat * vat_rate / 100 end def sum @@ -165,7 +165,7 @@ class Invoice < ActiveRecord::Base private - def apply_defaults - self.vat_rate = buyer.effective_vat_rate unless vat_rate + def apply_default_vat_rate + self.vat_rate = buyer.effective_vat_rate end end diff --git a/app/views/registrar/invoices/partials/_items.haml b/app/views/registrar/invoices/partials/_items.haml index 64fc71786..eef06c1ce 100644 --- a/app/views/registrar/invoices/partials/_items.haml +++ b/app/views/registrar/invoices/partials/_items.haml @@ -24,7 +24,7 @@ %td= currency(@invoice.sum_without_vat) %tr %th.no-border{colspan: 3} - %th= t('vat', rate: (@invoice.vat_rate * 100).round) + %th= t('vat', rate: number_to_percentage(@invoice.vat_rate, precision: 1)) %td= currency(@invoice.vat) %tr %th.no-border{colspan: 3} diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index 6f92c4a8a..a5f035fe0 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -242,7 +242,7 @@ %td= "#{currency(@invoice.sum_without_vat)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} - %th= t('vat', rate: (@invoice.vat_rate * 100).round) + %th= t('vat', rate: number_to_percentage(@invoice.vat_rate, precision: 1)) %td= "#{currency(@invoice.vat)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} diff --git a/config/locales/en.yml b/config/locales/en.yml index 19f62fe3f..73045367b 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -543,7 +543,7 @@ en: invoice_number: Invoice no. seller: 'Seller' prepayment: 'Prepayment' - vat: 'VAT (%{rate}%)' + vat: 'VAT (%{rate})' unpaid: 'Unpaid' your_current_account_balance_is: 'Your current account balance is %{balance} %{currency}' billing: 'Billing' diff --git a/test/models/invoice/vat_rate_test.rb b/test/models/invoice/vat_rate_test.rb index 27bc4b705..d1e892b72 100644 --- a/test/models/invoice/vat_rate_test.rb +++ b/test/models/invoice/vat_rate_test.rb @@ -35,17 +35,22 @@ class InvoiceVATRateTest < ActiveSupport::TestCase def test_vat_rate_defaults_to_effective_vat_rate_of_a_registrar registrar = registrars(:bestnames) + invoice = @invoice.dup + invoice.vat_rate = nil + invoice.buyer = registrar + invoice.invoice_items = @invoice.invoice_items - registrar.stub(:effective_vat_rate, 55) do - invoice = Invoice.new(buyer: registrar) - assert_equal 55, invoice.vat_rate + registrar.stub(:effective_vat_rate, BigDecimal(55)) do + invoice.save! end + + assert_equal BigDecimal(55), invoice.vat_rate end def test_vat_rate_cannot_be_updated - @invoice.vat_rate = 21 + @invoice.vat_rate = BigDecimal(21) @invoice.save! @invoice.reload - refute_equal 21, @invoice.vat_rate + refute_equal BigDecimal(21), @invoice.vat_rate end end From 3d107bd198b7d566d0388cef283837ef77772a07 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 09:39:32 +0200 Subject: [PATCH 45/87] Set association instead of foreign key #623 --- app/models/registrar.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 9e7d84a7b..9cba29faf 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -76,7 +76,7 @@ class Registrar < ActiveRecord::Base seller_url: Setting.registry_url, seller_email: Setting.registry_email, seller_contact_name: Setting.registry_invoice_contact, - buyer_id: id, + buyer: self, buyer_name: name, buyer_reg_no: reg_no, buyer_country_code: country_code, From b68fe6e3124b28f12d805247d48d91212e5c41c3 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 11:32:15 +0200 Subject: [PATCH 46/87] Rename attributes #623 --- .../admin/bank_statements_controller.rb | 2 +- .../registrar/invoices_controller.rb | 4 +- app/models/bank_link.rb | 4 +- app/models/bank_transaction.rb | 6 +- app/models/directo.rb | 4 +- app/models/invoice.rb | 15 ++-- app/views/registrar/invoices/index.haml | 6 +- .../registrar/invoices/partials/_items.haml | 4 +- app/views/registrar/invoices/pdf.haml | 4 +- ...0437_rename_invoices_sum_cache_to_total.rb | 5 ++ db/structure.sql | 4 +- doc/models_complete.svg | 2 +- test/fixtures/invoices.yml | 3 +- .../registrar/billing/balance_top_up_test.rb | 11 +-- test/models/invoice/vat_rate_test.rb | 56 --------------- test/models/invoice_test.rb | 72 ++++++++++++++++++- 16 files changed, 110 insertions(+), 92 deletions(-) create mode 100644 db/migrate/20180313090437_rename_invoices_sum_cache_to_total.rb delete mode 100644 test/models/invoice/vat_rate_test.rb diff --git a/app/controllers/admin/bank_statements_controller.rb b/app/controllers/admin/bank_statements_controller.rb index d7b6edae2..d7bb81fb1 100644 --- a/app/controllers/admin/bank_statements_controller.rb +++ b/app/controllers/admin/bank_statements_controller.rb @@ -24,7 +24,7 @@ module Admin @invoice = Invoice.find_by(id: params[:invoice_id]) @bank_transaction = @bank_statement.bank_transactions.build( description: @invoice.to_s, - sum: @invoice.sum, + sum: @invoice.amount, reference_no: @invoice.reference_no, paid_at: Time.zone.now.to_date, currency: 'EUR' diff --git a/app/controllers/registrar/invoices_controller.rb b/app/controllers/registrar/invoices_controller.rb index ac762e712..735df91a3 100644 --- a/app/controllers/registrar/invoices_controller.rb +++ b/app/controllers/registrar/invoices_controller.rb @@ -55,8 +55,8 @@ class Registrar end def normalize_search_parameters - params[:q][:sum_cache_gteq].gsub!(',', '.') if params[:q][:sum_cache_gteq] - params[:q][:sum_cache_lteq].gsub!(',', '.') if params[:q][:sum_cache_lteq] + params[:q][:total_gteq].gsub!(',', '.') if params[:q][:total_gteq] + params[:q][:total_lteq].gsub!(',', '.') if params[:q][:total_lteq] ca_cache = params[:q][:due_date_lteq] begin diff --git a/app/models/bank_link.rb b/app/models/bank_link.rb index 24c94a771..e388a0f8b 100644 --- a/app/models/bank_link.rb +++ b/app/models/bank_link.rb @@ -29,7 +29,7 @@ class BankLink hash["VK_VERSION"] = "008" hash["VK_SND_ID"] = ENV["payments_#{type}_seller_account"] hash["VK_STAMP"] = invoice.number - hash["VK_AMOUNT"] = number_with_precision(invoice.sum_cache, :precision => 2, :separator => ".") + hash["VK_AMOUNT"] = number_with_precision(invoice.total, :precision => 2, :separator => ".") hash["VK_CURR"] = invoice.currency hash["VK_REF"] = "" hash["VK_MSG"] = invoice.order @@ -140,7 +140,7 @@ class BankLink def validate_amount source = number_with_precision(BigDecimal.new(params["VK_AMOUNT"].to_s), precision: 2, separator: ".") - target = number_with_precision(invoice.sum_cache, precision: 2, separator: ".") + target = number_with_precision(invoice.total, precision: 2, separator: ".") source == target end diff --git a/app/models/bank_transaction.rb b/app/models/bank_transaction.rb index daf6abc29..3749f92b5 100644 --- a/app/models/bank_transaction.rb +++ b/app/models/bank_transaction.rb @@ -47,7 +47,7 @@ class BankTransaction < ActiveRecord::Base return if invoice.binded? return if invoice.cancelled? - return if invoice.sum != sum + return if invoice.total != sum create_activity(registrar, invoice) end # rubocop: enable Metrics/PerceivedComplexity @@ -76,7 +76,7 @@ class BankTransaction < ActiveRecord::Base return end - if invoice.sum != sum + if invoice.total != sum errors.add(:base, I18n.t('invoice_and_transaction_sums_do_not_match')) return end @@ -88,7 +88,7 @@ class BankTransaction < ActiveRecord::Base create_account_activity( account: registrar.cash_account, invoice: invoice, - sum: invoice.sum_without_vat, + sum: invoice.subtotal, currency: currency, description: description, activity_type: AccountActivity::ADD_CREDIT diff --git a/app/models/directo.rb b/app/models/directo.rb index f3a506ff3..2b794b0e4 100644 --- a/app/models/directo.rb +++ b/app/models/directo.rb @@ -15,7 +15,7 @@ class Directo < ActiveRecord::Base group.each do |invoice| if invoice.account_activity.nil? || invoice.account_activity.bank_transaction.nil? || - invoice.account_activity.bank_transaction.sum.nil? || invoice.account_activity.bank_transaction.sum != invoice.sum_cache + invoice.account_activity.bank_transaction.sum.nil? || invoice.account_activity.bank_transaction.sum != invoice.total Rails.logger.info("[DIRECTO] Invoice #{invoice.number} has been skipped") next end @@ -34,7 +34,7 @@ class Directo < ActiveRecord::Base xml.line( "ProductID" => Setting.directo_receipt_product_name, "Quantity" => 1, - "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.sum_cache/(1+invoice.vat_rate), precision: 2, separator: "."), + "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.total/(1+invoice.vat_rate), precision: 2, separator: "."), "ProductName" => invoice.order ) } diff --git a/app/models/invoice.rb b/app/models/invoice.rb index 7dd8bbe67..19e25d5c9 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -34,6 +34,7 @@ class Invoice < ActiveRecord::Base before_create :set_invoice_number before_create :apply_default_vat_rate, unless: :vat_rate? + before_create :save_total attribute :vat_rate, ::Type::VATRate.new attr_readonly :vat_rate @@ -54,8 +55,6 @@ class Invoice < ActiveRecord::Base false end - before_save -> { self.sum_cache = sum } - class << self def cancel_overdue_invoices STDOUT << "#{Time.zone.now.utc} - Cancelling overdue invoices\n" unless Rails.env.test? @@ -150,17 +149,13 @@ class Invoice < ActiveRecord::Base invoice_items end - def sum_without_vat + def subtotal (items.map(&:item_sum_without_vat).sum).round(2) end def vat return 0 unless vat_rate - sum_without_vat * vat_rate / 100 - end - - def sum - (sum_without_vat + vat).round(2) + subtotal * vat_rate / 100 end private @@ -168,4 +163,8 @@ class Invoice < ActiveRecord::Base def apply_default_vat_rate self.vat_rate = buyer.effective_vat_rate end + + def calculate_total + # (sum_without_vat + vat).round(2) + end end diff --git a/app/views/registrar/invoices/index.haml b/app/views/registrar/invoices/index.haml index b8c825c36..61d955708 100644 --- a/app/views/registrar/invoices/index.haml +++ b/app/views/registrar/invoices/index.haml @@ -33,11 +33,11 @@ .col-md-3 .form-group = f.label t(:minimum_total) - = f.search_field :sum_cache_gteq, class: 'form-control', placeholder: t(:minimum_total), autocomplete: 'off' + = f.search_field :total_gteq, class: 'form-control', placeholder: t(:minimum_total), autocomplete: 'off' .col-md-3 .form-group = f.label t(:maximum_total) - = f.search_field :sum_cache_lteq, class: 'form-control', placeholder: t(:maximum_total), autocomplete: 'off' + = f.search_field :total_lteq, class: 'form-control', placeholder: t(:maximum_total), autocomplete: 'off' .col-md-3{style: 'padding-top: 25px;'} %button.btn.btn-default   @@ -67,7 +67,7 @@ %td{class: 'text-danger'}= t(:unpaid) %td= l(x.due_date, format: :date_long) - %td= currency(x.sum) + %td= currency(x.total) .row .col-md-12 = paginate @invoices diff --git a/app/views/registrar/invoices/partials/_items.haml b/app/views/registrar/invoices/partials/_items.haml index eef06c1ce..560b3e1f6 100644 --- a/app/views/registrar/invoices/partials/_items.haml +++ b/app/views/registrar/invoices/partials/_items.haml @@ -21,7 +21,7 @@ %tr %th{colspan: 3} %th= t(:total_without_vat) - %td= currency(@invoice.sum_without_vat) + %td= currency(@invoice.subtotal) %tr %th.no-border{colspan: 3} %th= t('vat', rate: number_to_percentage(@invoice.vat_rate, precision: 1)) @@ -29,4 +29,4 @@ %tr %th.no-border{colspan: 3} %th= t(:total) - %td= currency(@invoice.sum) + %td= currency(@invoice.total) diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index a5f035fe0..e59ca618b 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -239,7 +239,7 @@ %tr %th{colspan: 3} %th= t(:total_without_vat) - %td= "#{currency(@invoice.sum_without_vat)} #{@invoice.currency}" + %td= "#{currency(@invoice.subtotal)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} %th= t('vat', rate: number_to_percentage(@invoice.vat_rate, precision: 1)) @@ -247,7 +247,7 @@ %tr %th.no-border{colspan: 3} %th= t(:total) - %td= "#{currency(@invoice.sum)} #{@invoice.currency}" + %td= "#{currency(@invoice.total)} #{@invoice.currency}" #footer %hr diff --git a/db/migrate/20180313090437_rename_invoices_sum_cache_to_total.rb b/db/migrate/20180313090437_rename_invoices_sum_cache_to_total.rb new file mode 100644 index 000000000..43e827dfc --- /dev/null +++ b/db/migrate/20180313090437_rename_invoices_sum_cache_to_total.rb @@ -0,0 +1,5 @@ +class RenameInvoicesSumCacheToTotal < ActiveRecord::Migration + def change + rename_column :invoices, :sum_cache, :total + end +end diff --git a/db/structure.sql b/db/structure.sql index d54caef91..6df03a91a 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -1036,7 +1036,7 @@ CREATE TABLE invoices ( updator_str character varying, number integer, cancelled_at timestamp without time zone, - sum_cache numeric(10,2), + total numeric(10,2), in_directo boolean DEFAULT false ); @@ -4714,3 +4714,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180309054510'); INSERT INTO schema_migrations (version) VALUES ('20180310142630'); +INSERT INTO schema_migrations (version) VALUES ('20180313090437'); + diff --git a/doc/models_complete.svg b/doc/models_complete.svg index b3194a90a..235aa433b 100644 --- a/doc/models_complete.svg +++ b/doc/models_complete.svg @@ -1651,7 +1651,7 @@ updator_str :string number :integer cancelled_at :datetime -sum_cache :decimal +total :decimal Registrar->Invoice diff --git a/test/fixtures/invoices.yml b/test/fixtures/invoices.yml index 6d1e0fc73..c395da425 100644 --- a/test/fixtures/invoices.yml +++ b/test/fixtures/invoices.yml @@ -7,6 +7,7 @@ DEFAULTS: &DEFAULTS buyer: bestnames buyer_name: Jane Doe vat_rate: 0.1 + total: 16.50 valid: <<: *DEFAULTS @@ -21,7 +22,7 @@ cancelled: paid: <<: *DEFAULTS - sum_cache: 1 + total: 1 outstanding: <<: *DEFAULTS diff --git a/test/integration/registrar/billing/balance_top_up_test.rb b/test/integration/registrar/billing/balance_top_up_test.rb index 36109e4c4..3c36b5f42 100644 --- a/test/integration/registrar/billing/balance_top_up_test.rb +++ b/test/integration/registrar/billing/balance_top_up_test.rb @@ -6,19 +6,20 @@ class BalanceTopUpTest < ActionDispatch::IntegrationTest end def test_creates_new_invoice + Setting.registry_vat_prc = 0.1 + visit registrar_invoices_url click_link_or_button 'Add deposit' fill_in 'Amount', with: '25.5' - Registry.instance.stub(:vat_rate, 10) do - assert_difference 'Invoice.count' do - click_link_or_button 'Add' - end + assert_difference 'Invoice.count' do + click_link_or_button 'Add' end invoice = Invoice.last - assert_equal BigDecimal('28.05'), invoice.sum_cache + assert_equal BigDecimal(10), invoice.vat_rate + assert_equal BigDecimal('28.05'), invoice.total assert_text 'Please pay the following invoice' end end diff --git a/test/models/invoice/vat_rate_test.rb b/test/models/invoice/vat_rate_test.rb deleted file mode 100644 index d1e892b72..000000000 --- a/test/models/invoice/vat_rate_test.rb +++ /dev/null @@ -1,56 +0,0 @@ -require 'test_helper' - -class InvoiceVATRateTest < ActiveSupport::TestCase - def setup - @invoice = invoices(:valid) - end - - def test_optional_vat_rate - @invoice.vat_rate = nil - assert @invoice.valid? - end - - def test_vat_rate_validation - @invoice.vat_rate = -1 - assert @invoice.invalid? - - @invoice.vat_rate = 0 - assert @invoice.valid? - - @invoice.vat_rate = 99.9 - assert @invoice.valid? - - @invoice.vat_rate = 100 - assert @invoice.invalid? - end - - def test_serializes_and_deserializes_vat_rate - invoice = @invoice.dup - invoice.invoice_items = @invoice.invoice_items - invoice.vat_rate = BigDecimal('25.5') - invoice.save! - invoice.reload - assert_equal BigDecimal('25.5'), invoice.vat_rate - end - - def test_vat_rate_defaults_to_effective_vat_rate_of_a_registrar - registrar = registrars(:bestnames) - invoice = @invoice.dup - invoice.vat_rate = nil - invoice.buyer = registrar - invoice.invoice_items = @invoice.invoice_items - - registrar.stub(:effective_vat_rate, BigDecimal(55)) do - invoice.save! - end - - assert_equal BigDecimal(55), invoice.vat_rate - end - - def test_vat_rate_cannot_be_updated - @invoice.vat_rate = BigDecimal(21) - @invoice.save! - @invoice.reload - refute_equal BigDecimal(21), @invoice.vat_rate - end -end diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index f7f2900aa..fb7e66821 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -9,15 +9,81 @@ class InvoiceTest < ActiveSupport::TestCase assert @invoice.valid? end - def test_calculates_subtotal - assert_equal BigDecimal('15'), @invoice.sum_without_vat + def test_optional_vat_rate + @invoice.vat_rate = nil + assert @invoice.valid? + end + + def test_vat_rate_validation + @invoice.vat_rate = -1 + assert @invoice.invalid? + + @invoice.vat_rate = 0 + assert @invoice.valid? + + @invoice.vat_rate = 99.9 + assert @invoice.valid? + + @invoice.vat_rate = 100 + assert @invoice.invalid? + end + + def test_serializes_and_deserializes_vat_rate + invoice = @invoice.dup + invoice.invoice_items = @invoice.invoice_items + invoice.vat_rate = BigDecimal('25.5') + invoice.save! + invoice.reload + assert_equal BigDecimal('25.5'), invoice.vat_rate + end + + def test_vat_rate_defaults_to_effective_vat_rate_of_a_registrar + registrar = registrars(:bestnames) + invoice = @invoice.dup + invoice.vat_rate = nil + invoice.buyer = registrar + invoice.invoice_items = @invoice.invoice_items + + registrar.stub(:effective_vat_rate, BigDecimal(55)) do + invoice.save! + end + + assert_equal BigDecimal(55), invoice.vat_rate + end + + def test_vat_rate_cannot_be_updated + @invoice.vat_rate = BigDecimal(21) + @invoice.save! + @invoice.reload + refute_equal BigDecimal(21), @invoice.vat_rate end def test_calculates_vat_amount assert_equal BigDecimal('1.5'), @invoice.vat end + def test_vat_amount_is_zero_when_vat_rate_is_blank + @invoice.vat_rate = nil + assert_equal 0, @invoice.vat + end + + def test_calculates_subtotal + line_item = InvoiceItem.new + invoice = Invoice.new(invoice_items: [line_item, line_item]) + + line_item.stub(:item_sum_without_vat, BigDecimal('2.5')) do + assert_equal BigDecimal(5), invoice.subtotal + end + end + def test_calculates_total - assert_equal BigDecimal('16.5'), @invoice.sum + line_item = InvoiceItem.new + invoice = Invoice.new + invoice.vat_rate = 10 + invoice.invoice_items = [line_item, line_item] + + line_item.stub(:item_sum_without_vat, BigDecimal('2.5')) do + assert_equal BigDecimal('5.50'), invoice.total + end end end From b3c4f794d0ac48cff609e02ce03c333ebdd6c3be Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 11:32:35 +0200 Subject: [PATCH 47/87] Reformat #623 --- test/test_helper.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/test/test_helper.rb b/test/test_helper.rb index 64cef7e0f..868cd39c0 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -13,7 +13,6 @@ require 'capybara/minitest' require 'webmock/minitest' Setting.address_processing = false - Setting.registry_country_code = 'US' class ActiveSupport::TestCase From 3ed6416e5064dca0a52c72c106adfebfef35cd86 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 11:32:44 +0200 Subject: [PATCH 48/87] Fix test #623 --- test/integration/admin/registrars/new_test.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/integration/admin/registrars/new_test.rb b/test/integration/admin/registrars/new_test.rb index 61a7a43be..88ff76288 100644 --- a/test/integration/admin/registrars/new_test.rb +++ b/test/integration/admin/registrars/new_test.rb @@ -9,9 +9,10 @@ class AdminAreaNewRegistrarTest < ActionDispatch::IntegrationTest visit admin_registrars_url click_link_or_button 'New registrar' - fill_in 'Name', with: 'Brand new names' + fill_in 'Name', with: 'Brand new name' fill_in 'Reg no', with: '55555555' fill_in 'Contact e-mail', with: 'test@example.com' + select 'United States', from: 'Country' fill_in 'Accounting customer code', with: 'test' fill_in 'Code', with: 'test' From beba81bc6dbef7b249caa5f248a25922f156d679 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 11:39:18 +0200 Subject: [PATCH 49/87] Introduce helper method #623 --- app/helpers/application_helper.rb | 4 ++++ app/views/admin/registrars/_billing.html.erb | 2 +- app/views/registrar/invoices/partials/_items.haml | 2 +- app/views/registrar/invoices/pdf.haml | 2 +- test/integration/admin/registrars/show_registrar_test.rb | 2 +- 5 files changed, 8 insertions(+), 4 deletions(-) diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 8203a630f..f0e42344b 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -89,4 +89,8 @@ module ApplicationHelper types.delete('ddoc') ".#{types.join(',.')}" end + + def vat_rate(value) + number_to_percentage(value, precision: 1) + end end diff --git a/app/views/admin/registrars/_billing.html.erb b/app/views/admin/registrars/_billing.html.erb index 7ba778d08..929487afe 100644 --- a/app/views/admin/registrars/_billing.html.erb +++ b/app/views/admin/registrars/_billing.html.erb @@ -8,7 +8,7 @@
<%= registrar.vat_no %>
<%= Registrar.human_attribute_name :vat_rate %>
-
<%= number_to_percentage registrar.vat_rate, precision: 1 %>
+
<%= vat_rate registrar.vat_rate %>
<%= Registrar.human_attribute_name :accounting_customer_code %>
<%= registrar.accounting_customer_code %>
diff --git a/app/views/registrar/invoices/partials/_items.haml b/app/views/registrar/invoices/partials/_items.haml index 560b3e1f6..2d485f0ab 100644 --- a/app/views/registrar/invoices/partials/_items.haml +++ b/app/views/registrar/invoices/partials/_items.haml @@ -24,7 +24,7 @@ %td= currency(@invoice.subtotal) %tr %th.no-border{colspan: 3} - %th= t('vat', rate: number_to_percentage(@invoice.vat_rate, precision: 1)) + %th= t('vat', rate: vat_rate(@invoice.vat_rate)) %td= currency(@invoice.vat) %tr %th.no-border{colspan: 3} diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index e59ca618b..fcc68f128 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -242,7 +242,7 @@ %td= "#{currency(@invoice.subtotal)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} - %th= t('vat', rate: number_to_percentage(@invoice.vat_rate, precision: 1)) + %th= t('vat', rate: vat_rate(@invoice.vat_rate)) %td= "#{currency(@invoice.vat)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} diff --git a/test/integration/admin/registrars/show_registrar_test.rb b/test/integration/admin/registrars/show_registrar_test.rb index d4ed927bd..94252705e 100644 --- a/test/integration/admin/registrars/show_registrar_test.rb +++ b/test/integration/admin/registrars/show_registrar_test.rb @@ -22,6 +22,6 @@ class ShowRegistrarTest < ActionDispatch::IntegrationTest end def test_vat_rate - assert_text number_to_percentage(@registrar.vat_rate, precision: 1) + assert_text vat_rate(@registrar.vat_rate) end end From a2c919b0fd1cc993797d6eb92b243b9196e38873 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 11:45:04 +0200 Subject: [PATCH 50/87] Use standard translation #623 --- app/views/registrar/invoices/partials/_items.haml | 4 ++-- app/views/registrar/invoices/pdf.haml | 4 ++-- config/locales/en.yml | 2 -- 3 files changed, 4 insertions(+), 6 deletions(-) diff --git a/app/views/registrar/invoices/partials/_items.haml b/app/views/registrar/invoices/partials/_items.haml index 2d485f0ab..3bf4f9a84 100644 --- a/app/views/registrar/invoices/partials/_items.haml +++ b/app/views/registrar/invoices/partials/_items.haml @@ -20,11 +20,11 @@ %tfoot %tr %th{colspan: 3} - %th= t(:total_without_vat) + %th= Invoice.human_attribute_name :subtotal %td= currency(@invoice.subtotal) %tr %th.no-border{colspan: 3} - %th= t('vat', rate: vat_rate(@invoice.vat_rate)) + %th= "VAT #{vat_rate(@invoice.vat_rate)}" %td= currency(@invoice.vat) %tr %th.no-border{colspan: 3} diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index fcc68f128..9a2dd80f2 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -238,11 +238,11 @@ %tfoot %tr %th{colspan: 3} - %th= t(:total_without_vat) + %th= Invoice.human_attribute_name :subtotal %td= "#{currency(@invoice.subtotal)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} - %th= t('vat', rate: vat_rate(@invoice.vat_rate)) + %th= "VAT #{vat_rate(@invoice.vat_rate)}" %td= "#{currency(@invoice.vat)} #{@invoice.currency}" %tr %th.no-border{colspan: 3} diff --git a/config/locales/en.yml b/config/locales/en.yml index 73045367b..2ca4a207b 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -543,7 +543,6 @@ en: invoice_number: Invoice no. seller: 'Seller' prepayment: 'Prepayment' - vat: 'VAT (%{rate})' unpaid: 'Unpaid' your_current_account_balance_is: 'Your current account balance is %{balance} %{currency}' billing: 'Billing' @@ -561,7 +560,6 @@ en: unit: 'Unit' price: 'Price' total: 'Total' - total_without_vat: 'Total without VAT' paid_at: 'Paid at' invoice: 'Invoice' bank_statements: 'Bank statements' From ef6f4825288956d55e0633ff074338402822fa7d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 14:32:17 +0200 Subject: [PATCH 51/87] Format inline #623 --- app/helpers/application_helper.rb | 4 ---- app/views/admin/registrars/_billing.html.erb | 2 +- app/views/registrar/invoices/partials/_items.haml | 8 ++++---- app/views/registrar/invoices/pdf.haml | 8 ++++---- 4 files changed, 9 insertions(+), 13 deletions(-) diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index f0e42344b..8203a630f 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -89,8 +89,4 @@ module ApplicationHelper types.delete('ddoc') ".#{types.join(',.')}" end - - def vat_rate(value) - number_to_percentage(value, precision: 1) - end end diff --git a/app/views/admin/registrars/_billing.html.erb b/app/views/admin/registrars/_billing.html.erb index 929487afe..7a7e49fea 100644 --- a/app/views/admin/registrars/_billing.html.erb +++ b/app/views/admin/registrars/_billing.html.erb @@ -8,7 +8,7 @@
<%= registrar.vat_no %>
<%= Registrar.human_attribute_name :vat_rate %>
-
<%= vat_rate registrar.vat_rate %>
+
<%= number_to_percentage(registrar.vat_rate, precision: 1) %>
<%= Registrar.human_attribute_name :accounting_customer_code %>
<%= registrar.accounting_customer_code %>
diff --git a/app/views/registrar/invoices/partials/_items.haml b/app/views/registrar/invoices/partials/_items.haml index 3bf4f9a84..ebd94be53 100644 --- a/app/views/registrar/invoices/partials/_items.haml +++ b/app/views/registrar/invoices/partials/_items.haml @@ -21,12 +21,12 @@ %tr %th{colspan: 3} %th= Invoice.human_attribute_name :subtotal - %td= currency(@invoice.subtotal) + %td= number_to_currency @invoice.subtotal %tr %th.no-border{colspan: 3} - %th= "VAT #{vat_rate(@invoice.vat_rate)}" - %td= currency(@invoice.vat) + %th= "VAT #{number_to_percentage(@invoice.vat_rate, precision: 1)}" + %td= number_to_currency @invoice.vat_amount %tr %th.no-border{colspan: 3} %th= t(:total) - %td= currency(@invoice.total) + %td= number_to_currency @invoice.total diff --git a/app/views/registrar/invoices/pdf.haml b/app/views/registrar/invoices/pdf.haml index 9a2dd80f2..4f3b499df 100644 --- a/app/views/registrar/invoices/pdf.haml +++ b/app/views/registrar/invoices/pdf.haml @@ -239,15 +239,15 @@ %tr %th{colspan: 3} %th= Invoice.human_attribute_name :subtotal - %td= "#{currency(@invoice.subtotal)} #{@invoice.currency}" + %td= number_to_currency @invoice.subtotal %tr %th.no-border{colspan: 3} - %th= "VAT #{vat_rate(@invoice.vat_rate)}" - %td= "#{currency(@invoice.vat)} #{@invoice.currency}" + %th= "VAT #{number_to_percentage(@invoice.vat_rate, precision: 1)}" + %td= number_to_currency @invoice.vat_amount %tr %th.no-border{colspan: 3} %th= t(:total) - %td= "#{currency(@invoice.total)} #{@invoice.currency}" + %td= number_to_currency @invoice.total #footer %hr From 9a4380fd96c3e003c5ff3cba6dd32c12788a9813 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 14:32:38 +0200 Subject: [PATCH 52/87] Fix invoice total calculation #623 --- app/models/invoice.rb | 12 ++++++++---- test/models/invoice_test.rb | 4 ++-- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/app/models/invoice.rb b/app/models/invoice.rb index 19e25d5c9..a5c3f3f85 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -34,7 +34,7 @@ class Invoice < ActiveRecord::Base before_create :set_invoice_number before_create :apply_default_vat_rate, unless: :vat_rate? - before_create :save_total + before_create :calculate_total, unless: :total? attribute :vat_rate, ::Type::VATRate.new attr_readonly :vat_rate @@ -150,14 +150,18 @@ class Invoice < ActiveRecord::Base end def subtotal - (items.map(&:item_sum_without_vat).sum).round(2) + invoice_items.collect { |line_item| line_item.item_sum_without_vat }.reduce(:+) end - def vat + def vat_amount return 0 unless vat_rate subtotal * vat_rate / 100 end + def total + calculate_total unless total? + end + private def apply_default_vat_rate @@ -165,6 +169,6 @@ class Invoice < ActiveRecord::Base end def calculate_total - # (sum_without_vat + vat).round(2) + self.total = subtotal + vat_amount end end diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index fb7e66821..9222f83b8 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -59,12 +59,12 @@ class InvoiceTest < ActiveSupport::TestCase end def test_calculates_vat_amount - assert_equal BigDecimal('1.5'), @invoice.vat + assert_equal BigDecimal('1.5'), @invoice.vat_amount end def test_vat_amount_is_zero_when_vat_rate_is_blank @invoice.vat_rate = nil - assert_equal 0, @invoice.vat + assert_equal 0, @invoice.vat_amount end def test_calculates_subtotal From fb35ec5cbb6dbc2ea9848733923447165c591ec8 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 13 Mar 2018 14:49:36 +0200 Subject: [PATCH 53/87] Require `invoices.total` #623 --- .../20180313124751_change_invoices_total_to_not_null.rb | 5 +++++ db/structure.sql | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 db/migrate/20180313124751_change_invoices_total_to_not_null.rb diff --git a/db/migrate/20180313124751_change_invoices_total_to_not_null.rb b/db/migrate/20180313124751_change_invoices_total_to_not_null.rb new file mode 100644 index 000000000..fb31f4084 --- /dev/null +++ b/db/migrate/20180313124751_change_invoices_total_to_not_null.rb @@ -0,0 +1,5 @@ +class ChangeInvoicesTotalToNotNull < ActiveRecord::Migration + def change + change_column_null :invoices, :total, false + end +end diff --git a/db/structure.sql b/db/structure.sql index 6df03a91a..32a7b3891 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -1036,7 +1036,7 @@ CREATE TABLE invoices ( updator_str character varying, number integer, cancelled_at timestamp without time zone, - total numeric(10,2), + total numeric(10,2) NOT NULL, in_directo boolean DEFAULT false ); @@ -4716,3 +4716,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180310142630'); INSERT INTO schema_migrations (version) VALUES ('20180313090437'); +INSERT INTO schema_migrations (version) VALUES ('20180313124751'); + From e1b0f3044e875888158650d4e630573102070cee Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 14 Mar 2018 12:19:44 +0200 Subject: [PATCH 54/87] Fix invoice #623 --- app/models/invoice.rb | 1 + test/models/invoice_test.rb | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/app/models/invoice.rb b/app/models/invoice.rb index a5c3f3f85..edd85108d 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -160,6 +160,7 @@ class Invoice < ActiveRecord::Base def total calculate_total unless total? + read_attribute(:total) end private diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index 9222f83b8..c9e1f9f11 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -76,6 +76,10 @@ class InvoiceTest < ActiveSupport::TestCase end end + def test_returns_persisted_total + assert_equal BigDecimal('16.50'), @invoice.total + end + def test_calculates_total line_item = InvoiceItem.new invoice = Invoice.new From 094c06bee712c9bac4021c2dd732179df90b6f74 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 14 Mar 2018 12:58:49 +0200 Subject: [PATCH 55/87] Fix directo #623 --- app/models/directo.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/directo.rb b/app/models/directo.rb index 2b794b0e4..53fa9b36f 100644 --- a/app/models/directo.rb +++ b/app/models/directo.rb @@ -34,7 +34,7 @@ class Directo < ActiveRecord::Base xml.line( "ProductID" => Setting.directo_receipt_product_name, "Quantity" => 1, - "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.total/(1+invoice.vat_rate), precision: 2, separator: "."), + "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.subtotal, precision: 2, separator: "."), "ProductName" => invoice.order ) } From 1107dc67b87607ef3e56069a294c239433fe42c2 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 14 Mar 2018 14:30:20 +0200 Subject: [PATCH 56/87] Simplify method #623 --- app/models/invoice.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/invoice.rb b/app/models/invoice.rb index edd85108d..8381ccbac 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -150,7 +150,7 @@ class Invoice < ActiveRecord::Base end def subtotal - invoice_items.collect { |line_item| line_item.item_sum_without_vat }.reduce(:+) + invoice_items.map(&:item_sum_without_vat).reduce(:+) end def vat_amount From 169c67839e37a77e6a8dc584f31ab54e6bd46647 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 14 Mar 2018 14:48:20 +0200 Subject: [PATCH 57/87] Add "VATCode" and "TotalVAT" fields to Directo #623 --- app/models/directo.rb | 6 ++++-- app/models/invoice.rb | 5 +++++ .../20180314122722_add_invoices_buyer_vat_no.rb | 5 +++++ db/structure.sql | 5 ++++- test/models/invoice_test.rb | 16 ++++++++++++++++ 5 files changed, 34 insertions(+), 3 deletions(-) create mode 100644 db/migrate/20180314122722_add_invoices_buyer_vat_no.rb diff --git a/app/models/directo.rb b/app/models/directo.rb index 53fa9b36f..9352c9356 100644 --- a/app/models/directo.rb +++ b/app/models/directo.rb @@ -29,12 +29,14 @@ class Directo < ActiveRecord::Base "InvoiceDate" => invoice.created_at.strftime("%Y-%m-%dT%H:%M:%S"), "PaymentTerm" => Setting.directo_receipt_payment_term, "Currency" => invoice.currency, - "CustomerCode"=> invoice.buyer.accounting_customer_code + "CustomerCode"=> invoice.buyer.accounting_customer_code, + 'TotalVAT' => ActionController::Base.helpers.number_with_precision(invoice.vat_amount, precision: 2, separator: '.') ){ xml.line( "ProductID" => Setting.directo_receipt_product_name, "Quantity" => 1, - "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.subtotal, precision: 2, separator: "."), + "UnitPriceWoVAT" => ActionController::Base.helpers.number_with_precision(invoice.subtotal, precision: 2, separator: '.'), + 'VATCode' => invoice.buyer_vat_no, "ProductName" => invoice.order ) } diff --git a/app/models/invoice.rb b/app/models/invoice.rb index 8381ccbac..a2e469ca0 100644 --- a/app/models/invoice.rb +++ b/app/models/invoice.rb @@ -35,6 +35,7 @@ class Invoice < ActiveRecord::Base before_create :set_invoice_number before_create :apply_default_vat_rate, unless: :vat_rate? before_create :calculate_total, unless: :total? + before_create :apply_default_buyer_vat_no, unless: :buyer_vat_no? attribute :vat_rate, ::Type::VATRate.new attr_readonly :vat_rate @@ -169,6 +170,10 @@ class Invoice < ActiveRecord::Base self.vat_rate = buyer.effective_vat_rate end + def apply_default_buyer_vat_no + self.buyer_vat_no = buyer.vat_no + end + def calculate_total self.total = subtotal + vat_amount end diff --git a/db/migrate/20180314122722_add_invoices_buyer_vat_no.rb b/db/migrate/20180314122722_add_invoices_buyer_vat_no.rb new file mode 100644 index 000000000..83f9f9abd --- /dev/null +++ b/db/migrate/20180314122722_add_invoices_buyer_vat_no.rb @@ -0,0 +1,5 @@ +class AddInvoicesBuyerVatNo < ActiveRecord::Migration + def change + add_column :invoices, :buyer_vat_no, :string + end +end diff --git a/db/structure.sql b/db/structure.sql index 32a7b3891..96a9637fe 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -1037,7 +1037,8 @@ CREATE TABLE invoices ( number integer, cancelled_at timestamp without time zone, total numeric(10,2) NOT NULL, - in_directo boolean DEFAULT false + in_directo boolean DEFAULT false, + buyer_vat_no character varying ); @@ -4718,3 +4719,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180313090437'); INSERT INTO schema_migrations (version) VALUES ('20180313124751'); +INSERT INTO schema_migrations (version) VALUES ('20180314122722'); + diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index c9e1f9f11..5ae54abd0 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -90,4 +90,20 @@ class InvoiceTest < ActiveSupport::TestCase assert_equal BigDecimal('5.50'), invoice.total end end + + def test_valid_without_buyer_vat_no + @invoice.buyer_vat_no = '' + assert @invoice.valid? + end + + def test_buyer_vat_no_is_taken_from_registrar_by_default + registrar = registrars(:bestnames) + registrar.vat_no = 'US1234' + invoice = @invoice.dup + invoice.buyer_vat_no = nil + invoice.buyer = registrar + invoice.invoice_items = @invoice.invoice_items + invoice.save! + assert_equal 'US1234', invoice.buyer_vat_no + end end From f8088df79800a6947403eeb03413a22bd1c569b0 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 14 Mar 2018 23:26:54 +0200 Subject: [PATCH 58/87] Fix tests #623 --- test/models/registrar/vat_test.rb | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/test/models/registrar/vat_test.rb b/test/models/registrar/vat_test.rb index 802a0d987..123eed671 100644 --- a/test/models/registrar/vat_test.rb +++ b/test/models/registrar/vat_test.rb @@ -13,7 +13,16 @@ class RegistrarVATTest < ActiveSupport::TestCase assert @registrar.valid? end - def test_vat_is_not_applied_when_registrar_is_local_vat_payer + def test_apply_vat_rate_from_registry_when_registrar_is_local_vat_payer + Setting.registry_country_code = 'US' + @registrar.country_code = 'US' + + Registry.instance.stub(:vat_rate, BigDecimal('5.5')) do + assert_equal BigDecimal('5.5'), @registrar.effective_vat_rate + end + end + + def test_require_no_vat_rate_when_registrar_is_local_vat_payer @registrar.vat_rate = 1 assert @registrar.invalid? @@ -21,7 +30,14 @@ class RegistrarVATTest < ActiveSupport::TestCase assert @registrar.valid? end - def test_requires_vat_rate_when_registrar_is_foreign_vat_payer_and_vat_no_is_absent + def test_apply_vat_rate_from_registrar_when_registrar_is_foreign_vat_payer + Setting.registry_country_code = 'US' + @registrar.country_code = 'DE' + @registrar.vat_rate = BigDecimal('5.6') + assert_equal BigDecimal('5.6'), @registrar.effective_vat_rate + end + + def test_require_vat_rate_when_registrar_is_foreign_vat_payer_and_vat_no_is_absent @registrar.country_code = 'DE' @registrar.vat_no = '' @@ -30,7 +46,7 @@ class RegistrarVATTest < ActiveSupport::TestCase assert @registrar.errors.added?(:vat_rate, :blank) end - def test_vat_is_not_applied_when_registrar_is_foreign_vat_payer_and_vat_no_is_present + def test_require_no_vat_rate_when_registrar_is_foreign_vat_payer_and_vat_no_is_present @registrar.country_code = 'DE' @registrar.vat_no = 'valid' From 69d0a2011512db57238308d7c2906ef8358534eb Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 14 Mar 2018 23:53:53 +0200 Subject: [PATCH 59/87] Fix tests #623 --- test/fixtures/registrars.yml | 11 ++++++++ .../admin/registrars/details_test.rb | 18 +++++++++++++ .../admin/registrars/show_registrar_test.rb | 27 ------------------- test/models/registrar/vat_test.rb | 3 +++ 4 files changed, 32 insertions(+), 27 deletions(-) create mode 100644 test/integration/admin/registrars/details_test.rb delete mode 100644 test/integration/admin/registrars/show_registrar_test.rb diff --git a/test/fixtures/registrars.yml b/test/fixtures/registrars.yml index a0faee8b4..e54d79fad 100644 --- a/test/fixtures/registrars.yml +++ b/test/fixtures/registrars.yml @@ -30,3 +30,14 @@ not_in_use: country_code: US accounting_customer_code: any language: en + +complete: + name: Complete Names + reg_no: 123456 + code: completenames + email: completenames@example.com + country_code: US + accounting_customer_code: US0001 + language: en + vat_no: US12345 + vat_rate: 0.05 diff --git a/test/integration/admin/registrars/details_test.rb b/test/integration/admin/registrars/details_test.rb new file mode 100644 index 000000000..3ea7fa187 --- /dev/null +++ b/test/integration/admin/registrars/details_test.rb @@ -0,0 +1,18 @@ +require 'test_helper' + +class AdminAreaRegistrarDetailsTest < ActionDispatch::IntegrationTest + include ActionView::Helpers::NumberHelper + + def setup + login_as users(:admin) + @registrar = registrars(:complete) + end + + def test_registrar_details + visit admin_registrar_path(@registrar) + assert_text 'Accounting customer code US0001' + assert_text 'VAT number US12345' + assert_text 'VAT rate 5.0%' + assert_text 'Language English' + end +end diff --git a/test/integration/admin/registrars/show_registrar_test.rb b/test/integration/admin/registrars/show_registrar_test.rb deleted file mode 100644 index 94252705e..000000000 --- a/test/integration/admin/registrars/show_registrar_test.rb +++ /dev/null @@ -1,27 +0,0 @@ -require 'test_helper' - -class ShowRegistrarTest < ActionDispatch::IntegrationTest - include ActionView::Helpers::NumberHelper - - def setup - login_as users(:admin) - @registrar = registrars(:bestnames) - visit admin_registrar_path(@registrar) - end - - def test_accounting_customer_code - assert_text 'bestnames' - end - - def test_language - assert_text 'Language English' - end - - def test_vat_no - assert_text 'US12345' - end - - def test_vat_rate - assert_text vat_rate(@registrar.vat_rate) - end -end diff --git a/test/models/registrar/vat_test.rb b/test/models/registrar/vat_test.rb index 123eed671..6ba16f8e4 100644 --- a/test/models/registrar/vat_test.rb +++ b/test/models/registrar/vat_test.rb @@ -44,6 +44,9 @@ class RegistrarVATTest < ActiveSupport::TestCase @registrar.vat_rate = '' assert @registrar.invalid? assert @registrar.errors.added?(:vat_rate, :blank) + + @registrar.vat_rate = 5 + assert @registrar.valid? end def test_require_no_vat_rate_when_registrar_is_foreign_vat_payer_and_vat_no_is_present From 18449b7ef379ae62c078c72d4086048c8299de2d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 26 Mar 2018 07:21:54 +0300 Subject: [PATCH 60/87] Remove logging on activerecord validation #231 --- config/initializers/eis_custom_active_record.rb | 8 -------- 1 file changed, 8 deletions(-) delete mode 100644 config/initializers/eis_custom_active_record.rb diff --git a/config/initializers/eis_custom_active_record.rb b/config/initializers/eis_custom_active_record.rb deleted file mode 100644 index 60dcebeb9..000000000 --- a/config/initializers/eis_custom_active_record.rb +++ /dev/null @@ -1,8 +0,0 @@ -# Log all user issues raised by active record -# rubocop: disable Metrics/LineLength -class ActiveRecord::Base - after_validation do |m| - Rails.logger.info "USER MSG: ACTIVERECORD: #{m.class} ##{m.id} #{m.errors.full_messages} #{m.errors['epp_errors']}" if m.errors.present? - true - end -end From 2f86b3c76bc6103ba6a8297a8a8649eb5c7a86c4 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 27 Mar 2018 18:34:48 +0300 Subject: [PATCH 61/87] Remove Domain#valid_from As unused. `created_at` is the same value as `valid_from`. #787 --- app/models/domain.rb | 1 + app/models/epp/domain.rb | 1 - app/views/admin/domains/partials/_general.html.erb | 3 --- app/views/admin/domains/partials/_version.haml | 4 +--- app/views/registrant/domains/partials/_general.html.erb | 3 --- db/migrate/20180327151906_remove_domains_valid_from.rb | 5 +++++ db/structure.sql | 3 ++- doc/models_complete.svg | 1 - lib/tasks/dev.rake | 2 -- lib/tasks/import.rake | 1 - spec/models/epp/domain_spec.rb | 6 +----- 11 files changed, 10 insertions(+), 20 deletions(-) create mode 100644 db/migrate/20180327151906_remove_domains_valid_from.rb diff --git a/app/models/domain.rb b/app/models/domain.rb index 9eef0f30f..f8500a206 100644 --- a/app/models/domain.rb +++ b/app/models/domain.rb @@ -654,6 +654,7 @@ class Domain < ActiveRecord::Base def as_json(_options) hash = super hash['auth_info'] = hash.delete('transfer_code') # API v1 requirement + hash['valid_from'] = hash['created_at'] # API v1 requirement hash end diff --git a/app/models/epp/domain.rb b/app/models/epp/domain.rb index fb01fe38a..2ead51ea3 100644 --- a/app/models/epp/domain.rb +++ b/app/models/epp/domain.rb @@ -54,7 +54,6 @@ class Epp::Domain < Domain domain.attributes = domain.attrs_from(frame, current_user) domain.attach_default_contacts domain.registered_at = Time.zone.now - domain.valid_from = Time.zone.now period = domain.period.to_i plural_period_unit_name = (domain.period_unit == 'm' ? 'months' : 'years').to_sym diff --git a/app/views/admin/domains/partials/_general.html.erb b/app/views/admin/domains/partials/_general.html.erb index eeeef6eb5..7e09a7756 100644 --- a/app/views/admin/domains/partials/_general.html.erb +++ b/app/views/admin/domains/partials/_general.html.erb @@ -22,9 +22,6 @@ class: 'form-control input-sm' %> -
<%= t(:valid_from) %>
-
<%= l(@domain.valid_from) %>
-
<%= t(:valid_to) %>
<%= l(@domain.valid_to) %>
diff --git a/app/views/admin/domains/partials/_version.haml b/app/views/admin/domains/partials/_version.haml index b15bda0dd..da0a0ece1 100644 --- a/app/views/admin/domains/partials/_version.haml +++ b/app/views/admin/domains/partials/_version.haml @@ -66,12 +66,10 @@ %p = link_to t(:pending_epp), '#', class: 'js-pending' - %td{class: changing_css_class(version, "period", "period_unit", "valid_from", "valid_to")} + %td{class: changing_css_class(version, "period", "period_unit", "valid_to")} %p = "#{domain.period}#{domain.period_unit}" %br - = "#{l(domain.valid_from, format: :date)}" - %br = "#{l(domain.valid_to, format: :date)}" %td diff --git a/app/views/registrant/domains/partials/_general.html.erb b/app/views/registrant/domains/partials/_general.html.erb index 72ae8aad7..eed058437 100644 --- a/app/views/registrant/domains/partials/_general.html.erb +++ b/app/views/registrant/domains/partials/_general.html.erb @@ -22,9 +22,6 @@ class: 'form-control input-sm' %> -
<%= t(:valid_from) %>
-
<%= l(@domain.valid_from) %>
-
<%= t(:valid_to) %>
<%= l(@domain.valid_to) %>
diff --git a/db/migrate/20180327151906_remove_domains_valid_from.rb b/db/migrate/20180327151906_remove_domains_valid_from.rb new file mode 100644 index 000000000..3514fc9f2 --- /dev/null +++ b/db/migrate/20180327151906_remove_domains_valid_from.rb @@ -0,0 +1,5 @@ +class RemoveDomainsValidFrom < ActiveRecord::Migration + def change + remove_column :domains, :valid_from, :datetime + end +end diff --git a/db/structure.sql b/db/structure.sql index a9257ffa3..e5f1dad00 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -872,7 +872,6 @@ CREATE TABLE domains ( registrar_id integer NOT NULL, registered_at timestamp without time zone, status character varying, - valid_from timestamp without time zone, valid_to timestamp without time zone, registrant_id integer NOT NULL, transfer_code character varying NOT NULL, @@ -4679,3 +4678,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180306183549'); INSERT INTO schema_migrations (version) VALUES ('20180308123240'); +INSERT INTO schema_migrations (version) VALUES ('20180327151906'); + diff --git a/doc/models_complete.svg b/doc/models_complete.svg index 296d20c67..d014e0c8d 100644 --- a/doc/models_complete.svg +++ b/doc/models_complete.svg @@ -852,7 +852,6 @@ registrar_id :integer registered_at :datetime status :string -valid_from :datetime valid_to :datetime registrant_id :integer transfer_code :string diff --git a/lib/tasks/dev.rake b/lib/tasks/dev.rake index f4b133ed1..e4c120a8d 100644 --- a/lib/tasks/dev.rake +++ b/lib/tasks/dev.rake @@ -23,7 +23,6 @@ namespace :dev do period: period, period_unit: period_unit, registered_at: reg_time, - valid_from: reg_time, expire_time: reg_time + period.send(duration.second.to_sym), created_at: reg_time, updated_at: reg_time, @@ -151,7 +150,6 @@ namespace :dev do period: period, period_unit: 'y', registered_at: Time.zone.now, - valid_from: Time.zone.now, expire_time: Time.zone.now + period.years, registrar: registrar, registrant: registrants.sample) diff --git a/lib/tasks/import.rake b/lib/tasks/import.rake index f26ad252f..45f3cae99 100644 --- a/lib/tasks/import.rake +++ b/lib/tasks/import.rake @@ -337,7 +337,6 @@ namespace :import do name registrar_id registered_at - valid_from valid_to transfer_code created_at diff --git a/spec/models/epp/domain_spec.rb b/spec/models/epp/domain_spec.rb index de0b8f4e1..402305527 100644 --- a/spec/models/epp/domain_spec.rb +++ b/spec/models/epp/domain_spec.rb @@ -21,11 +21,7 @@ RSpec.describe Epp::Domain, db: false do expect(domain.registered_at).to eq(Time.zone.parse('05.07.2010')) end - it 'has :valid_from set to now' do - expect(domain.valid_from).to eq(Time.zone.parse('05.07.2010')) - end - - it 'has :valid_to set to the beginning of next day after :valid_from' do + it 'has :valid_to set to the beginning of next day after :registered_at' do expect(domain.valid_to).to eq(Time.zone.parse('06.07.2011 00:00')) end end From 77104582f79107af4c38f7c4fab95f4ac969d585 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sat, 31 Mar 2018 12:55:08 +0300 Subject: [PATCH 62/87] Fix attribute name --- app/models/domain.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/domain.rb b/app/models/domain.rb index f8500a206..d5c860913 100644 --- a/app/models/domain.rb +++ b/app/models/domain.rb @@ -654,7 +654,7 @@ class Domain < ActiveRecord::Base def as_json(_options) hash = super hash['auth_info'] = hash.delete('transfer_code') # API v1 requirement - hash['valid_from'] = hash['created_at'] # API v1 requirement + hash['valid_from'] = hash['registered_at'] # API v1 requirement hash end From c675512b22a566ba0b5d9ef7690c92b747fa5cde Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sat, 31 Mar 2018 23:31:40 +0300 Subject: [PATCH 63/87] Change `domains.valid_to` DB column to NOT NULL #800 --- app/models/whois_record.rb | 2 +- app/views/epp/domains/create.xml.builder | 2 +- app/views/epp/domains/info.xml.builder | 2 +- app/views/epp/domains/partials/_transfer.xml.builder | 2 +- app/views/epp/domains/renew.xml.builder | 2 +- .../20180331200125_change_domains_valid_to_to_not_null.rb | 5 +++++ db/structure.sql | 4 +++- spec/factories/domain.rb | 1 + 8 files changed, 14 insertions(+), 6 deletions(-) create mode 100644 db/migrate/20180331200125_change_domains_valid_to_to_not_null.rb diff --git a/app/models/whois_record.rb b/app/models/whois_record.rb index bd16e0c99..45cff4036 100644 --- a/app/models/whois_record.rb +++ b/app/models/whois_record.rb @@ -48,7 +48,7 @@ class WhoisRecord < ActiveRecord::Base h[:status] = domain.statuses.map { |x| status_map[x] || x } h[:registered] = domain.registered_at.try(:to_s, :iso8601) h[:changed] = domain.updated_at.try(:to_s, :iso8601) - h[:expire] = domain.valid_to.try(:to_date).try(:to_s) + h[:expire] = domain.valid_to.to_date.to_s h[:outzone] = domain.outzone_at.try(:to_date).try(:to_s) h[:delete] = [domain.delete_at, domain.force_delete_at].compact.min.try(:to_date).try(:to_s) diff --git a/app/views/epp/domains/create.xml.builder b/app/views/epp/domains/create.xml.builder index 213a2aa8f..2293f5657 100644 --- a/app/views/epp/domains/create.xml.builder +++ b/app/views/epp/domains/create.xml.builder @@ -8,7 +8,7 @@ xml.epp_head do xml.tag!('domain:creData', 'xmlns:domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd') do xml.tag!('domain:name', @domain.name) xml.tag!('domain:crDate', @domain.created_at.try(:iso8601)) - xml.tag!('domain:exDate', @domain.valid_to.try(:iso8601)) + xml.tag!('domain:exDate', @domain.valid_to.iso8601) end end diff --git a/app/views/epp/domains/info.xml.builder b/app/views/epp/domains/info.xml.builder index 8bf169acd..2d10f8baf 100644 --- a/app/views/epp/domains/info.xml.builder +++ b/app/views/epp/domains/info.xml.builder @@ -48,7 +48,7 @@ xml.epp_head do xml.tag!('domain:upDate', @domain.updated_at.try(:iso8601)) end - xml.tag!('domain:exDate', @domain.valid_to.try(:iso8601)) + xml.tag!('domain:exDate', @domain.valid_to.iso8601) # TODO Make domain transferrable #xml.tag!('domain:trDate', @domain.transferred_at) if @domain.transferred_at diff --git a/app/views/epp/domains/partials/_transfer.xml.builder b/app/views/epp/domains/partials/_transfer.xml.builder index 151af28b3..bfcc7db94 100644 --- a/app/views/epp/domains/partials/_transfer.xml.builder +++ b/app/views/epp/domains/partials/_transfer.xml.builder @@ -5,5 +5,5 @@ builder.tag!('domain:trnData', 'xmlns:domain' => 'https://epp.tld.ee/schema/doma builder.tag!('domain:reDate', dt.transfer_requested_at.try(:iso8601)) builder.tag!('domain:acID', dt.old_registrar.code) builder.tag!('domain:acDate', dt.transferred_at.try(:iso8601) || dt.wait_until.try(:iso8601)) - builder.tag!('domain:exDate', dt.domain_valid_to.try(:iso8601)) + builder.tag!('domain:exDate', dt.domain_valid_to.iso8601) end diff --git a/app/views/epp/domains/renew.xml.builder b/app/views/epp/domains/renew.xml.builder index 5d03c7128..e407ff0e7 100644 --- a/app/views/epp/domains/renew.xml.builder +++ b/app/views/epp/domains/renew.xml.builder @@ -7,7 +7,7 @@ xml.epp_head do xml.resData do xml.tag!('domain:renData', 'xmlns:domain' => 'https://epp.tld.ee/schema/domain-eis-1.0.xsd') do xml.tag!('domain:name', @domain[:name]) - xml.tag!('domain:exDate', @domain.valid_to.try(:iso8601)) + xml.tag!('domain:exDate', @domain.valid_to.iso8601) end end diff --git a/db/migrate/20180331200125_change_domains_valid_to_to_not_null.rb b/db/migrate/20180331200125_change_domains_valid_to_to_not_null.rb new file mode 100644 index 000000000..049b08806 --- /dev/null +++ b/db/migrate/20180331200125_change_domains_valid_to_to_not_null.rb @@ -0,0 +1,5 @@ +class ChangeDomainsValidToToNotNull < ActiveRecord::Migration + def change + change_column_null :domains, :valid_to, false + end +end diff --git a/db/structure.sql b/db/structure.sql index 374b17b5b..2e662f6f5 100644 --- a/db/structure.sql +++ b/db/structure.sql @@ -873,7 +873,7 @@ CREATE TABLE domains ( registered_at timestamp without time zone, status character varying, valid_from timestamp without time zone, - valid_to timestamp without time zone, + valid_to timestamp without time zone NOT NULL, registrant_id integer NOT NULL, transfer_code character varying NOT NULL, created_at timestamp without time zone, @@ -4695,3 +4695,5 @@ INSERT INTO schema_migrations (version) VALUES ('20180309053921'); INSERT INTO schema_migrations (version) VALUES ('20180309054510'); +INSERT INTO schema_migrations (version) VALUES ('20180331200125'); + diff --git a/spec/factories/domain.rb b/spec/factories/domain.rb index fc4d3a97b..8abac339c 100644 --- a/spec/factories/domain.rb +++ b/spec/factories/domain.rb @@ -3,6 +3,7 @@ FactoryBot.define do sequence(:name) { |n| "test#{n}.com" } period 1 period_unit 'y' # Year + valid_to Time.zone.parse('2010-07-05') registrar registrant From b694f0c2f643d2d61d65927b5b1b72c1e1790129 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Sat, 31 Mar 2018 12:00:19 +0300 Subject: [PATCH 64/87] Remove unused methods --- app/models/domain.rb | 11 ----------- app/models/epp/domain.rb | 10 ---------- app/models/registrar.rb | 5 ----- app/models/whois_record.rb | 14 -------------- 4 files changed, 40 deletions(-) diff --git a/app/models/domain.rb b/app/models/domain.rb index 9eef0f30f..bb671dc2d 100644 --- a/app/models/domain.rb +++ b/app/models/domain.rb @@ -220,17 +220,6 @@ class Domain < ActiveRecord::Base end class << self - def included - includes( - :registrant, - :registrar, - :nameservers, - :whois_record, - { tech_contacts: :registrar }, - { admin_contacts: :registrar } - ) - end - def nameserver_required? Setting.nameserver_required end diff --git a/app/models/epp/domain.rb b/app/models/epp/domain.rb index fb01fe38a..43ad15bc7 100644 --- a/app/models/epp/domain.rb +++ b/app/models/epp/domain.rb @@ -38,16 +38,6 @@ class Epp::Domain < Domain ok end - before_save :link_contacts - def link_contacts - #TODO: cleanup cache if we think to cache dynamic statuses - end - - after_destroy :unlink_contacts - def unlink_contacts - #TODO: cleanup cache if we think to cache dynamic statuses - end - class << self def new_from_epp(frame, current_user) domain = Epp::Domain.new diff --git a/app/models/registrar.rb b/app/models/registrar.rb index 5fd0008b5..525415cbe 100644 --- a/app/models/registrar.rb +++ b/app/models/registrar.rb @@ -105,11 +105,6 @@ class Registrar < ActiveRecord::Base cash_account.account_activities.create!(args) end - def credit!(args) - args[:currency] = 'EUR' - cash_account.account_activities.create!(args) - end - def address [street, city, state, zip].reject(&:blank?).compact.join(', ') end diff --git a/app/models/whois_record.rb b/app/models/whois_record.rb index bd16e0c99..e0ea4253d 100644 --- a/app/models/whois_record.rb +++ b/app/models/whois_record.rb @@ -9,20 +9,6 @@ class WhoisRecord < ActiveRecord::Base after_save :update_whois_server after_destroy :destroy_whois_record - class << self - def included - includes( - domain: [ - :registrant, - :registrar, - :nameservers, - { tech_contacts: :registrar }, - { admin_contacts: :registrar } - ] - ) - end - end - def self.find_by_name(name) WhoisRecord.where("lower(name) = ?", name.downcase) end From d57c737a80fd2c9df2b9920363944b1a7898843d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Mon, 2 Apr 2018 09:13:53 +0300 Subject: [PATCH 65/87] Convert specs to tests #792 --- app/models/concerns/domain/deletable.rb | 5 ++ spec/factories/domain.rb | 4 -- spec/models/concerns/domain/deletable_spec.rb | 19 -------- .../epp/domain/delete/discarded_spec.rb | 47 ------------------- .../epp/domain/update/discarded_spec.rb | 42 ----------------- .../epp/domain/domain_delete_test.rb | 27 +++++++++++ .../epp/domain/domain_update_test.rb | 23 +++++++++ test/models/domain/deletable_test.rb | 14 ++++++ 8 files changed, 69 insertions(+), 112 deletions(-) delete mode 100644 spec/models/concerns/domain/deletable_spec.rb delete mode 100644 spec/requests/epp/domain/delete/discarded_spec.rb delete mode 100644 spec/requests/epp/domain/update/discarded_spec.rb create mode 100644 test/integration/epp/domain/domain_update_test.rb create mode 100644 test/models/domain/deletable_test.rb diff --git a/app/models/concerns/domain/deletable.rb b/app/models/concerns/domain/deletable.rb index f724162e5..a0d49decb 100644 --- a/app/models/concerns/domain/deletable.rb +++ b/app/models/concerns/domain/deletable.rb @@ -5,6 +5,11 @@ module Concerns::Domain::Deletable alias_attribute :delete_time, :delete_at end + def discard + self.statuses << DomainStatus::DELETE_CANDIDATE + save + end + def discarded? statuses.include?(DomainStatus::DELETE_CANDIDATE) end diff --git a/spec/factories/domain.rb b/spec/factories/domain.rb index fc4d3a97b..4e15a8713 100644 --- a/spec/factories/domain.rb +++ b/spec/factories/domain.rb @@ -10,9 +10,5 @@ FactoryBot.define do domain.admin_domain_contacts << FactoryBot.build(:admin_domain_contact) domain.tech_domain_contacts << FactoryBot.build(:tech_domain_contact) end - - factory :domain_discarded do - statuses [DomainStatus::DELETE_CANDIDATE] - end end end diff --git a/spec/models/concerns/domain/deletable_spec.rb b/spec/models/concerns/domain/deletable_spec.rb deleted file mode 100644 index 826299b69..000000000 --- a/spec/models/concerns/domain/deletable_spec.rb +++ /dev/null @@ -1,19 +0,0 @@ -require 'rails_helper' - -RSpec.describe Domain, db: false do - it { is_expected.to alias_attribute(:delete_time, :delete_at) } - - describe '#discarded?' do - context 'when :deleteCandidate status is present' do - let(:domain) { described_class.new(statuses: [DomainStatus::DELETE_CANDIDATE]) } - - specify { expect(domain).to be_discarded } - end - - context 'when :deleteCandidate status is absent' do - let(:domain) { described_class.new(statuses: []) } - - specify { expect(domain).to_not be_discarded } - end - end -end diff --git a/spec/requests/epp/domain/delete/discarded_spec.rb b/spec/requests/epp/domain/delete/discarded_spec.rb deleted file mode 100644 index 55e74d965..000000000 --- a/spec/requests/epp/domain/delete/discarded_spec.rb +++ /dev/null @@ -1,47 +0,0 @@ -require 'rails_helper' - -RSpec.describe 'EPP domain:delete' do - let(:registrar) { create(:registrar) } - let(:user) { create(:api_user_epp, registrar: registrar) } - let(:session_id) { create(:epp_session, user: user).session_id } - let(:request_xml) { <<-XML - - - - - - test.com - - - - - dGVzdCBmYWlsCg== - - - - - XML - } - - before :example do - login_as user - end - - context 'when domain is not discarded' do - let!(:domain) { create(:domain, name: 'test.com') } - - it 'returns epp code of 1001' do - post '/epp/command/delete', { frame: request_xml }, 'HTTP_COOKIE' => "session=#{session_id}" - expect(response).to have_code_of(1001) - end - end - - context 'when domain is discarded' do - let!(:domain) { create(:domain_discarded, name: 'test.com') } - - it 'returns epp code of 2105' do - post '/epp/command/delete', { frame: request_xml }, 'HTTP_COOKIE' => "session=#{session_id}" - expect(response).to have_code_of(2105) - end - end -end diff --git a/spec/requests/epp/domain/update/discarded_spec.rb b/spec/requests/epp/domain/update/discarded_spec.rb deleted file mode 100644 index 4a31b7d10..000000000 --- a/spec/requests/epp/domain/update/discarded_spec.rb +++ /dev/null @@ -1,42 +0,0 @@ -require 'rails_helper' - -RSpec.describe 'EPP domain:update' do - let(:registrar) { create(:registrar) } - let(:user) { create(:api_user_epp, registrar: registrar) } - let(:session_id) { create(:epp_session, user: user).session_id } - let(:request_xml) { <<-XML - - - - - - test.com - - - - - XML - } - - before :example do - login_as user - end - - context 'when domain is not discarded' do - let!(:domain) { create(:domain, name: 'test.com') } - - it 'returns epp code of 1000' do - post '/epp/command/update', { frame: request_xml }, 'HTTP_COOKIE' => "session=#{session_id}" - expect(response).to have_code_of(1000) - end - end - - context 'when domain is discarded' do - let!(:domain) { create(:domain_discarded, name: 'test.com') } - - it 'returns epp code of 2105' do - post '/epp/command/update', { frame: request_xml }, 'HTTP_COOKIE' => "session=#{session_id}" - expect(response).to have_code_of(2105) - end - end -end diff --git a/test/integration/epp/domain/domain_delete_test.rb b/test/integration/epp/domain/domain_delete_test.rb index bdd326a3e..eae4d39ff 100644 --- a/test/integration/epp/domain/domain_delete_test.rb +++ b/test/integration/epp/domain/domain_delete_test.rb @@ -25,4 +25,31 @@ class EppDomainDeleteTest < ActionDispatch::IntegrationTest assert_equal '1001', Nokogiri::XML(response.body).at_css('result')[:code] assert_equal 1, Nokogiri::XML(response.body).css('result').size end + + def test_discarded_domain_cannot_be_deleted + domains(:shop).discard + + request_xml = <<-XML + + + + + + shop.test + + + + + dGVzdCBmYWlsCg== + + + + + XML + + assert_no_difference 'Domain.count' do + post '/epp/command/delete', { frame: request_xml }, 'HTTP_COOKIE' => 'session=api_bestnames' + end + assert_equal '2105', Nokogiri::XML(response.body).at_css('result')[:code] + end end diff --git a/test/integration/epp/domain/domain_update_test.rb b/test/integration/epp/domain/domain_update_test.rb new file mode 100644 index 000000000..9519b37bc --- /dev/null +++ b/test/integration/epp/domain/domain_update_test.rb @@ -0,0 +1,23 @@ +require 'test_helper' + +class EppDomainUpdateTest < ActionDispatch::IntegrationTest + def test_discarded_domain_cannot_be_updated + domains(:shop).discard + + request_xml = <<-XML + + + + + + shop.test + + + + + XML + + post '/epp/command/update', { frame: request_xml }, 'HTTP_COOKIE' => 'session=api_bestnames' + assert_equal '2105', Nokogiri::XML(response.body).at_css('result')[:code] + end +end diff --git a/test/models/domain/deletable_test.rb b/test/models/domain/deletable_test.rb new file mode 100644 index 000000000..499ff20b6 --- /dev/null +++ b/test/models/domain/deletable_test.rb @@ -0,0 +1,14 @@ +require 'test_helper' + +class DomainDeletableTest < ActiveSupport::TestCase + def setup + @domain = domains(:shop) + end + + def test_discard + refute @domain.discarded? + @domain.discard + @domain.reload + assert @domain.discarded? + end +end From 593c546c3048b6e038a13fe054486e032993e35a Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 00:06:35 +0300 Subject: [PATCH 66/87] Convert HAML to ERB --- app/views/admin/domains/show.haml | 24 ----------- app/views/admin/domains/show.html.erb | 58 +++++++++++++++++++++++++++ 2 files changed, 58 insertions(+), 24 deletions(-) delete mode 100644 app/views/admin/domains/show.haml create mode 100644 app/views/admin/domains/show.html.erb diff --git a/app/views/admin/domains/show.haml b/app/views/admin/domains/show.haml deleted file mode 100644 index 17a85b841..000000000 --- a/app/views/admin/domains/show.haml +++ /dev/null @@ -1,24 +0,0 @@ -- content_for :actions do - = link_to(t(:edit_statuses), edit_admin_domain_path(@domain), class: 'btn btn-primary') - = link_to(t(:history), admin_domain_domain_versions_path(@domain.id), method: :get, class: 'btn btn-primary') - -= render 'shared/title', name: @domain.name - -.row - .col-md-6= render 'admin/domains/partials/general' - .col-md-6= render 'admin/domains/partials/owner' -.row - .col-md-12= render 'admin/domains/partials/tech_contacts' -.row - .col-md-12= render 'admin/domains/partials/admin_contacts' -.row - .col-md-12= render 'admin/domains/partials/statuses' -.row - .col-md-12= render 'admin/domains/partials/nameservers' -.row - .col-md-12= render 'admin/domains/partials/dnskeys' -.row - .col-md-12= render 'admin/domains/partials/keyrelays' -.row - .col-md-12 - = render 'admin/domains/partials/legal_documents', legal_documents: @domain.legal_documents diff --git a/app/views/admin/domains/show.html.erb b/app/views/admin/domains/show.html.erb new file mode 100644 index 000000000..61e20e1ca --- /dev/null +++ b/app/views/admin/domains/show.html.erb @@ -0,0 +1,58 @@ +<% content_for :actions do %> + <%= link_to(t(:edit_statuses), edit_admin_domain_path(@domain), class: 'btn btn-primary') %> + <%= link_to(t(:history), admin_domain_domain_versions_path(@domain.id), method: :get, + class: 'btn btn-primary') %> +<% end %> +<%= render 'shared/title', name: @domain.name %> + +
+
+ <%= render 'admin/domains/partials/general' %> +
+
+ <%= render 'admin/domains/partials/owner' %> +
+
+ +
+
+ <%= render 'admin/domains/partials/tech_contacts' %> +
+
+ +
+
+ <%= render 'admin/domains/partials/admin_contacts' %> +
+
+ +
+
+ <%= render 'admin/domains/partials/statuses' %> +
+
+ +
+
+ <%= render 'admin/domains/partials/nameservers' %> +
+
+ +
+
+ <%= render 'admin/domains/partials/dnskeys' %> +
+
+ +
+
+ <%= render 'admin/domains/partials/keyrelays' %> +
+
+ +
+
+ <%= render 'admin/domains/partials/legal_documents', legal_documents: + @domain.legal_documents %> +
+
From 74b681db077a6e8c96317fae254a83c5393b65e0 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 00:12:17 +0300 Subject: [PATCH 67/87] Improve UI #792 --- app/views/admin/domains/show.html.erb | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/app/views/admin/domains/show.html.erb b/app/views/admin/domains/show.html.erb index 61e20e1ca..fb8d44046 100644 --- a/app/views/admin/domains/show.html.erb +++ b/app/views/admin/domains/show.html.erb @@ -1,9 +1,20 @@ -<% content_for :actions do %> - <%= link_to(t(:edit_statuses), edit_admin_domain_path(@domain), class: 'btn btn-primary') %> - <%= link_to(t(:history), admin_domain_domain_versions_path(@domain.id), method: :get, - class: 'btn btn-primary') %> -<% end %> -<%= render 'shared/title', name: @domain.name %> +
    +
  1. <%= link_to t('admin.domains.index.header'), admin_domains_path %>
    2. +
+ +
+
+
+

<%= @domain %>

+
+ +
+ <%= link_to(t(:edit_statuses), edit_admin_domain_path(@domain), class: 'btn btn-primary') %> + <%= link_to(t(:history), admin_domain_domain_versions_path(@domain.id), method: :get, + class: 'btn btn-primary') %> +
+
+
From f35f2045949421e1f99233fdefe7d2058decccdd Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 00:15:00 +0300 Subject: [PATCH 68/87] Extract translation #792 --- app/views/admin/domains/show.html.erb | 4 ++-- config/locales/admin/domains.en.yml | 4 ++++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/app/views/admin/domains/show.html.erb b/app/views/admin/domains/show.html.erb index fb8d44046..63be7256d 100644 --- a/app/views/admin/domains/show.html.erb +++ b/app/views/admin/domains/show.html.erb @@ -9,8 +9,8 @@
- <%= link_to(t(:edit_statuses), edit_admin_domain_path(@domain), class: 'btn btn-primary') %> - <%= link_to(t(:history), admin_domain_domain_versions_path(@domain.id), method: :get, + <%= link_to(t('.edit_btn'), edit_admin_domain_path(@domain), class: 'btn btn-primary') %> + <%= link_to(t('.history_btn'), admin_domain_domain_versions_path(@domain.id), method: :get, class: 'btn btn-primary') %>
diff --git a/config/locales/admin/domains.en.yml b/config/locales/admin/domains.en.yml index 200657d01..87c2c9c6a 100644 --- a/config/locales/admin/domains.en.yml +++ b/config/locales/admin/domains.en.yml @@ -5,6 +5,10 @@ en: header: Domains registrant: Registrant + show: + edit_btn: Edit statuses + history_btn: History + search_form: reset_btn: Reset From 639021d2620a40184905182e8199a887093fa269 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 00:15:29 +0300 Subject: [PATCH 69/87] Reformat #792 --- app/views/admin/domains/show.html.erb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/views/admin/domains/show.html.erb b/app/views/admin/domains/show.html.erb index 63be7256d..7d81a696f 100644 --- a/app/views/admin/domains/show.html.erb +++ b/app/views/admin/domains/show.html.erb @@ -9,9 +9,9 @@
- <%= link_to(t('.edit_btn'), edit_admin_domain_path(@domain), class: 'btn btn-primary') %> - <%= link_to(t('.history_btn'), admin_domain_domain_versions_path(@domain.id), method: :get, - class: 'btn btn-primary') %> + <%= link_to t('.edit_btn'), edit_admin_domain_path(@domain), class: 'btn btn-primary' %> + <%= link_to t('.history_btn'), admin_domain_domain_versions_path(@domain.id), method: :get, + class: 'btn btn-primary' %>
From 4435183a0ee1b55525012f082401aba3fa575595 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 00:17:27 +0300 Subject: [PATCH 70/87] Clean up #792 --- app/views/admin/domains/show.html.erb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/views/admin/domains/show.html.erb b/app/views/admin/domains/show.html.erb index 7d81a696f..9bfb4b9b5 100644 --- a/app/views/admin/domains/show.html.erb +++ b/app/views/admin/domains/show.html.erb @@ -10,7 +10,7 @@
<%= link_to t('.edit_btn'), edit_admin_domain_path(@domain), class: 'btn btn-primary' %> - <%= link_to t('.history_btn'), admin_domain_domain_versions_path(@domain.id), method: :get, + <%= link_to t('.history_btn'), admin_domain_domain_versions_path(@domain), class: 'btn btn-primary' %>
From 1dcf7bcd62eed09c5426af7fa8f788b0c8a5dca1 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 00:30:49 +0300 Subject: [PATCH 71/87] Remove unused partial #792 --- app/views/admin/domains/zonefile.haml | 8 -------- config/locales/en.yml | 1 - 2 files changed, 9 deletions(-) delete mode 100644 app/views/admin/domains/zonefile.haml diff --git a/app/views/admin/domains/zonefile.haml b/app/views/admin/domains/zonefile.haml deleted file mode 100644 index 72473ef53..000000000 --- a/app/views/admin/domains/zonefile.haml +++ /dev/null @@ -1,8 +0,0 @@ -- content_for :actions do - = link_to(t(:back_to_domain), admin_domain_path(@domain), class: 'btn btn-default') -= render 'shared/title', name: t(:zonefile) - -.row - .col-md-12 - = preserve do - %pre= @zonefile diff --git a/config/locales/en.yml b/config/locales/en.yml index 1e65f7e29..64b8c39c2 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -337,7 +337,6 @@ en: transfer_requested: 'Transfer requested.' message_was_not_found: 'Message was not found' - zonefile: 'Zonefile' only_one_parameter_allowed: 'Only one parameter allowed: %{param_1} or %{param_2}' exactly_one_parameter_required: 'Exactly one parameter required: %{params}' ds_data_with_key_allowed: 'Allow DS data with key' From f2697963827043e0cd7d3659d111144ec7b693d3 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 15:12:43 +0300 Subject: [PATCH 72/87] Show label in UI if a domain is discarded #792 --- app/models/concerns/domain/deletable.rb | 2 +- app/presenters/domain_presenter.rb | 11 +++++++++++ app/views/admin/domains/show.html.erb | 4 +++- test/integration/admin/domains/details_test.rb | 16 ++++++++++++++++ 4 files changed, 31 insertions(+), 2 deletions(-) create mode 100644 test/integration/admin/domains/details_test.rb diff --git a/app/models/concerns/domain/deletable.rb b/app/models/concerns/domain/deletable.rb index a0d49decb..86c296d88 100644 --- a/app/models/concerns/domain/deletable.rb +++ b/app/models/concerns/domain/deletable.rb @@ -6,7 +6,7 @@ module Concerns::Domain::Deletable end def discard - self.statuses << DomainStatus::DELETE_CANDIDATE + statuses << DomainStatus::DELETE_CANDIDATE save end diff --git a/app/presenters/domain_presenter.rb b/app/presenters/domain_presenter.rb index 9597330b8..aac3c1527 100644 --- a/app/presenters/domain_presenter.rb +++ b/app/presenters/domain_presenter.rb @@ -6,6 +6,17 @@ class DomainPresenter @view = view end + def name_with_status + html = domain.name + + if domain.discarded? + label = view.content_tag(:span, 'deleteCandidate', class: 'label label-warning') + html += " #{label}" + end + + html.html_safe + end + def expire_time view.l(domain.expire_time) end diff --git a/app/views/admin/domains/show.html.erb b/app/views/admin/domains/show.html.erb index 9bfb4b9b5..1501b35bb 100644 --- a/app/views/admin/domains/show.html.erb +++ b/app/views/admin/domains/show.html.erb @@ -1,3 +1,5 @@ +<% domain = DomainPresenter.new(domain: @domain, view: self) %> +
  1. <%= link_to t('admin.domains.index.header'), admin_domains_path %>
@@ -5,7 +7,7 @@
-

<%= @domain %>

+

<%= domain.name_with_status %>

diff --git a/test/integration/admin/domains/details_test.rb b/test/integration/admin/domains/details_test.rb new file mode 100644 index 000000000..afb62b6ea --- /dev/null +++ b/test/integration/admin/domains/details_test.rb @@ -0,0 +1,16 @@ +require 'test_helper' + +class AdminAreaDomainDetailsTest < ActionDispatch::IntegrationTest + def setup + login_as users(:admin) + @domain = domains(:shop) + end + + def test_discarded_domain_has_corresponding_label + visit admin_domain_url(@domain) + assert_no_css 'span.label.label-warning', text: 'deleteCandidate' + @domain.discard + visit admin_domain_url(@domain) + assert_css 'span.label.label-warning', text: 'deleteCandidate' + end +end From af414c9267b602ee20acb4136ebf34dcbc2ff265 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Tue, 3 Apr 2018 18:01:56 +0300 Subject: [PATCH 73/87] Reorganize tests #792 --- .../epp/domain/domain_update_test.rb | 25 +++++++++++++++++ .../epp/domain/update/transfer_code_test.rb | 28 ------------------- 2 files changed, 25 insertions(+), 28 deletions(-) delete mode 100644 test/integration/epp/domain/update/transfer_code_test.rb diff --git a/test/integration/epp/domain/domain_update_test.rb b/test/integration/epp/domain/domain_update_test.rb index 9519b37bc..ac7160558 100644 --- a/test/integration/epp/domain/domain_update_test.rb +++ b/test/integration/epp/domain/domain_update_test.rb @@ -1,6 +1,31 @@ require 'test_helper' class EppDomainUpdateTest < ActionDispatch::IntegrationTest + def test_update_domain + request_xml = <<-XML + + + + + + shop.test + + + f0ff7d17b0 + + + + + + + XML + + post '/epp/command/update', { frame: request_xml }, 'HTTP_COOKIE' => 'session=api_bestnames' + assert_equal 'f0ff7d17b0', domains(:shop).transfer_code + assert_equal '1000', Nokogiri::XML(response.body).at_css('result')[:code] + assert_equal 1, Nokogiri::XML(response.body).css('result').size + end + def test_discarded_domain_cannot_be_updated domains(:shop).discard diff --git a/test/integration/epp/domain/update/transfer_code_test.rb b/test/integration/epp/domain/update/transfer_code_test.rb deleted file mode 100644 index 92ee58a02..000000000 --- a/test/integration/epp/domain/update/transfer_code_test.rb +++ /dev/null @@ -1,28 +0,0 @@ -require 'test_helper' - -class EppDomainUpdateTest < ActionDispatch::IntegrationTest - def test_overwrites_existing - request_xml = <<-XML - - - - - - shop.test - - - f0ff7d17b0 - - - - - - - XML - - post '/epp/command/update', { frame: request_xml }, { 'HTTP_COOKIE' => 'session=api_bestnames' } - assert_equal 'f0ff7d17b0', domains(:shop).transfer_code - assert_equal '1000', Nokogiri::XML(response.body).at_css('result')[:code] - assert_equal 1, Nokogiri::XML(response.body).css('result').size - end -end From b1e037118832b4359477c94941d5aa971a73226b Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 4 Apr 2018 01:41:40 +0300 Subject: [PATCH 74/87] Run tests in random order --- config/environments/test.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/config/environments/test.rb b/config/environments/test.rb index df9ab2bbf..9b22ad426 100644 --- a/config/environments/test.rb +++ b/config/environments/test.rb @@ -43,6 +43,7 @@ Rails.application.configure do config.log_level = :debug config.active_job.queue_adapter = :test config.logger = ActiveSupport::Logger.new(nil) + config.active_support.test_order = :random # :random is the default in Rails 5 end # In this mode, any jobs you queue will be run in the same thread, synchronously From f8eda810f99c49b34b5aed3332b795761baa8607 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 4 Apr 2018 01:42:59 +0300 Subject: [PATCH 75/87] Use setup/teardown blocks instead of methods --- test/integration/admin/domains/force_delete_test.rb | 2 +- test/integration/admin/domains_test.rb | 2 +- test/integration/admin/registrars/delete_test.rb | 2 +- test/integration/admin/registrars/edit_test.rb | 2 +- test/integration/admin/registrars/new_test.rb | 2 +- test/integration/admin/registrars/show_registrar_test.rb | 2 +- test/integration/api/domain_transfers_test.rb | 2 +- test/integration/epp/domain/create/transfer_code_test.rb | 2 +- test/integration/epp/domain/domain_renew_test.rb | 2 +- test/integration/epp/domain/transfer/request_test.rb | 2 +- test/integration/epp/login/session_limit_test.rb | 2 +- test/integration/registrant/domains_test.rb | 2 +- test/integration/registrar/domain_transfers_test.rb | 2 +- test/integration/registrar/nameserver_replacement_test.rb | 2 +- test/models/contact/contact_test.rb | 2 +- test/models/contact/identical_test.rb | 2 +- test/models/contact/postal_address_test.rb | 2 +- test/models/contact/transfer_test.rb | 2 +- test/models/domain/domain_test.rb | 2 +- test/models/domain/transferable_test.rb | 2 +- test/models/domain_transfer_test.rb | 2 +- test/models/epp_session_test.rb | 2 +- test/models/message_test.rb | 2 +- test/models/nameserver/glue_record_test.rb | 2 +- test/models/nameserver_test.rb | 2 +- test/models/registrar/code_test.rb | 2 +- test/models/registrar/delete_test.rb | 2 +- test/models/registrar_test.rb | 2 +- test/test_helper.rb | 2 +- 29 files changed, 29 insertions(+), 29 deletions(-) diff --git a/test/integration/admin/domains/force_delete_test.rb b/test/integration/admin/domains/force_delete_test.rb index 731de151b..0ef017c11 100644 --- a/test/integration/admin/domains/force_delete_test.rb +++ b/test/integration/admin/domains/force_delete_test.rb @@ -3,7 +3,7 @@ require 'test_helper' class AdminAreaDomainForceDeleteTest < ActionDispatch::IntegrationTest include ActionMailer::TestHelper - def setup + setup do login_as users(:admin) @domain = domains(:shop) ActionMailer::Base.deliveries.clear diff --git a/test/integration/admin/domains_test.rb b/test/integration/admin/domains_test.rb index 1dfb0dfd1..81261f017 100644 --- a/test/integration/admin/domains_test.rb +++ b/test/integration/admin/domains_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class AdminDomainsTestTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:admin) end diff --git a/test/integration/admin/registrars/delete_test.rb b/test/integration/admin/registrars/delete_test.rb index 34eb142bf..737c830c1 100644 --- a/test/integration/admin/registrars/delete_test.rb +++ b/test/integration/admin/registrars/delete_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class AdminAreaDeleteRegistrarTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:admin) end diff --git a/test/integration/admin/registrars/edit_test.rb b/test/integration/admin/registrars/edit_test.rb index af420b179..22b93657d 100644 --- a/test/integration/admin/registrars/edit_test.rb +++ b/test/integration/admin/registrars/edit_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class AdminAreaEditRegistrarTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:admin) @registrar = registrars(:bestnames) end diff --git a/test/integration/admin/registrars/new_test.rb b/test/integration/admin/registrars/new_test.rb index 61a7a43be..47b7aded6 100644 --- a/test/integration/admin/registrars/new_test.rb +++ b/test/integration/admin/registrars/new_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class AdminAreaNewRegistrarTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:admin) end diff --git a/test/integration/admin/registrars/show_registrar_test.rb b/test/integration/admin/registrars/show_registrar_test.rb index cb310317c..bb1320d2a 100644 --- a/test/integration/admin/registrars/show_registrar_test.rb +++ b/test/integration/admin/registrars/show_registrar_test.rb @@ -3,7 +3,7 @@ require 'test_helper' class ShowRegistrarTest < ActionDispatch::IntegrationTest include ActionView::Helpers::NumberHelper - def setup + setup do login_as users(:admin) @registrar = registrars(:bestnames) visit admin_registrar_path(@registrar) diff --git a/test/integration/api/domain_transfers_test.rb b/test/integration/api/domain_transfers_test.rb index 08ce2e34c..04301e3e3 100644 --- a/test/integration/api/domain_transfers_test.rb +++ b/test/integration/api/domain_transfers_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class APIDomainTransfersTest < ActionDispatch::IntegrationTest - def setup + setup do @domain = domains(:shop) @new_registrar = registrars(:goodnames) Setting.transfer_wait_time = 0 # Auto-approval diff --git a/test/integration/epp/domain/create/transfer_code_test.rb b/test/integration/epp/domain/create/transfer_code_test.rb index 276386851..60133cb35 100644 --- a/test/integration/epp/domain/create/transfer_code_test.rb +++ b/test/integration/epp/domain/create/transfer_code_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class EppDomainCreateTransferCodeTest < ActionDispatch::IntegrationTest - def setup + setup do travel_to Time.zone.parse('2010-07-05') end diff --git a/test/integration/epp/domain/domain_renew_test.rb b/test/integration/epp/domain/domain_renew_test.rb index a60ec6843..338ecf766 100644 --- a/test/integration/epp/domain/domain_renew_test.rb +++ b/test/integration/epp/domain/domain_renew_test.rb @@ -3,7 +3,7 @@ require 'test_helper' class EppDomainRenewTest < ActionDispatch::IntegrationTest self.use_transactional_fixtures = false - def setup + setup do travel_to Time.zone.parse('2010-07-05') end diff --git a/test/integration/epp/domain/transfer/request_test.rb b/test/integration/epp/domain/transfer/request_test.rb index 508ec4334..f3ea644fc 100644 --- a/test/integration/epp/domain/transfer/request_test.rb +++ b/test/integration/epp/domain/transfer/request_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class EppDomainTransferRequestTest < ActionDispatch::IntegrationTest - def setup + setup do @domain = domains(:shop) @new_registrar = registrars(:goodnames) Setting.transfer_wait_time = 0 diff --git a/test/integration/epp/login/session_limit_test.rb b/test/integration/epp/login/session_limit_test.rb index 513699415..b2fdfe88f 100644 --- a/test/integration/epp/login/session_limit_test.rb +++ b/test/integration/epp/login/session_limit_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class EppLoginSessionLimitTest < ActionDispatch::IntegrationTest - def setup + setup do travel_to Time.zone.parse('2010-07-05') EppSession.delete_all end diff --git a/test/integration/registrant/domains_test.rb b/test/integration/registrant/domains_test.rb index 9b1151b84..db1acfeea 100644 --- a/test/integration/registrant/domains_test.rb +++ b/test/integration/registrant/domains_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class RegistrantDomainsTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:registrant) Setting.days_to_keep_business_registry_cache = 1 diff --git a/test/integration/registrar/domain_transfers_test.rb b/test/integration/registrar/domain_transfers_test.rb index e29a11070..4cd6fc66c 100644 --- a/test/integration/registrar/domain_transfers_test.rb +++ b/test/integration/registrar/domain_transfers_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class RegistrarDomainTransfersTest < ActionDispatch::IntegrationTest - def setup + setup do WebMock.reset! login_as users(:api_goodnames) end diff --git a/test/integration/registrar/nameserver_replacement_test.rb b/test/integration/registrar/nameserver_replacement_test.rb index 9523d2bbd..8ed0f9419 100644 --- a/test/integration/registrar/nameserver_replacement_test.rb +++ b/test/integration/registrar/nameserver_replacement_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class RegistrarNameserverReplacementTest < ActionDispatch::IntegrationTest - def setup + setup do WebMock.reset! login_as users(:api_goodnames) end diff --git a/test/models/contact/contact_test.rb b/test/models/contact/contact_test.rb index 286ded534..b2a7a02a8 100644 --- a/test/models/contact/contact_test.rb +++ b/test/models/contact/contact_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class ContactTest < ActiveSupport::TestCase - def setup + setup do @contact = contacts(:john) end diff --git a/test/models/contact/identical_test.rb b/test/models/contact/identical_test.rb index 61d0946bc..43df8b7d2 100644 --- a/test/models/contact/identical_test.rb +++ b/test/models/contact/identical_test.rb @@ -12,7 +12,7 @@ class ContactIdenticalTest < ActiveSupport::TestCase org_name ] - def setup + setup do @contact = contacts(:william) @identical = contacts(:identical_to_william) end diff --git a/test/models/contact/postal_address_test.rb b/test/models/contact/postal_address_test.rb index 645207a9f..24a5c8002 100644 --- a/test/models/contact/postal_address_test.rb +++ b/test/models/contact/postal_address_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class ContactPostalAddressTest < ActiveSupport::TestCase - def setup + setup do @contact = contacts(:john) end diff --git a/test/models/contact/transfer_test.rb b/test/models/contact/transfer_test.rb index a695c112a..cc35cfdea 100644 --- a/test/models/contact/transfer_test.rb +++ b/test/models/contact/transfer_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class ContactTransferTest < ActiveSupport::TestCase - def setup + setup do @contact = contacts(:john) @new_registrar = registrars(:goodnames) end diff --git a/test/models/domain/domain_test.rb b/test/models/domain/domain_test.rb index c67c8ee87..9be97a8ec 100644 --- a/test/models/domain/domain_test.rb +++ b/test/models/domain/domain_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class DomainTest < ActiveSupport::TestCase - def setup + setup do @domain = domains(:shop) end diff --git a/test/models/domain/transferable_test.rb b/test/models/domain/transferable_test.rb index 35bf5f96b..56b2b21fb 100644 --- a/test/models/domain/transferable_test.rb +++ b/test/models/domain/transferable_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class DomainTransferableTest < ActiveSupport::TestCase - def setup + setup do @domain = domains(:shop) @new_registrar = registrars(:goodnames) end diff --git a/test/models/domain_transfer_test.rb b/test/models/domain_transfer_test.rb index 7f11caf80..9dde09fa7 100644 --- a/test/models/domain_transfer_test.rb +++ b/test/models/domain_transfer_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class DomainTransferTest < ActiveSupport::TestCase - def setup + setup do @domain_transfer = domain_transfers(:shop) end diff --git a/test/models/epp_session_test.rb b/test/models/epp_session_test.rb index fd795b23c..6f90e2445 100644 --- a/test/models/epp_session_test.rb +++ b/test/models/epp_session_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class EppSessionTest < ActiveSupport::TestCase - def setup + setup do @epp_session = epp_sessions(:api_bestnames) end diff --git a/test/models/message_test.rb b/test/models/message_test.rb index 1c7c7bad1..5c557150d 100644 --- a/test/models/message_test.rb +++ b/test/models/message_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class MessageTest < ActiveSupport::TestCase - def setup + setup do @message = messages(:greeting) end diff --git a/test/models/nameserver/glue_record_test.rb b/test/models/nameserver/glue_record_test.rb index 7d5e39d8d..599f04ec6 100644 --- a/test/models/nameserver/glue_record_test.rb +++ b/test/models/nameserver/glue_record_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class NameserverGlueRecordTest < ActiveSupport::TestCase - def setup + setup do @nameserver = nameservers(:shop_ns1) end diff --git a/test/models/nameserver_test.rb b/test/models/nameserver_test.rb index 5a552a7ee..8f2a07334 100644 --- a/test/models/nameserver_test.rb +++ b/test/models/nameserver_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class NameserverTest < ActiveSupport::TestCase - def setup + setup do @nameserver = nameservers(:shop_ns1) end diff --git a/test/models/registrar/code_test.rb b/test/models/registrar/code_test.rb index 7b5188878..579e4d660 100644 --- a/test/models/registrar/code_test.rb +++ b/test/models/registrar/code_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class RegistrarCodeTest < ActiveSupport::TestCase - def setup + setup do @registrar = registrars(:bestnames).dup end diff --git a/test/models/registrar/delete_test.rb b/test/models/registrar/delete_test.rb index 2020c76dc..e9cb20acf 100644 --- a/test/models/registrar/delete_test.rb +++ b/test/models/registrar/delete_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class DeleteRegistrarTest < ActiveSupport::TestCase - def setup + setup do @registrar = registrars(:not_in_use) end diff --git a/test/models/registrar_test.rb b/test/models/registrar_test.rb index fda377d9b..7316633a1 100644 --- a/test/models/registrar_test.rb +++ b/test/models/registrar_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class RegistrarTest < ActiveSupport::TestCase - def setup + setup do @registrar = registrars(:bestnames) end diff --git a/test/test_helper.rb b/test/test_helper.rb index d85d5de73..2f39e06d8 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -28,7 +28,7 @@ class ActionDispatch::IntegrationTest include Capybara::Minitest::Assertions include AbstractController::Translation - def teardown + teardown do Warden.test_reset! Capybara.reset_sessions! Capybara.use_default_driver From 817a672143031e972682f3c3fe06c22373479d26 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 4 Apr 2018 01:43:40 +0300 Subject: [PATCH 76/87] Reset time after each test --- test/test_helper.rb | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/test_helper.rb b/test/test_helper.rb index 2f39e06d8..01392b83c 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -20,6 +20,10 @@ class ActiveSupport::TestCase ActiveRecord::Migration.check_pending! fixtures :all + + teardown do + travel_back + end end class ActionDispatch::IntegrationTest From c2aa9a815e6a5e58343ad735afe5a50422a1c6c0 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 4 Apr 2018 13:31:40 +0300 Subject: [PATCH 77/87] Use setup/teardown blocks instead of methods #792 --- test/integration/admin/domains/details_test.rb | 2 +- test/models/domain/deletable_test.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/test/integration/admin/domains/details_test.rb b/test/integration/admin/domains/details_test.rb index afb62b6ea..a3ec6bdeb 100644 --- a/test/integration/admin/domains/details_test.rb +++ b/test/integration/admin/domains/details_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class AdminAreaDomainDetailsTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:admin) @domain = domains(:shop) end diff --git a/test/models/domain/deletable_test.rb b/test/models/domain/deletable_test.rb index 499ff20b6..dbec7bc7f 100644 --- a/test/models/domain/deletable_test.rb +++ b/test/models/domain/deletable_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class DomainDeletableTest < ActiveSupport::TestCase - def setup + setup do @domain = domains(:shop) end From 2908fd512faa53e6de9549bc4cfc9f6920f02290 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Wed, 4 Apr 2018 15:26:51 +0300 Subject: [PATCH 78/87] Reset setting after each test --- test/models/contact/identical_test.rb | 23 +++++++++++----------- test/models/contact/postal_address_test.rb | 5 +++++ 2 files changed, 16 insertions(+), 12 deletions(-) diff --git a/test/models/contact/identical_test.rb b/test/models/contact/identical_test.rb index 43df8b7d2..61e960a7e 100644 --- a/test/models/contact/identical_test.rb +++ b/test/models/contact/identical_test.rb @@ -13,10 +13,15 @@ class ContactIdenticalTest < ActiveSupport::TestCase ] setup do + @original_address_processing = Setting.address_processing @contact = contacts(:william) @identical = contacts(:identical_to_william) end + teardown do + Setting.address_processing = @original_address_processing + end + def test_returns_identical assert_equal @identical, @contact.identical(@identical.registrar) end @@ -33,30 +38,24 @@ class ContactIdenticalTest < ActiveSupport::TestCase assert_nil @contact.identical(@identical.registrar) end - def test_takes_address_into_account_when_processing_enabled + def test_takes_address_into_account_when_address_processing_is_on + Setting.address_processing = true + Contact.address_attribute_names.each do |attribute| previous_value = @identical.public_send(attribute) @identical.update_attribute(attribute, 'other') - - Contact.stub :address_processing?, true do - assert_nil @contact.identical(@identical.registrar) - end - + assert_nil @contact.identical(@identical.registrar) @identical.update_attribute(attribute, previous_value) end end - def test_ignores_address_when_processing_disabled + def test_ignores_address_when_address_processing_is_off Setting.address_processing = false Contact.address_attribute_names.each do |attribute| previous_value = @identical.public_send(attribute) @identical.update_attribute(attribute, 'other') - - Contact.stub :address_processing?, false do - assert_equal @identical, @contact.identical(@identical.registrar) - end - + assert_equal @identical, @contact.identical(@identical.registrar) @identical.update_attribute(attribute, previous_value) end end diff --git a/test/models/contact/postal_address_test.rb b/test/models/contact/postal_address_test.rb index 24a5c8002..baf06d9f4 100644 --- a/test/models/contact/postal_address_test.rb +++ b/test/models/contact/postal_address_test.rb @@ -2,9 +2,14 @@ require 'test_helper' class ContactPostalAddressTest < ActiveSupport::TestCase setup do + @original_address_processing = Setting.address_processing @contact = contacts(:john) end + teardown do + Setting.address_processing = @original_address_processing + end + def test_invalid_if_country_code_is_invalid_and_address_processing_is_on Setting.address_processing = true @contact.country_code = 'invalid' From 8b8b8df394438a996d7cbcdac991f84eb2fbd3f4 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 5 Apr 2018 16:06:10 +0300 Subject: [PATCH 79/87] Reset WebMock after integration tests --- test/test_helper.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/test/test_helper.rb b/test/test_helper.rb index 01392b83c..931a0d21b 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -34,6 +34,7 @@ class ActionDispatch::IntegrationTest teardown do Warden.test_reset! + WebMock.reset! Capybara.reset_sessions! Capybara.use_default_driver end From 9059779ecee6ed5825a97f973f682a8cde1e6c97 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Fri, 23 Mar 2018 15:47:04 +0200 Subject: [PATCH 80/87] Remove unused CRON tasks --- .reek | 1 - app/models/domain.rb | 5 ----- app/models/domain_cron.rb | 42 --------------------------------------- config/schedule.rb | 4 ---- 4 files changed, 52 deletions(-) diff --git a/.reek b/.reek index 0d5c66aa3..d0f2aaba3 100644 --- a/.reek +++ b/.reek @@ -290,7 +290,6 @@ DuplicateMethodCall: - Domain#renewable? - Domain#set_graceful_expired - DomainCron#self.clean_expired_pendings - - DomainCron#self.delete_legal_doc_duplicates - DomainCron#self.destroy_delete_candidates - DomainCron#self.start_expire_period - DomainCron#self.start_redemption_grace_period diff --git a/app/models/domain.rb b/app/models/domain.rb index 9eef0f30f..f93589c0a 100644 --- a/app/models/domain.rb +++ b/app/models/domain.rb @@ -209,11 +209,6 @@ class Domain < ActiveRecord::Base DomainCron.send(__method__) end - def self.start_delete_period - ActiveSupport::Deprecation.instance.deprecation_warning(DomainCron, __method__) - DomainCron.send(__method__) - end - def self.destroy_delete_candidates ActiveSupport::Deprecation.instance.deprecation_warning(DomainCron, __method__) DomainCron.send(__method__) diff --git a/app/models/domain_cron.rb b/app/models/domain_cron.rb index dcd99328a..fb3b7644d 100644 --- a/app/models/domain_cron.rb +++ b/app/models/domain_cron.rb @@ -76,29 +76,6 @@ class DomainCron marked end - #doing nothing, deprecated - - def self.start_delete_period - # begin - # STDOUT << "#{Time.zone.now.utc} - Setting delete_candidate to domains\n" unless Rails.env.test? - # - # d = Domain.where('delete_at <= ?', Time.zone.now) - # marked = 0 - # real = 0 - # d.each do |domain| - # next unless domain.delete_candidateable? - # real += 1 - # domain.statuses << DomainStatus::DELETE_CANDIDATE - # STDOUT << "#{Time.zone.now.utc} DomainCron.start_delete_period: ##{domain.id} (#{domain.name})\n" unless Rails.env.test? - # ::PaperTrail.whodunnit = "cron - #{__method__}" - # domain.save(validate: false) and marked += 1 - # end - # ensure # the operator should see what was accomplished - # STDOUT << "#{Time.zone.now.utc} - Finished setting delete_candidate - #{marked} out of #{real} successfully set\n" unless Rails.env.test? - # end - # marked - end - def self.destroy_delete_candidates STDOUT << "#{Time.zone.now.utc} - Destroying domains\n" unless Rails.env.test? @@ -128,23 +105,4 @@ class DomainCron STDOUT << "#{Time.zone.now.utc} - Job destroy added for #{c} domains\n" unless Rails.env.test? end - - # rubocop: enable Metrics/AbcSize - # rubocop:enable Rails/FindEach - # rubocop: enable Metrics/LineLength - def self.destroy_with_message(domain) - domain.destroy - bye_bye = domain.versions.last - domain.registrar.messages.create!( - body: "#{I18n.t(:domain_deleted)}: #{domain.name}", - attached_obj_id: bye_bye.id, - attached_obj_type: bye_bye.class.to_s # DomainVersion - ) - end - - def self.delete_legal_doc_duplicates - Rake::Task['legal_doc:remove_duplicates'].reenable - Rake::Task['legal_doc:remove_duplicates'].invoke - end - end diff --git a/config/schedule.rb b/config/schedule.rb index a136ba5c3..6413a9c56 100644 --- a/config/schedule.rb +++ b/config/schedule.rb @@ -46,10 +46,6 @@ if @cron_group == 'registry' runner 'DomainCron.start_expire_period' end - every 50.minutes do - runner 'DomainCron.start_delete_period' - end - every 52.minutes do runner 'DomainCron.start_redemption_grace_period' end From 18ad5e8e5e73eab860d35c4f8d741e100b2a4b34 Mon Sep 17 00:00:00 2001 From: Maciej Szlosarczyk Date: Wed, 11 Apr 2018 09:39:24 +0300 Subject: [PATCH 81/87] Fix missing translation error --- app/views/admin/mail_templates/_form.haml | 4 ++-- app/views/admin/mail_templates/new.haml | 2 +- config/locales/admin/mail_templates.en.yml | 5 +++++ config/locales/en.yml | 1 - test/integration/admin/mail_templates/new_test.rb | 14 ++++++++++++++ 5 files changed, 22 insertions(+), 4 deletions(-) create mode 100644 config/locales/admin/mail_templates.en.yml create mode 100644 test/integration/admin/mail_templates/new_test.rb diff --git a/app/views/admin/mail_templates/_form.haml b/app/views/admin/mail_templates/_form.haml index 739cce665..70873c22a 100644 --- a/app/views/admin/mail_templates/_form.haml +++ b/app/views/admin/mail_templates/_form.haml @@ -37,7 +37,7 @@ = f.email_field :bcc, class: 'form-control' .form-group .col-md-12 - = f.label :body, t(:html_body) + = f.label :body, t('admin.mail_templates.html_body') = liquid_help .col-md-12 = f.text_area(:body, class: 'form-control', size: '15x15') @@ -48,7 +48,7 @@ .col-md-12 = f.text_area(:text_body, class: 'form-control', size: '15x15') - %hr + %hr .row .col-md-12.text-right = button_tag(t(:save), class: 'btn btn-primary') diff --git a/app/views/admin/mail_templates/new.haml b/app/views/admin/mail_templates/new.haml index e5889e2b3..ecf2ea1e8 100644 --- a/app/views/admin/mail_templates/new.haml +++ b/app/views/admin/mail_templates/new.haml @@ -1,3 +1,3 @@ -= render 'shared/title', name: t(:new_mail_template) += render 'shared/title', name: t('admin.mail_templates.new_mail_template') = render 'form', mail_template: @mail_template diff --git a/config/locales/admin/mail_templates.en.yml b/config/locales/admin/mail_templates.en.yml new file mode 100644 index 000000000..2aa6ad52d --- /dev/null +++ b/config/locales/admin/mail_templates.en.yml @@ -0,0 +1,5 @@ +en: + admin: + mail_templates: + html_body: HTML body + new_mail_template: New mail template diff --git a/config/locales/en.yml b/config/locales/en.yml index 1e65f7e29..265293de1 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -731,7 +731,6 @@ en: is_registrant: 'Is registrant' force_delete_set_on_domain: 'Force delete set on domain %{domain_name}' mail_templates: Mail Templates - new_mail_template: New mail template failure: "It was not saved" contact_is_not_valid: 'Contact %{value} is not valid, please fix the invalid contact' welcome_to_eis_registrar_portal: 'Welcome to EIS Registrar portal' diff --git a/test/integration/admin/mail_templates/new_test.rb b/test/integration/admin/mail_templates/new_test.rb new file mode 100644 index 000000000..c7b9dff41 --- /dev/null +++ b/test/integration/admin/mail_templates/new_test.rb @@ -0,0 +1,14 @@ +require 'test_helper' + +class AdminAreaNewMailTemplateTest < ActionDispatch::IntegrationTest + def setup + login_as users(:admin) + end + + def test_new_mail_template_does_not_throw_template_error + visit admin_mail_templates_url + click_link_or_button 'New' + assert_text "HTML body" + assert_text "New mail template" + end +end From bee233e00bc7c7c87155c9060baeb9ea1dae998e Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 12 Apr 2018 12:06:42 +0300 Subject: [PATCH 82/87] Fix incorrect method #623 --- app/controllers/admin/bank_statements_controller.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/admin/bank_statements_controller.rb b/app/controllers/admin/bank_statements_controller.rb index d7bb81fb1..a70387317 100644 --- a/app/controllers/admin/bank_statements_controller.rb +++ b/app/controllers/admin/bank_statements_controller.rb @@ -24,7 +24,7 @@ module Admin @invoice = Invoice.find_by(id: params[:invoice_id]) @bank_transaction = @bank_statement.bank_transactions.build( description: @invoice.to_s, - sum: @invoice.amount, + sum: @invoice.total, reference_no: @invoice.reference_no, paid_at: Time.zone.now.to_date, currency: 'EUR' From a8046ce9cffa8592a4fd198b89e7061dd4cd545b Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 12 Apr 2018 17:27:39 +0300 Subject: [PATCH 83/87] Remove global setting for tests #623 --- test/integration/admin/registrars/new_test.rb | 7 +++++++ test/test_helper.rb | 1 - 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/test/integration/admin/registrars/new_test.rb b/test/integration/admin/registrars/new_test.rb index 47b7aded6..cadb513e9 100644 --- a/test/integration/admin/registrars/new_test.rb +++ b/test/integration/admin/registrars/new_test.rb @@ -3,6 +3,12 @@ require 'test_helper' class AdminAreaNewRegistrarTest < ActionDispatch::IntegrationTest setup do login_as users(:admin) + @original_registry_country_code = Setting.registry_country_code + Setting.registry_country_code = 'DE' + end + + teardown do + Setting.registry_country_code = @original_registry_country_code end def test_new_registrar_creation_with_required_params @@ -12,6 +18,7 @@ class AdminAreaNewRegistrarTest < ActionDispatch::IntegrationTest fill_in 'Name', with: 'Brand new names' fill_in 'Reg no', with: '55555555' fill_in 'Contact e-mail', with: 'test@example.com' + select 'Germany', from: 'Country' fill_in 'Accounting customer code', with: 'test' fill_in 'Code', with: 'test' diff --git a/test/test_helper.rb b/test/test_helper.rb index e319656be..931a0d21b 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -14,7 +14,6 @@ require 'webmock/minitest' require 'support/rails5_assetions' # Remove once upgraded to Rails 5 Setting.address_processing = false -Setting.registry_country_code = 'US' class ActiveSupport::TestCase include FactoryBot::Syntax::Methods From 963f6f87cf9338c24f6f5021e5f01bc395218ce3 Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 12 Apr 2018 17:31:04 +0300 Subject: [PATCH 84/87] Use setup block instead of method in tests #623 --- test/integration/admin/mail_templates/new_test.rb | 2 +- test/integration/admin/registrars/details_test.rb | 2 +- test/integration/registrar/billing/balance_top_up_test.rb | 2 +- test/models/invoice_test.rb | 2 +- test/models/registrar/vat_test.rb | 2 +- test/models/registry_test.rb | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/test/integration/admin/mail_templates/new_test.rb b/test/integration/admin/mail_templates/new_test.rb index c7b9dff41..42d50ef43 100644 --- a/test/integration/admin/mail_templates/new_test.rb +++ b/test/integration/admin/mail_templates/new_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class AdminAreaNewMailTemplateTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:admin) end diff --git a/test/integration/admin/registrars/details_test.rb b/test/integration/admin/registrars/details_test.rb index 3ea7fa187..3c2f39072 100644 --- a/test/integration/admin/registrars/details_test.rb +++ b/test/integration/admin/registrars/details_test.rb @@ -3,7 +3,7 @@ require 'test_helper' class AdminAreaRegistrarDetailsTest < ActionDispatch::IntegrationTest include ActionView::Helpers::NumberHelper - def setup + setup do login_as users(:admin) @registrar = registrars(:complete) end diff --git a/test/integration/registrar/billing/balance_top_up_test.rb b/test/integration/registrar/billing/balance_top_up_test.rb index 3c36b5f42..c77c9ff37 100644 --- a/test/integration/registrar/billing/balance_top_up_test.rb +++ b/test/integration/registrar/billing/balance_top_up_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class BalanceTopUpTest < ActionDispatch::IntegrationTest - def setup + setup do login_as users(:api_bestnames) end diff --git a/test/models/invoice_test.rb b/test/models/invoice_test.rb index 5ae54abd0..45ca57417 100644 --- a/test/models/invoice_test.rb +++ b/test/models/invoice_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class InvoiceTest < ActiveSupport::TestCase - def setup + setup do @invoice = invoices(:valid) end diff --git a/test/models/registrar/vat_test.rb b/test/models/registrar/vat_test.rb index 6ba16f8e4..2dc5de3ce 100644 --- a/test/models/registrar/vat_test.rb +++ b/test/models/registrar/vat_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class RegistrarVATTest < ActiveSupport::TestCase - def setup + setup do @registrar = registrars(:bestnames) end diff --git a/test/models/registry_test.rb b/test/models/registry_test.rb index 70bc037c6..90a603e15 100644 --- a/test/models/registry_test.rb +++ b/test/models/registry_test.rb @@ -1,7 +1,7 @@ require 'test_helper' class RegistryTest < ActiveSupport::TestCase - def setup + setup do @registry = Registry.send(:new) end From 5061962c40836fabb7ca4ef1d8056b2ab7b4236d Mon Sep 17 00:00:00 2001 From: Artur Beljajev Date: Thu, 12 Apr 2018 17:48:33 +0300 Subject: [PATCH 85/87] Fix test #623 --- test/integration/admin/registrars/new_test.rb | 8 +------- test/test_helper.rb | 1 + 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/test/integration/admin/registrars/new_test.rb b/test/integration/admin/registrars/new_test.rb index cadb513e9..d2efc31a2 100644 --- a/test/integration/admin/registrars/new_test.rb +++ b/test/integration/admin/registrars/new_test.rb @@ -3,12 +3,6 @@ require 'test_helper' class AdminAreaNewRegistrarTest < ActionDispatch::IntegrationTest setup do login_as users(:admin) - @original_registry_country_code = Setting.registry_country_code - Setting.registry_country_code = 'DE' - end - - teardown do - Setting.registry_country_code = @original_registry_country_code end def test_new_registrar_creation_with_required_params @@ -18,7 +12,7 @@ class AdminAreaNewRegistrarTest < ActionDispatch::IntegrationTest fill_in 'Name', with: 'Brand new names' fill_in 'Reg no', with: '55555555' fill_in 'Contact e-mail', with: 'test@example.com' - select 'Germany', from: 'Country' + select 'United States', from: 'Country' fill_in 'Accounting customer code', with: 'test' fill_in 'Code', with: 'test' diff --git a/test/test_helper.rb b/test/test_helper.rb index 931a0d21b..e319656be 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -14,6 +14,7 @@ require 'webmock/minitest' require 'support/rails5_assetions' # Remove once upgraded to Rails 5 Setting.address_processing = false +Setting.registry_country_code = 'US' class ActiveSupport::TestCase include FactoryBot::Syntax::Methods From 359e4448392a0c71d619591dca880ff2260a37ff Mon Sep 17 00:00:00 2001 From: Maciej Szlosarczyk Date: Fri, 13 Apr 2018 15:44:19 +0300 Subject: [PATCH 86/87] Config for test and development docker containers --- Dockerfile | 23 +++++++++++++++++++++++ config/application.rb | 8 ++++++++ config/database-example.yml | 12 ++++++------ docker-compose.yml | 33 +++++++++++++++++++++++++++++++++ docker/docker_dev.sh | 6 ++++++ docker/docker_test.sh | 8 ++++++++ docker/nginx.template | 29 +++++++++++++++++++++++++++++ 7 files changed, 113 insertions(+), 6 deletions(-) create mode 100644 Dockerfile create mode 100644 docker-compose.yml create mode 100755 docker/docker_dev.sh create mode 100755 docker/docker_test.sh create mode 100644 docker/nginx.template diff --git a/Dockerfile b/Dockerfile new file mode 100644 index 000000000..0c2c51e25 --- /dev/null +++ b/Dockerfile @@ -0,0 +1,23 @@ +FROM ruby:2.2 +MAINTAINER maciej.szlosarczyk@internet.ee + +RUN apt-get update > /dev/null && apt-get install -y > /dev/null \ + build-essential \ + nodejs \ + imagemagick \ + postgresql-client + +RUN apt-get install -y > /dev/null \ + qt5-default \ + libqt5webkit5-dev \ + gstreamer1.0-plugins-base \ + gstreamer1.0-tools \ + qtdeclarative5-dev \ + gstreamer1.0-x + +RUN mkdir -p /opt/webapps/app/tmp/pids +WORKDIR /opt/webapps/app + +COPY Gemfile Gemfile.lock ./ +RUN gem install bundler && bundle install --jobs 20 --retry 5 +EXPOSE 3000 diff --git a/config/application.rb b/config/application.rb index 04d697f55..8232ddf68 100644 --- a/config/application.rb +++ b/config/application.rb @@ -16,6 +16,14 @@ require 'rails/all' Bundler.require(*Rails.groups) module DomainNameRegistry + + # Add "db" to the list hosts on which you can run `rake db:setup:all` +# Only allow that in test and development. +if ['development', 'test'].include?(Rails.env) + ActiveRecord::Tasks::DatabaseTasks::LOCAL_HOSTS << "db" +end + +module Registry class Application < Rails::Application # Settings in config/environments/* take precedence over those specified here. # Application configuration should go into files in config/initializers diff --git a/config/database-example.yml b/config/database-example.yml index 8def2a36f..ca47a9979 100644 --- a/config/database-example.yml +++ b/config/database-example.yml @@ -4,14 +4,14 @@ # Registrant example is at database-example-registrant.yml file default: &default - host: localhost adapter: postgresql encoding: unicode - pool: 5 - username: registry - password: registry_pwd + pool: <%= ENV.fetch("APP_DB_MAX_THREADS") { 5 } %> + host: <%= ENV.fetch("APP_DBHOST") { "localhost" } %> + username: <%= ENV.fetch("APP_DBUSER") { "postgres" } %> + password: -# +# # Staging config For EPP, REPP, Admin, Registrar # @@ -27,7 +27,7 @@ api_log_staging: <<: *default database: registry_api_log_staging -# +# # Production config For EPP, REPP, Admin, Registrar # production: diff --git a/docker-compose.yml b/docker-compose.yml new file mode 100644 index 000000000..5c5f523de --- /dev/null +++ b/docker-compose.yml @@ -0,0 +1,33 @@ +version: "3.2" + +services: + app: + tty: true + stdin_open: true + build: + context: . + dockerfile: Dockerfile + links: + - db + environment: + - APP_DBHOST=db + volumes: + - .:/opt/webapps/app + ports: + - "3000:3000" + command: bash -c "rm -f tmp/pids/server.pid && bundle exec rails server -p 3000 -b '0.0.0.0'" + + web: + image: nginx + volumes: + - ./docker/nginx.template:/etc/nginx/conf.d/nginx.template + ports: + - "80:80" + links: + - app + environment: + APP: 'app' + command: /bin/bash -c "envsubst '$$APP' < /etc/nginx/conf.d/nginx.template > /etc/nginx/conf.d/default.conf && nginx -g 'daemon off;'" + + db: + image: postgres:9.4 diff --git a/docker/docker_dev.sh b/docker/docker_dev.sh new file mode 100755 index 000000000..f5592517f --- /dev/null +++ b/docker/docker_dev.sh @@ -0,0 +1,6 @@ +# /bin/sh +docker-compose down +docker-compose build +docker-compose run app rake db:setup:all +docker-compose run app rake db:migrate +docker-compose run app rake dev:prime diff --git a/docker/docker_test.sh b/docker/docker_test.sh new file mode 100755 index 000000000..7239f78a9 --- /dev/null +++ b/docker/docker_test.sh @@ -0,0 +1,8 @@ +# /bin/sh +docker-compose down +docker-compose build + +# Setup test database +docker-compose run app rake db:setup:all test +# Finally run tests to check if everything is in order +docker-compose run app rspec diff --git a/docker/nginx.template b/docker/nginx.template new file mode 100644 index 000000000..50f6e8548 --- /dev/null +++ b/docker/nginx.template @@ -0,0 +1,29 @@ +log_format le_json '{ "time": "$time_iso8601", ' + '"remote_addr": "$remote_addr", ' + '"remote_user": "$remote_user", ' + '"body_bytes_sent": "$body_bytes_sent", ' + '"request_time": "$request_time", ' + '"status": "$status", ' + '"request": "$request", ' + '"request_method": "$request_method", ' + '"http_referrer": "$http_referer", ' + '"http_user_agent": "$http_user_agent" }'; + +upstream app { + server ${APP}:3000; +} + +server { + listen 80; + + access_log /var/log/nginx/access.log le_json; + + location / { + proxy_set_header X-Real-IP $remote_addr; + proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for; + proxy_set_header Host $http_host; + proxy_redirect off; + proxy_pass http://app; + break; + } +} From 5ad5491fc114d9347dc93eaa766fcfe7fc672e5e Mon Sep 17 00:00:00 2001 From: Maciej Szlosarczyk Date: Fri, 13 Apr 2018 15:50:51 +0300 Subject: [PATCH 87/87] Fix syntax error that came from merge --- config/application.rb | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/config/application.rb b/config/application.rb index 8232ddf68..0d043fa5e 100644 --- a/config/application.rb +++ b/config/application.rb @@ -15,15 +15,13 @@ require 'rails/all' # you've limited to :test, :development, or :production. Bundler.require(*Rails.groups) -module DomainNameRegistry - - # Add "db" to the list hosts on which you can run `rake db:setup:all` +# Add "db" to the list hosts on which you can run `rake db:setup:all` # Only allow that in test and development. if ['development', 'test'].include?(Rails.env) ActiveRecord::Tasks::DatabaseTasks::LOCAL_HOSTS << "db" end -module Registry +module DomainNameRegistry class Application < Rails::Application # Settings in config/environments/* take precedence over those specified here. # Application configuration should go into files in config/initializers