From 12102247930cbb62688249d347e4e4b65a297998 Mon Sep 17 00:00:00 2001
From: Maksim Stepanov <17935127+delatrie@users.noreply.github.com>
Date: Thu, 28 Sep 2023 02:24:52 +0700
Subject: [PATCH] Add a shorthand for testplan's IsMatch

---
 Allure.NUnit/Core/AllureNUnitHelper.cs        |  7 +---
 .../SelectiveRunTests/TestPlanTests.cs        | 36 +++++++++++++++++++
 Allure.Net.Commons/TestPlan/AllureTestPlan.cs | 10 ++++++
 3 files changed, 47 insertions(+), 6 deletions(-)

diff --git a/Allure.NUnit/Core/AllureNUnitHelper.cs b/Allure.NUnit/Core/AllureNUnitHelper.cs
index 7434ccc9..bdad89cb 100644
--- a/Allure.NUnit/Core/AllureNUnitHelper.cs
+++ b/Allure.NUnit/Core/AllureNUnitHelper.cs
@@ -191,12 +191,7 @@ void UpdateTestDataFromAllureAttributes(TestResult testResult)
         }
 
         static bool IsSelectedByTestPlan(TestResult testResult) =>
-            TestPlan.Value.IsMatch(
-                testResult.fullName,
-                AllureTestPlan.GetAllureId(
-                    testResult.labels ?? Enumerable.Empty<Label>()
-                )
-            );
+            TestPlan.Value.IsMatch(testResult);
 
         IEnumerable<AllureTestCaseAttribute> IterateAllAllureAttribites() =>
             this._test.Method
diff --git a/Allure.Net.Commons.Tests/SelectiveRunTests/TestPlanTests.cs b/Allure.Net.Commons.Tests/SelectiveRunTests/TestPlanTests.cs
index 4016b012..8aba0f13 100644
--- a/Allure.Net.Commons.Tests/SelectiveRunTests/TestPlanTests.cs
+++ b/Allure.Net.Commons.Tests/SelectiveRunTests/TestPlanTests.cs
@@ -187,6 +187,42 @@ bool expectedMatch
             );
         }
 
+        [Test]
+        public void TestResultMatchShorthandTest(
+            [Values(
+                "[]",
+                "[{\"id\": \"100\"}]",
+                "[{\"selector\": \"a\"}]",
+                "[{\"selector\": \"a\", \"id\": \"100\"}]",
+                "[{\"id\": \"100\"}, {\"id\": \"101\"}]",
+                "[{\"selector\": \"a\"}, {\"selector\": \"b\"}]",
+                "[{\"id\": \"100\"}, {\"selector\": \"a\"}]",
+                "[{\"selector\": \"a\", \"id\": \"100\"}, {\"selector\": \"b\", \"id\": \"101\"}]"
+            )] string testPlanItemsJson,
+            [Values("a", "b", "c")] string fullNameOfTestCase,
+            [Values("100", "101", "102")] string allureIdOfTestCase
+        )
+        {
+            var testResult = new TestResult
+            {
+                fullName = fullNameOfTestCase,
+                labels = allureIdOfTestCase is null ? new() : new()
+                {
+                    new() { name = "ALLURE_ID", value = allureIdOfTestCase }
+                }
+            };
+            var testPlan = AllureTestPlan.FromJson(
+                $"{{\"tests\": {testPlanItemsJson}}}"
+            );
+
+            Assert.That(
+                testPlan.IsMatch(testResult),
+                Is.EqualTo(
+                    testPlan.IsMatch(fullNameOfTestCase, allureIdOfTestCase)
+                )
+            );
+        }
+
         [TestCase(new string[] { }, null)]
         [TestCase(new[] { "ALLURE_ID", "100" }, "100")]
         [TestCase(new[] { "l1", "v1" }, null)]
diff --git a/Allure.Net.Commons/TestPlan/AllureTestPlan.cs b/Allure.Net.Commons/TestPlan/AllureTestPlan.cs
index 6cb67f29..2a9ab4b9 100644
--- a/Allure.Net.Commons/TestPlan/AllureTestPlan.cs
+++ b/Allure.Net.Commons/TestPlan/AllureTestPlan.cs
@@ -41,6 +41,16 @@ public bool IsMatch(string fullName, string? allureId) =>
             || this.IsFullNameMatch(fullName)
             || this.IsAllureIdMatch(allureId);
 
+    /// <summary>
+    /// A shorthand for <see cref="IsMatch(string, string?)"/> with the
+    /// fullName and the allure id taken from the provided test result.
+    /// </summary>
+    public bool IsMatch(TestResult testResult) =>
+        this.IsMatch(
+            testResult.fullName,
+            GetAllureId(testResult.labels)
+        );
+
     /// <summary>
     /// Creates the testplan from a JSON string.
     /// </summary>