From 0b7dd5b965d81c9c3a9273adaeeee73dd7f0e4d2 Mon Sep 17 00:00:00 2001 From: Agniva De Sarker Date: Thu, 27 Oct 2022 13:57:46 +0530 Subject: [PATCH] MM-46450: Enable skipped test (#21526) After looking at this for some time, I don't see any way for this test to fail other than model.NewId returning the same value for BasicUser and BasicUser2. I think edge cases like this can exist from time to time. Nothing much we can do about it. https://mattermost.atlassian.net/browse/MM-46450 ```release-note NONE ``` --- api4/user_test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/api4/user_test.go b/api4/user_test.go index 80bdea263e..ccfcee0be2 100644 --- a/api4/user_test.go +++ b/api4/user_test.go @@ -1027,8 +1027,9 @@ func TestGetUserByEmail(t *testing.T) { }) } +// This test can flake if two calls to model.NewId can return the same value. +// Not much can be done about it. func TestSearchUsers(t *testing.T) { - t.Skip("MM-46450") th := Setup(t).InitBasic() defer th.TearDown()