No need to assert node was created after drupalCreateNode (as mentioned by catch)

CommentFileSizeAuthor
comment_remove_assert_1.patch497 bytesamitaibu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Hm. Should we move this into drupalCreateNode() instead so that it's always clear when this is called?

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Needs review » Fixed

Nevermind. I just read #311989: Check node is in database after drupalCreateNode.

Committed. Thanks!

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
CreditAttribution: amitaibu commented

Aha, the joy of first core patch being committed :)) thanks!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Hey!! Congrats!! :D

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.