diff options
| author | Teddy Wing | 2018-02-07 12:50:46 +0100 |
|---|---|---|
| committer | Teddy Wing | 2018-02-08 11:29:31 +0100 |
| commit | 6134ed98a3fe29276a17a0d79a70a8ce3e3f548b (patch) | |
| tree | f52fb6a48008171ce49546be300861155617f8fe /spec/services/parent_notifier_spec.rb | |
| parent | ee1781d525bc04aff3f7f743be8297ee003ee9e0 (diff) | |
| download | chouette-core-6134ed98a3fe29276a17a0d79a70a8ce3e3f548b.tar.bz2 | |
Add `ParentNotifier` service
This is a takeoff on `ParentImportNotifier`. I just copied over the
class and spec and generalised the service to work with `Import`-style
interfaces. We can now use the same notifier service class for both
imports and compliance check sets.
Refs #4758
Diffstat (limited to 'spec/services/parent_notifier_spec.rb')
| -rw-r--r-- | spec/services/parent_notifier_spec.rb | 90 |
1 files changed, 90 insertions, 0 deletions
diff --git a/spec/services/parent_notifier_spec.rb b/spec/services/parent_notifier_spec.rb new file mode 100644 index 000000000..ecf508fcd --- /dev/null +++ b/spec/services/parent_notifier_spec.rb @@ -0,0 +1,90 @@ +RSpec.describe ParentNotifier do + let(:workbench_import) { create(:workbench_import) } + + describe ".notify_when_finished" do + it "calls #notify_parent on finished imports" do + workbench_import = build_stubbed(:workbench_import) + + finished_statuses = [:failed, :successful, :aborted, :canceled] + netex_imports = [] + + finished_statuses.each do |status| + netex_imports << build_stubbed( + :netex_import, + parent: workbench_import, + status: status + ) + end + + netex_imports.each do |netex_import| + expect(netex_import).to receive(:notify_parent) + end + + ParentNotifier.new(Import).notify_when_finished(netex_imports) + end + + it "doesn't call #notify_parent if its `notified_parent_at` is set" do + netex_import = create( + :netex_import, + parent: workbench_import, + status: :failed, + notified_parent_at: DateTime.now + ) + + expect(netex_import).not_to receive(:notify_parent) + + ParentNotifier.new(Import).notify_when_finished + end + end + + describe ".objects_pending_notification" do + it "includes imports with a parent and `notified_parent_at` unset" do + netex_import = create( + :netex_import, + parent: workbench_import, + status: :successful, + notified_parent_at: nil + ) + + expect( + ParentNotifier.new(Import).objects_pending_notification + ).to eq([netex_import]) + end + + it "doesn't include imports without a parent" do + create(:import, parent: nil) + + expect( + ParentNotifier.new(Import).objects_pending_notification + ).to be_empty + end + + it "doesn't include imports that aren't finished" do + [:new, :pending, :running].each do |status| + create( + :netex_import, + parent: workbench_import, + status: status, + notified_parent_at: nil + ) + end + + expect( + ParentNotifier.new(Import).objects_pending_notification + ).to be_empty + end + + it "doesn't include imports that have already notified their parent" do + create( + :netex_import, + parent: workbench_import, + status: :successful, + notified_parent_at: DateTime.now + ) + + expect( + ParentNotifier.new(Import).objects_pending_notification + ).to be_empty + end + end +end |
