fixes bug in ParameterizedResourceFilterAttribute which disabled wrong tests (#349)

Author: adrianoc

Cecilifier.Core.Tests/Framework/Attributes/FilterByContextBase.cs

@@ -27,11 +27,11 @@ protected bool HasMatchingContext(Test test)
     protected bool DisableTestIfNotApplicable(ITest test, string testName)
     {
         if (_enabledTests.Contains(testName))
-            return true;
+            return false;
 
         ((Test) test).RunState = RunState.Ignored;
         test.Properties[PropertyNames.SkipReason].Add(IgnoreReason);
 
-        return false;
+        return true;
     }
 }

Cecilifier.Core.Tests/Framework/Attributes/ParameterizedResourceFilterAttribute.cs

@@ -1,10 +1,12 @@
 #nullable enable
+using System;
 using Cecilifier.Core.AST;
 using NUnit.Framework.Interfaces;
 using NUnit.Framework.Internal.Commands;
 
 namespace Cecilifier.Core.Tests.Framework.Attributes;
 
+[AttributeUsage(AttributeTargets.Method)]
 internal class ParameterizedResourceFilterAttribute<T> : FilterByContextBase<T>, IWrapTestMethod where T : IVisitorContext
 {
     public ParameterizedResourceFilterAttribute(params string[] enabled) : base(enabled)
@@ -15,7 +17,8 @@ public TestCommand Wrap(TestCommand command)
     {
         if (HasMatchingContext(command.Test) && command.Test.Arguments.Length > 0)
         {
-            DisableTestIfNotApplicable(command.Test, (string) command.Test.Arguments[0]!);
+            if(DisableTestIfNotApplicable(command.Test, (string) command.Test.Arguments[0]!))
+                return new SkipCommand(command.Test);
         }
         return command;
     }