Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Commit 0708a73

Browse files
sanderginnBolajiOlajide
authored andcommitted
codeintel: disable flaky test (#43470)
* codeintel: disable flaky test * Add explicit flake
1 parent df4277b commit 0708a73

File tree

3 files changed

+14
-1
lines changed

3 files changed

+14
-1
lines changed
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
[
2+
{
3+
"path": "internal/codeintel/policies/internal/store",
4+
"prefix": "TestSelectPoliciesForRepositoryMembershipUpdate",
5+
"reason": "The same policies are returned but in an unexpected order."
6+
}
7+
]

internal/codeintel/policies/internal/store/store_repo_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -199,6 +199,8 @@ func TestUpdateReposMatchingPatternsOverLimit(t *testing.T) {
199199
}
200200

201201
func TestSelectPoliciesForRepositoryMembershipUpdate(t *testing.T) {
202+
t.Skip("Flaky due to unexpected ordering of results: https://github.com/sourcegraph/sourcegraph/issues/43472")
203+
202204
logger := logtest.Scoped(t)
203205
db := database.NewDB(logger, dbtest.NewDB(logger, t))
204206
store := testStoreWithoutConfigurationPolicies(t, db)

internal/search/result/result_type.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
package result
22

3-
import "strings"
3+
import (
4+
"sort"
5+
"strings"
6+
)
47

58
// Types represents a set of result types.
69
// It's a bitset corresponding to the disjunction of types it represents.
@@ -50,5 +53,6 @@ func (r Types) String() string {
5053
}
5154
names = append(names, name)
5255
}
56+
sort.Strings(names)
5357
return strings.Join(names, "|")
5458
}

0 commit comments

Comments
 (0)