Add tests for missing and undefined values

Author: camdecoster

test/jasmine/tests/lib_test.js

@@ -2706,13 +2706,24 @@ describe('Test lib.js:', function () {
         });
 
         // This test must come after the warning count since it will affect the count
-        it('replaces missing matches with fallback value', function () {
+        it('replaces template specifiers where the key is missing with the specifier', () => {
             expect(
                 Lib.hovertemplateString({
                     data: [{ group: 1 }],
                     fallback: 'Planet Express',
                     template: 'foo %{group} %{trace}'
                 })
+            ).toEqual('foo 1 %{trace}');
+        });
+
+        // This test must come after the warning count since it will affect the count
+        it('replaces template specifiers where the value is undefined with the fallback value', () => {
+            expect(
+                Lib.hovertemplateString({
+                    data: [{ group: 1, trace: undefined }],
+                    fallback: 'Planet Express',
+                    template: 'foo %{group} %{trace}'
+                })
             ).toEqual('foo 1 Planet Express');
         });
     });
@@ -2762,13 +2773,24 @@ describe('Test lib.js:', function () {
         });
 
         // This test must come after the warning count since it will affect the count
-        it('replaces missing matches with fallback value', function () {
+        it('replaces template specifiers where the key is missing with the specifier', () => {
             expect(
                 Lib.texttemplateString({
                     data: [{ group: 1 }],
                     fallback: 'Zoidberg',
                     template: 'foo %{group} %{trace}'
                 })
+            ).toEqual('foo 1 %{trace}');
+        });
+
+        // This test must come after the warning count since it will affect the count
+        it('replaces template specifiers where the value is undefined with the fallback value', () => {
+            expect(
+                Lib.texttemplateString({
+                    data: [{ group: 1, trace: undefined }],
+                    fallback: 'Zoidberg',
+                    template: 'foo %{group} %{trace}'
+                })
             ).toEqual('foo 1 Zoidberg');
         });
     });