Correct name casing

Author: millerds

test/end-to-end/src/test-helpers.ts

@@ -17,23 +17,23 @@ export async function closeWorkbook(): Promise<void> {
 
 export type TestResult = {
   expectedValue: any;
-  Name: string;
-  Value: any;
+  name: string;
+  value: any;
 };
 
-export function addTestResult(testValues: any[], Name: string, Value: any, expectedValue: any) {
+export function addTestResult(testValues: any[], name: string, value: any, expectedValue: any) {
   const data: TestResult = {
     expectedValue,
-    Name,
-    Value,
+    name,
+    value,
   };
   testValues.push(data);
 }
 
 export function addErrorResult(testValues: any[], errorMessage: string) {
   testValues.push({
-    Name: "test-error",
-    Value: errorMessage,
+    name: "test-error",
+    value: errorMessage,
     expectedValue: "no-error",
   });
 }

test/end-to-end/src/test-taskpane.ts

@@ -65,10 +65,10 @@ export async function readCFData(cfName: string, readCount: number): Promise<voi
   });
 }
 
-function addTestResult(Name: string, Value: any) {
+function addTestResult(name: string, value: any) {
   const data = {
-    Name,
-    Value,
+    name,
+    value,
   };
   testValues.push(data);
 }

test/end-to-end/tests.ts

@@ -64,48 +64,48 @@ describe("Test Excel Custom Functions", function () {
           clearTimeout(timeoutId);
         }
 
-        const errorResult = testValues.find((value: any) => value.Name === "test-error");
+        const errorResult = testValues.find((value: any) => value.name === "test-error");
         if (errorResult) {
-          assert.fail(`[${host}] Taskpane reported error: ${errorResult.Value}`);
+          assert.fail(`[${host}] Taskpane reported error: ${errorResult.value}`);
         }
 
         // Expecting six result values + user agent
-        testValues = testValues.filter((value: any) => value.Name !== "test-error");
+        testValues = testValues.filter((value: any) => value.name !== "test-error");
         assert.strictEqual(testValues.length > 0, true, `No test results received from ${host} add-in`);
-        console.log(`User Agent: ${testValues[0].Value}`);
+        console.log(`User Agent: ${testValues[0].value}`);
         assert.strictEqual(testValues.length, 7);
       });
       it("ADD function should return expected value", async function () {
-        assert.strictEqual(testJsonData.functions.ADD.result, testValues[1].Value);
+        assert.strictEqual(testJsonData.functions.ADD.result, testValues[1].value);
       });
       it("CLOCK function should return expected value", async function () {
         // Check that captured values are different to ensure the function is streaming
-        assert.notStrictEqual(testValues[2].Value, testValues[3].Value);
+        assert.notStrictEqual(testValues[2].value, testValues[3].value);
         // Check if the returned string contains 'AM', 'PM', or 'GMT', indicating it's a time-stamp
         assert.strictEqual(
-          testValues[2].Value.includes(testJsonData.functions.CLOCK.result.amString) ||
-          testValues[2].Value.includes(testJsonData.functions.CLOCK.result.pmString) ||
-          testValues[2].Value.includes(testJsonData.functions.CLOCK.result.timeZoneString),
+          testValues[2].value.includes(testJsonData.functions.CLOCK.result.amString) ||
+          testValues[2].value.includes(testJsonData.functions.CLOCK.result.pmString) ||
+          testValues[2].value.includes(testJsonData.functions.CLOCK.result.timeZoneString),
           true,
-          "Found timestamp indicator string in first value '" + testValues[2].Value + "'"
+          "Found timestamp indicator string in first value '" + testValues[2].value + "'"
         );
         assert.strictEqual(
-          testValues[3].Value.includes(testJsonData.functions.CLOCK.result.amString) ||
-          testValues[3].Value.includes(testJsonData.functions.CLOCK.result.pmString) ||
-          testValues[3].Value.includes(testJsonData.functions.CLOCK.result.timeZoneString),
+          testValues[3].value.includes(testJsonData.functions.CLOCK.result.amString) ||
+          testValues[3].value.includes(testJsonData.functions.CLOCK.result.pmString) ||
+          testValues[3].value.includes(testJsonData.functions.CLOCK.result.timeZoneString),
           true,
-          "Found timestamp indicator string in second value '" + testValues[3].Value + "'"
+          "Found timestamp indicator string in second value '" + testValues[3].value + "'"
         );
       });
       it("INCREMENT function should return expected value", async function () {
         // Check that captured values are different to ensure the function is streaming
-        assert.notStrictEqual(testValues[3].Value, testValues[4].Value);
+        assert.notStrictEqual(testValues[3].value, testValues[4].value);
         // Check to see that both captured streaming values are divisible by 4
-        assert.strictEqual(0, testValues[4].Value % testJsonData.functions.INCREMENT.result);
-        assert.strictEqual(0, testValues[5].Value % testJsonData.functions.INCREMENT.result);
+        assert.strictEqual(0, testValues[4].value % testJsonData.functions.INCREMENT.result);
+        assert.strictEqual(0, testValues[5].value % testJsonData.functions.INCREMENT.result);
       });
       it("LOG function should return expected value", async function () {
-        assert.strictEqual(testJsonData.functions.LOG.result, testValues[6].Value);
+        assert.strictEqual(testJsonData.functions.LOG.result, testValues[6].value);
       });
     });
     after("Teardown test environment", async function () {