From 41d9c8b5ecd0ea31cd12f8fce052b47abd341ff8 Mon Sep 17 00:00:00 2001 From: Dylan Guedes Date: Mon, 10 Feb 2025 12:27:25 -0300 Subject: [PATCH] fix: Make policy test non-flaky (#16158) --- pkg/validation/ingestion_policies_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/validation/ingestion_policies_test.go b/pkg/validation/ingestion_policies_test.go index 802a7a03eb..591624dfcd 100644 --- a/pkg/validation/ingestion_policies_test.go +++ b/pkg/validation/ingestion_policies_test.go @@ -31,7 +31,7 @@ func Test_PolicyStreamMapping_PolicyFor(t *testing.T) { "policy3": []*PriorityStream{ { Selector: `{qyx="qzx", qox="qox"}`, - Priority: 1, + Priority: 2, Matchers: []*labels.Matcher{ labels.MustNewMatcher(labels.MatchEqual, "qyx", "qzx"), labels.MustNewMatcher(labels.MatchEqual, "qox", "qox"), @@ -51,7 +51,7 @@ func Test_PolicyStreamMapping_PolicyFor(t *testing.T) { "policy5": []*PriorityStream{ { Selector: `{qab=~"qzx.*"}`, - Priority: 1, + Priority: 2, Matchers: []*labels.Matcher{ labels.MustNewMatcher(labels.MatchRegexp, "qab", "qzx.*"), }, @@ -94,7 +94,7 @@ func Test_PolicyStreamMapping_PolicyFor(t *testing.T) { require.Equal(t, "policy1", mapping.PolicyFor(labels.FromStrings("foo", "bar"))) // matches both policy2 and policy1 but policy1 has higher priority. require.Equal(t, "policy1", mapping.PolicyFor(labels.FromStrings("foo", "bar", "daz", "baz"))) - // matches policy3 and policy4 but policy3 appears first. + // matches policy3 and policy4 but policy3 has higher priority.. require.Equal(t, "policy3", mapping.PolicyFor(labels.FromStrings("qyx", "qzx", "qox", "qox"))) // matches no policy. require.Equal(t, "", mapping.PolicyFor(labels.FromStrings("foo", "fooz", "daz", "qux", "quux", "corge")))