removed redundant tests and renamed previously created tests

Author: Said Shah

src/utilities/collection-utils.test.ts

@@ -4,28 +4,6 @@ import { ResultRecord } from "../view-models/result-record";
 
 describe("CollectionUtils", () => {
     describe("hasValues", () => {
-        test("given collections is null, it returns false", (): void => {
-            // Arrange
-            const collection = null;
-
-            // Act
-            const result = CollectionUtils.hasValues(collection);
-
-            // Assert
-            expect(result).toBeFalse();
-        });
-
-        test("given collections is undefined, it returns false", (): void => {
-            // Arrange
-            const collection = undefined;
-
-            // Act
-            const result = CollectionUtils.hasValues(collection);
-
-            // Assert
-            expect(result).toBeFalse();
-        });
-
         test("when collections param is an array and has elements, it returns true", (): void => {
             // Arrange
             const collection = [{}];
@@ -94,7 +72,7 @@ describe("CollectionUtils", () => {
             expect(result).toBeFalse();
         });
 
-        test("when collections param is undefined, it returns false", (): void => {
+        test("given collections is undefined, it returns false", (): void => {
             // Arrange & Act
             const result = CollectionUtils.hasValues(
                 (undefined as unknown) as any[]
@@ -104,7 +82,7 @@ describe("CollectionUtils", () => {
             expect(result).toBeFalse();
         });
 
-        test("when collections param is null, it returns false", (): void => {
+        test("given collections is null, it returns false", (): void => {
             // Arrange & Act
             const result = CollectionUtils.hasValues(
                 (null as unknown) as any[]
@@ -133,28 +111,6 @@ describe("CollectionUtils", () => {
     }); // end hasValues
 
     describe("isEmpty", () => {
-        test(`given collections is null, it returns true`, (): void => {
-            // Arrange
-            const collection = null;
-
-            // Act
-            const result = CollectionUtils.isEmpty(collection);
-
-            // Assert
-            expect(result).toBeTrue();
-        });
-
-        test(`given collections is undefined, it returns true`, (): void => {
-            // Arrange
-            const collection = undefined;
-
-            // Act
-            const result = CollectionUtils.isEmpty(collection);
-
-            // Assert
-            expect(result).toBeTrue();
-        });
-
         test(`when collections param is an array and has elements, it returns false`, (): void => {
             // Arrange
             const collection = [{}];
@@ -223,7 +179,7 @@ describe("CollectionUtils", () => {
             expect(result).toBeTrue();
         });
 
-        test("when collections param is undefined, it returns true", (): void => {
+        test("given collections is undefined, it returns true", (): void => {
             // Arrange & Act
             const result = CollectionUtils.isEmpty(
                 (undefined as unknown) as any[]
@@ -233,7 +189,7 @@ describe("CollectionUtils", () => {
             expect(result).toBeTrue();
         });
 
-        test("when collections param is null, it returns true", (): void => {
+        test("given collections is null, it returns true", (): void => {
             // Arrange & Act
             const result = CollectionUtils.isEmpty((null as unknown) as any[]);