diff options
| author | Teddy Wing | 2017-11-13 17:16:24 +0100 |
|---|---|---|
| committer | Teddy Wing | 2017-11-13 17:16:24 +0100 |
| commit | 336dab9900aee831e73113e400609ec2ebd40c20 (patch) | |
| tree | 4db533e1e07d1c6f55871bef61b4adeb7945ea15 /spec | |
| parent | 95095886ceb51a2036eea6ffda103d3ad590322b (diff) | |
| download | chouette-core-336dab9900aee831e73113e400609ec2ebd40c20.tar.bz2 | |
Import: Destroy associated `Referential` on destroy if ready:false
If the `Import`'s associated `Referential` has `ready: false`, destroy
that `Referential` when the import is destroyed.
Aaand looking back at the task it turns out that's not what I was
supposed to do. Right, it does seem a little weird in retrospect.
Instead we want to delete the `Referential`s of child imports.
Refs #4962
Diffstat (limited to 'spec')
| -rw-r--r-- | spec/models/import_spec.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/spec/models/import_spec.rb b/spec/models/import_spec.rb index 7be05908a..dfb947711 100644 --- a/spec/models/import_spec.rb +++ b/spec/models/import_spec.rb @@ -57,6 +57,24 @@ RSpec.describe Import, type: :model do expect(ImportMessage.count).to eq(0) end + + it "must destroy its associated Referential if ready: false" do + referential = create(:referential, ready: false) + import = create(:import, referential: referential) + + import.destroy + + expect(referential).to be_destroyed + end + + it "must not destroy its associated Referential if ready: true" do + referential = create(:referential, ready: true) + import = create(:import, referential: referential) + + import.destroy + + expect(referential).not_to be_destroyed + end end describe "#notify_parent" do |
