Merge pull request #4 from hlaueriksson/dencode

DenCode

Author: hlaueriksson

docs/dencode.png

@@ -11,7 +11,7 @@ public void GetDenCodeMethods()
         {
             var result = Constants.Methods.GetDenCodeMethods();
             result.Should().NotBeEmpty();
-            result.Should().HaveCount(67);
+            result.Should().HaveCount(68);
 
             var last = result.Last().Value;
             last.Should().BeEquivalentTo(new DenCodeMethod()
@@ -33,7 +33,7 @@ public void GetDenCodeLabels()
         {
             var result = Constants.Methods.GetDenCodeMethods().GetDenCodeLabels();
             result.Should().NotBeEmpty();
-            result.Should().HaveCount(104);
+            result.Should().HaveCount(106);
 
             var last = result.Last().Value;
             last.Should().BeEquivalentTo(new DenCodeMethod()
@@ -60,5 +60,33 @@ public void GetRequestType()
             var result = method.GetRequestType();
             result.Should().Be("hash");
         }
+
+        [TestMethod]
+        public void IsRoot()
+        {
+            var result = Constants.Methods.GetDenCodeMethods();
+            var first = result.First().Value;
+            first.IsRoot().Should().BeTrue();
+            result.Skip(1).Should().AllSatisfy(x => x.Value.IsRoot().Should().BeFalse());
+        }
+
+        [TestMethod]
+        public void IsBranch()
+        {
+            string[] branches = ["all.all", "string.all", "number.all", "date.all", "color.all", "cipher.all", "hash.all"];
+            var result = Constants.Methods.GetDenCodeMethods();
+
+            foreach (var method in result.Values)
+            {
+                if (branches.Contains(method.Key))
+                {
+                    method.IsBranch().Should().BeTrue();
+                }
+                else
+                {
+                    method.IsBranch().Should().BeFalse();
+                }
+            }
+        }
     }
 }

src/Community.PowerToys.Run.Plugin.DenCode.UnitTests/ExtensionsTests.cs

@@ -14,11 +14,23 @@ public class MainTests
         [TestInitialize]
         public void TestInitialize()
         {
-            var json = JsonDocument.Parse("{ \"encStrHex\": \"48656c6c6f2c20776f726c6421\", \"encStrURLEncoding\": \"Hello%2C%20world%21\" }");
+            var json = JsonDocument.Parse(
+                """
+                {
+                    "encStrHex": "48656c6c6f2c20776f726c6421",
+                    "encStrURLEncoding": "Hello%2C%20world%21",
+                    "encStrHTMLEscape": "Hello, world!",
+                    "encStrHTMLEscapeFully": "Hello, world!",
+                    "decStrHTMLEscape": "Hello, world!"
+                }
+                """
+            );
 
             var mock = new Mock<IDenCodeClient>();
             mock.Setup(x => x.DenCodeAsync("Hello, world!")).ReturnsAsync(AllDenCodeResponse());
             mock.Setup(x => x.DenCodeAsync(It.Is<DenCodeMethod>(x => x.Key == "string.hex"), "Hello, world!")).ReturnsAsync(HexDenCodeResponse());
+            mock.Setup(x => x.DenCodeAsync(It.Is<DenCodeMethod>(x => x.Key == "string.all"), "Hello, world!")).ReturnsAsync(HtmlEscapeDenCodeResponse());
+            mock.Setup(x => x.DenCodeAsync(It.Is<DenCodeMethod>(x => x.Key == "string.html-escape"), "Hello, world!")).ReturnsAsync(HtmlEscapeDenCodeResponse());
 
             _subject = new Main(mock.Object);
 
@@ -35,7 +47,17 @@ public void TestInitialize()
             {
                 response = new Dictionary<string, JsonElement>
                 {
-                    { "encStrHex", json.RootElement.GetProperty("encStrHex") }
+                    { "encStrHex", json.RootElement.GetProperty("encStrHex") },
+                }
+            };
+
+            DenCodeResponse HtmlEscapeDenCodeResponse() => new()
+            {
+                response = new Dictionary<string, JsonElement>
+                {
+                    { "encStrHTMLEscape", json.RootElement.GetProperty("encStrHTMLEscape") },
+                    { "encStrHTMLEscapeFully", json.RootElement.GetProperty("encStrHTMLEscapeFully") },
+                    { "decStrHTMLEscape", json.RootElement.GetProperty("decStrHTMLEscape") },
                 }
             };
         }
@@ -54,7 +76,7 @@ public void Query_without_delayedExecution_should_return_empty_result()
         public void Query_with_empty_args_should_return_all_DenCodeMethods()
         {
             _subject.Query(new(""), true)
-                .Should().HaveCount(67);
+                .Should().HaveCount(68);
         }
 
         [TestMethod]
@@ -78,6 +100,20 @@ public void Query_with_value_args_should_return_matching_DenCodeMethods()
                 .Should().HaveCount(2);
         }
 
