From d399113e0c82b919177d37c96ed980596499dc71 Mon Sep 17 00:00:00 2001 From: Chris Lu Date: Wed, 28 Jan 2026 19:42:19 -0800 Subject: [PATCH] test: fix duplicate subtest names in permissions_test.go Rename duplicate 'combined * and ?' test cases to include singular/plural suffix for clarity and to support targeted test runs. --- weed/s3api/s3tables/permissions_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/weed/s3api/s3tables/permissions_test.go b/weed/s3api/s3tables/permissions_test.go index dc093d6ba..4652084f5 100644 --- a/weed/s3api/s3tables/permissions_test.go +++ b/weed/s3api/s3tables/permissions_test.go @@ -32,8 +32,8 @@ func TestMatchesActionPattern(t *testing.T) { {"question mark no match", "GetTable?", "GetTableXY", false}, // Combined wildcards - {"combined * and ?", "s3tables:Get?able*", "s3tables:GetTable", true}, - {"combined * and ?", "s3tables:Get?able*", "s3tables:GetTables", true}, + {"combined * and ? singular", "s3tables:Get?able*", "s3tables:GetTable", true}, + {"combined * and ? plural", "s3tables:Get?able*", "s3tables:GetTables", true}, {"combined no match - ? needs 1 char", "s3tables:Get?able*", "s3tables:Getable", false}, }