Make DailyLearning.has also be case-insensitive

hebcal · Jan 12, 2025 · 770fa2b · 770fa2b
1 parent d7d629d
commit 770fa2b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@hebcal/core",
-  "version": "5.8.12",
+  "version": "5.8.13",
   "author": "Michael J. Radwin (https://github.com/mjradwin)",
   "contributors": [
     "Eyal Schachter (https://github.com/Scimonster)",

diff --git a/src/DailyLearning.ts b/src/DailyLearning.ts
@@ -41,7 +41,7 @@ export class DailyLearning {
    * @param name case insensitive
    */
   static has(name: string): boolean {
-    return cals.has(name);
+    return cals.has(name.toLowerCase());
   }
 
   /** Returns the names of all calendars registered */

diff --git a/test/DailyLearning.spec.ts b/test/DailyLearning.spec.ts
@@ -6,7 +6,10 @@ test('DailyLearning', () => {
   expect(DailyLearning.getCalendars()).toEqual([]);
   const dummy = () => {return null};
   const hd = new HDate();
+  expect(DailyLearning.has('Foo')).toBe(false);
   DailyLearning.addCalendar('Foo', dummy);
+  expect(DailyLearning.has('Foo')).toBe(true);
+  expect(DailyLearning.has('foo')).toBe(true);
   DailyLearning.addCalendar('Bar', dummy);
   expect(DailyLearning.getCalendars()).toEqual(['foo', 'bar']);
   const dummy2 = () => {return {bogus: true}};