From 1f0af303f3b03c1b6644e5ca9f2eb72b476d8061 Mon Sep 17 00:00:00 2001 From: gregoirenovel Date: Mon, 15 Oct 2018 15:44:34 +0200 Subject: [PATCH] Remove the unused ModuleApiCarto#classes method --- app/models/module_api_carto.rb | 14 ---------- spec/models/module_api_carto_spec.rb | 40 ---------------------------- 2 files changed, 54 deletions(-) diff --git a/app/models/module_api_carto.rb b/app/models/module_api_carto.rb index 179d89ef2..2dee12b93 100644 --- a/app/models/module_api_carto.rb +++ b/app/models/module_api_carto.rb @@ -4,18 +4,4 @@ class ModuleAPICarto < ApplicationRecord validates :use_api_carto, presence: true, allow_blank: true, allow_nil: false validates :quartiers_prioritaires, presence: true, allow_blank: true, allow_nil: false validates :cadastre, presence: true, allow_blank: true, allow_nil: false - - def classes - modules = '' - - if quartiers_prioritaires? - modules += 'qp ' - end - - if cadastre? - modules += 'cadastre ' - end - - modules - end end diff --git a/spec/models/module_api_carto_spec.rb b/spec/models/module_api_carto_spec.rb index 80fca6b82..7672fd865 100644 --- a/spec/models/module_api_carto_spec.rb +++ b/spec/models/module_api_carto_spec.rb @@ -10,44 +10,4 @@ describe ModuleAPICarto do it { is_expected.to have_db_column(:quartiers_prioritaires) } it { is_expected.to have_db_column(:cadastre) } end - - describe '#classes' do - let(:module_api_carto) { create(:module_api_carto, quartiers_prioritaires: qp, cadastre: cadastre) } - - context 'when module api carto qp is true' do - let(:qp) { true } - let(:cadastre) { false } - - subject { module_api_carto.classes } - - it { is_expected.to eq 'qp ' } - end - - context 'when module api carto cadastre is true' do - let(:qp) { false } - let(:cadastre) { true } - - subject { module_api_carto.classes } - - it { is_expected.to eq 'cadastre ' } - end - - context 'when module api carto qp is true and cadastre is true' do - let(:qp) { true } - let(:cadastre) { true } - - subject { module_api_carto.classes } - - it { is_expected.to eq 'qp cadastre ' } - end - - context 'when module api carto qp is false and cadastre is false' do - let(:qp) { false } - let(:cadastre) { false } - - subject { module_api_carto.classes } - - it { is_expected.to eq '' } - end - end end