aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--app/workers/workbench_import_worker.rb2
-rw-r--r--config/application.rb1
-rw-r--r--config/initializers/workbench_import.rb2
-rw-r--r--spec/concerns/configurable_spec.rb4
-rw-r--r--spec/models/import_spec.rb3
-rw-r--r--spec/models/netex_import_spec.rb4
-rw-r--r--spec/services/retry_service_spec.rb5
-rw-r--r--spec/services/zip_service/zip_output_streams_spec.rb2
-rw-r--r--spec/support/fixtures_helper.rb3
-rw-r--r--spec/tasks/reflex_rake_spec.rb4
-rw-r--r--spec/workers/workbench_import_worker_spec.rb6
11 files changed, 21 insertions, 15 deletions
diff --git a/app/workers/workbench_import_worker.rb b/app/workers/workbench_import_worker.rb
index 982d3e0d6..488fbfc0b 100644
--- a/app/workers/workbench_import_worker.rb
+++ b/app/workers/workbench_import_worker.rb
@@ -106,7 +106,7 @@ class WorkbenchImportWorker
Rails.application.config.front_end_host
end
def import_path
- @__import_path__ ||= File.join(download_workbench_import_path(@import.workbench, @import))
+ @__import_path__ ||= download_workbench_import_path(@import.workbench, @import)
end
def import_url
@__import_url__ ||= File.join(import_host, import_path)
diff --git a/config/application.rb b/config/application.rb
index e2f6f8e7b..05a9752b6 100644
--- a/config/application.rb
+++ b/config/application.rb
@@ -15,7 +15,6 @@ module ChouetteIhm
# Application configuration should go into files in config/initializers
# -- all .rb files in that directory are automatically loaded.
config.autoload_paths << config.root.join('lib')
- config.autoload_paths << config.root.join('concerns')
# custom exception pages
config.exceptions_app = self.routes
diff --git a/config/initializers/workbench_import.rb b/config/initializers/workbench_import.rb
index 1e405c9ca..89ddd72ef 100644
--- a/config/initializers/workbench_import.rb
+++ b/config/initializers/workbench_import.rb
@@ -1,5 +1,5 @@
WorkbenchImportWorker.config do | config |
config.dir = ENV.fetch('WORKBENCH_IMPORT_DIR'){ Rails.root.join 'tmp/workbench_import' }
- FileUtils.mkdir_p config.dir rescue nil
+ FileUtils.mkdir_p config.dir
end
diff --git a/spec/concerns/configurable_spec.rb b/spec/concerns/configurable_spec.rb
index 822f572c1..330241b72 100644
--- a/spec/concerns/configurable_spec.rb
+++ b/spec/concerns/configurable_spec.rb
@@ -14,7 +14,9 @@ RSpec.describe Configurable do
subject.config.something = something
expect( subject.config.something ).to eq(something)
+ # Instances delegate to the class
expect( subject.new.send(:config).something ).to eq(something)
+ # **All** instances delegate to the class
expect( subject.new.send(:config).something ).to eq(something)
end
@@ -25,7 +27,9 @@ RSpec.describe Configurable do
end
expect( subject.config.something ).to eq(something)
+ # Instances delegate to the class
expect( subject.new.send(:config).something ).to eq(something)
+ # **All** instances delegate to the class
expect( subject.new.send(:config).something ).to eq(something)
end
end
diff --git a/spec/models/import_spec.rb b/spec/models/import_spec.rb
index ee30ce8ba..6546b14ab 100644
--- a/spec/models/import_spec.rb
+++ b/spec/models/import_spec.rb
@@ -1,5 +1,6 @@
RSpec.describe Import, :type => :model do
subject{ build_stubbed :import }
+
it { should belong_to(:referential) }
it { should belong_to(:workbench) }
it { should belong_to(:parent).class_name(described_class.to_s) }
@@ -7,4 +8,6 @@ RSpec.describe Import, :type => :model do
it { should enumerize(:status).in("aborted", "canceled", "failed", "new", "pending", "running", "successful") }
it { should validate_presence_of(:file) }
+ it { should validate_presence_of(:referential) }
+ it { should validate_presence_of(:workbench) }
end
diff --git a/spec/models/netex_import_spec.rb b/spec/models/netex_import_spec.rb
deleted file mode 100644
index 066595f06..000000000
--- a/spec/models/netex_import_spec.rb
+++ /dev/null
@@ -1,4 +0,0 @@
-RSpec.describe NetexImport, type: :model do
- it { should validate_presence_of(:referential) }
- it { should validate_presence_of(:workbench) }
-end
diff --git a/spec/services/retry_service_spec.rb b/spec/services/retry_service_spec.rb
index ce150f808..bb3416373 100644
--- a/spec/services/retry_service_spec.rb
+++ b/spec/services/retry_service_spec.rb
@@ -42,7 +42,7 @@ RSpec.describe RetryService do
expect( subject ).to receive(:sleep).with(2)
end
- it 'succeds the second time' do
+ it 'succeeds the second time' do
expect( subject.execute{ succeed_later(ArgumentError){ 42 } } ).to eq(Result.ok(42))
end
@@ -58,7 +58,8 @@ RSpec.describe RetryService do
expect( subject ).to receive(:sleep).with(3)
end
it 'succeeds the third time with try again (automatically registered exception)' do
- expect( subject.execute{ succeed_later(RetryService::Retry, count: 2){ 42 } } ).to eq(Result.ok(42))
+ result = subject.execute{ succeed_later(RetryService::Retry, count: 2){ 42 } }
+ expect( result ).to eq( Result.ok(42) )
end
end
diff --git a/spec/services/zip_service/zip_output_streams_spec.rb b/spec/services/zip_service/zip_output_streams_spec.rb
index b99e7bc2d..d2452bfd8 100644
--- a/spec/services/zip_service/zip_output_streams_spec.rb
+++ b/spec/services/zip_service/zip_output_streams_spec.rb
@@ -15,7 +15,7 @@ RSpec.describe ZipService do
expect( ref2_lines ).to eq %w(multiref/ref2/ multiref/ref2/datum-1 multiref/ref2/datum-2)
end
- it "exposes it's size" do
+ it "exposes its size" do
expect( subject.entry_group_streams.size ).to eq(2)
end
end
diff --git a/spec/support/fixtures_helper.rb b/spec/support/fixtures_helper.rb
index 81f6ce838..20963261b 100644
--- a/spec/support/fixtures_helper.rb
+++ b/spec/support/fixtures_helper.rb
@@ -4,6 +4,9 @@ module Support
Rails.root.join( fixture_path, *segments )
end
+ def open_fixture *segments
+ File.open(fixtures_path(*segments))
+ end
def read_fixture *segments
File.read(fixtures_path(*segments))
end
diff --git a/spec/tasks/reflex_rake_spec.rb b/spec/tasks/reflex_rake_spec.rb
index 04c5886aa..6ece223d2 100644
--- a/spec/tasks/reflex_rake_spec.rb
+++ b/spec/tasks/reflex_rake_spec.rb
@@ -5,7 +5,7 @@ describe 'reflex:sync' do
before(:each) do
['getOP', 'getOR'].each do |method|
stub_request(:get, "#{Rails.application.config.reflex_api_url}/?format=xml&idRefa=0&method=#{method}").
- to_return(body: File.open("#{fixture_path}/reflex.zip"), status: 200)
+ to_return(body: open_fixture('reflex.zip'), status: 200)
end
stop_area_ref = create(:stop_area_referential, name: 'Reflex')
@@ -43,7 +43,7 @@ describe 'reflex:sync' do
before(:each) do
['getOP', 'getOR'].each do |method|
stub_request(:get, "#{Rails.application.config.reflex_api_url}/?format=xml&idRefa=0&method=#{method}").
- to_return(body: File.open("#{fixture_path}/reflex_updated.zip"), status: 200)
+ to_return(body: open_fixture('reflex_updated.zip'), status: 200)
end
Stif::ReflexSynchronization.synchronize
end
diff --git a/spec/workers/workbench_import_worker_spec.rb b/spec/workers/workbench_import_worker_spec.rb
index 03206113c..3ed5833dc 100644
--- a/spec/workers/workbench_import_worker_spec.rb
+++ b/spec/workers/workbench_import_worker_spec.rb
@@ -1,7 +1,7 @@
RSpec.describe WorkbenchImportWorker, type: [:worker, :request] do
let( :worker ) { described_class.new }
- let( :import ){ build_stubbed :import, token_download: download_token, file: File.open(zip_file) }
+ let( :import ){ build_stubbed :import, token_download: download_token, file: zip_file }
let( :workbench ){ import.workbench }
let( :referential ){ import.referential }
@@ -21,7 +21,7 @@ RSpec.describe WorkbenchImportWorker, type: [:worker, :request] do
let( :download_token ){ SecureRandom.urlsafe_base64 }
- let( :upload_path ) { '/api/v1/netex_imports.json' }
+ let( :upload_path ) { api_v1_netex_imports_path(format: :json) }
let( :entry_group_streams ) do
entry_count.times.map{ |i| double( "entry group stream #{i}" ) }
@@ -33,7 +33,7 @@ RSpec.describe WorkbenchImportWorker, type: [:worker, :request] do
end
let( :zip_service ){ double("zip service") }
- let( :zip_file ){ File.join(fixture_path, 'multiref.zip') }
+ let( :zip_file ){ open_fixture('multiref.zip') }
let( :post_response_ok ){ response(status: 201, boody: "{}") }