Skip to content

Commit 5d44bd4

Browse files
authored
MONGOID-5394 Failing test: Destroying embeds_many document when _id is nil deletes all docs (#5342)
* MONGOID-5394 add failing test * MONGOID-5394 add another failing test and pend both
1 parent 36542c6 commit 5d44bd4

File tree

1 file changed

+22
-0
lines changed

1 file changed

+22
-0
lines changed

spec/mongoid/association/embedded/embeds_many/proxy_spec.rb

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3960,6 +3960,28 @@ class TrackingIdValidationHistory
39603960
end
39613961
end
39623962

3963+
context "when destroying a document with multiple nil _ids" do
3964+
let(:congress) { EmmCongress.create! }
3965+
3966+
before do
3967+
congress.legislators << EmmLegislator.new(_id: nil, a: 1)
3968+
congress.legislators << EmmLegislator.new(_id: nil, a: 2)
3969+
3970+
congress.legislators[0].destroy
3971+
end
3972+
3973+
it "deletes the correct document locally" do
3974+
pending "MONGOID-5394"
3975+
expect(congress.legislators.length).to eq(1)
3976+
expect(congress.legislators.first.a).to eq(1)
3977+
end
3978+
3979+
it "only deletes the one document" do
3980+
pending "MONGOID-5394"
3981+
expect(congress.reload.legislators.length).to eq(1)
3982+
end
3983+
end
3984+
39633985
context "when adding a document" do
39643986

39653987
let(:person) do

0 commit comments

Comments
 (0)