+        [TestMethod]
+        public void Query_branch_method_removes_unchanged_results()
+        {
+            _subject.Query(new("string.all Hello, world!"), true)
+                .Should().HaveCount(1);
+        }
+
+        [TestMethod]
+        public void Query_leaf_method_keeps_unchanged_results()
+        {
+            _subject.Query(new("string.html-escape Hello, world!"), true)
+                .Should().HaveCount(3);
+        }
+
         [TestMethod]
         public void LoadContextMenus_with_no_ContextData_should_return_empty_result()
         {

src/Community.PowerToys.Run.Plugin.DenCode.UnitTests/MainTests.cs

@@ -92,7 +92,7 @@ public string GetUrl(DenCodeMethod method)
         {
             ArgumentNullException.ThrowIfNull(method);
 
-            if (method.Method == Constants.AllMethod)
+            if (method.IsRoot())
             {
                 return "https://dencode.com";
             }

src/Community.PowerToys.Run.Plugin.DenCode/DenCodeClient.cs

@@ -29,5 +29,15 @@ public static string GetRequestType(this DenCodeMethod method)
         {
             return method.Key.Substring(0, method.Key.IndexOf('.', StringComparison.Ordinal));
         }
+
+        public static bool IsRoot(this DenCodeMethod method)
+        {
+            return method.Key.Equals("all.all", StringComparison.Ordinal);
+        }
+
+        public static bool IsBranch(this DenCodeMethod method)
+        {
+            return method.Key.Contains(".all", StringComparison.Ordinal);
+        }
     }
 }

src/Community.PowerToys.Run.Plugin.DenCode/Extensions.cs

@@ -114,7 +114,7 @@ public List<Result> Query(Query query, bool delayedExecution)
 #pragma warning restore VSTHRD104 // Offer async methods
 #pragma warning restore VSTHRD002 // Avoid problematic synchronous waits
 
-                return GetResultsFromDenCodeResponse(response, value);
+                return GetResultsFromDenCodeResponse(response, value, method.IsBranch());
             }
             else
             {
@@ -137,7 +137,7 @@ public List<Result> Query(Query query, bool delayedExecution)
                 ContextData = method,
             };
 
-            List<Result> GetResultsFromDenCodeResponse(DenCodeResponse? response, string value)
+            List<Result> GetResultsFromDenCodeResponse(DenCodeResponse? response, string value, bool removeUnchangedResults = true)
             {
                 var results = new List<Result>();
 
@@ -155,7 +155,7 @@ List<Result> GetResultsFromDenCodeResponse(DenCodeResponse? response, string val
                         continue;
                     }
 
-                    if (result == value)
+                    if (removeUnchangedResults && result == value)
                     {
                         continue;
                     }

src/Community.PowerToys.Run.Plugin.DenCode/Images/dencode.dark.png

@@ -5,8 +5,6 @@ namespace Community.PowerToys.Run.Plugin.DenCode.Models
     /// </summary>
     public static partial class Constants
     {
-        public const string AllMethod = "ALL";
-
         public const string AllRequest =
 """
 {