Patch #34169

MessagesControllerTest#test_post_new randomly fails

Added by Mizuki ISHIKAWA about 1 month ago. Updated 26 days ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:Code cleanup/refactoring
Target version:4.2.0

Description

MessagesControllerTest # test_post_new test fails (trunk)

Failure:
MessagesControllerTest#test_post_new [/root/project/test/functional/messages_controller_test.rb:150]:
Expected: ["jsmith@somenet.foo"]
  Actual: ["dlopper@somenet.foo"]

bin/rails test test/functional/messages_controller_test.rb:121

There is no guarantee that the elements will be in the order of "first: author, second: member" in the array mails, so the test should be modified so that it does not depend on the order.

fix_test_post_new.patch Magnifier (732 Bytes) Mizuki ISHIKAWA, 2020-10-26 07:39

Associated revisions

Revision 20242
Added by Go MAEDA 26 days ago

MessagesControllerTest#test_post_new randomly fails (#34169).

Patch by Mizuki ISHIKAWA.

History

#1 Updated by Go MAEDA about 1 month ago

  • Subject changed from MessagesControllerTest#test_post_new fails to MessagesControllerTest#test_post_new randomly fails
  • Target version set to 4.2.0

Setting the target version to 4.2.0.

#2 Updated by vzvu 3k6k about 1 month ago

LGTM and thanks for fixing. This test was very flaky on PostgreSQL and caused many false positive failures.

This fix will make development much easier.

#3 Updated by Go MAEDA 26 days ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed the patch. Thank you.

Also available in: Atom PDF