[REFACTOR] [CodeFactor] Apply fixes

Author: Gonzalo Diaz

src/algorithm_exercises_csharp/hackerrank/interview_preparation_kit/dictionaries_and_hashmaps/FrequencyQueries.cs

@@ -7,9 +7,8 @@ namespace algorithm_exercises_csharp.hackerrank.interview_preparation_kit.dictio
 
 public class FrequencyQueries
 {
-  private FrequencyQueries()
-  {
-  }
+  [ExcludeFromCodeCoverage]
+  private FrequencyQueries() { }
 
   private static readonly long __INITIAL__ = 1L;
 
@@ -20,8 +19,6 @@ private FrequencyQueries()
   private static readonly int __NOT_FOUND__ = 0;
   private static readonly int __FOUND__ = 1;
 
-  private const string OPERATION_ERROR = "Operation %d not supported";
-
   readonly Dictionary<long, long> valueFreqs = [];
   readonly Dictionary<long, List<long>> freqDictionary = [];
   List<int> result = [];
@@ -145,8 +142,7 @@ public static List<int> freqQuery(List<List<int>> queries)
           fq.select(value);
           break;
         default:
-          throw new InvalidOperationException(
-             String.Format(OPERATION_ERROR, operation));
+          throw new InvalidOperationException($"Operation {operation} not supported");
       }
     }
 

src/algorithm_exercises_csharp_test/hackerrank/interview_preparation_kit/dictionaries_and_hashmaps/FrequencyQueries.Test.cs

@@ -45,7 +45,11 @@ public void testFrequencyQueries()
     foreach (FrequencyQueriesTestCase test in testCases)
     {
       solutionFound = FrequencyQueries.freqQuery(test.input);
-      CollectionAssert.AreEqual(test.expected, solutionFound);
+      CollectionAssert.AreEqual(
+        test.expected,
+        solutionFound,
+        $"FrequencyQueries.freqQuery({test.input}) answer must be: {test.expected}"
+      );
     }
   }
 
@@ -57,10 +61,11 @@ public void testFrequencyQueriesBigCases()
     foreach (FrequencyQueriesTestCase test in testCase6)
     {
       solutionFound = FrequencyQueries.freqQuery(test.input);
-      CollectionAssert.AreEqual(test.expected, solutionFound, String.Format("%s(%s) answer must be: %s" +
-              "FrequencyQueriesTest.freqQuery",
-              test.input.ToString(),
-              test.expected.ToString()));
+      CollectionAssert.AreEqual(
+        test.expected,
+        solutionFound,
+        $"FrequencyQueries.freqQuery({test.input}) answer must be: {test.expected}"
+      );
     }
   }
 
@@ -72,10 +77,11 @@ public void testFrequencyQueriesBorderCases()
     foreach (FrequencyQueriesTestCase test in testCaseBorderCases)
     {
       solutionFound = FrequencyQueries.freqQuery(test.input);
-      CollectionAssert.AreEqual(test.expected, solutionFound, String.Format("%s(%s) answer must be: %s" +
-              "FrequencyQueriesTest.freqQuery",
-              test.input.ToString(),
-              test.expected.ToString()));
+      CollectionAssert.AreEqual(
+        test.expected,
+        solutionFound,
+        $"FrequencyQueries.freqQuery({test.input}) answer must be: {test.expected}"
+      );
     }
   }