fix logic

MetaMask · Dec 13, 2024 · 0e85a3d · 0e85a3d
1 parent e6cb6aa
commit 0e85a3d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/packages/snaps-rpc-methods/jest.config.js b/packages/snaps-rpc-methods/jest.config.js
@@ -10,10 +10,10 @@ module.exports = deepmerge(baseConfig, {
   ],
   coverageThreshold: {
     global: {
-      branches: 92.8,
-      functions: 97.35,
-      lines: 97.35,
-      statements: 96.93,
+      branches: 92.83,
+      functions: 97.36,
+      lines: 97.92,
+      statements: 97.47,
     },
   },
 });
diff --git a/packages/snaps-rpc-methods/src/permitted/scheduleBackgroundEvent.ts b/packages/snaps-rpc-methods/src/permitted/scheduleBackgroundEvent.ts
@@ -58,7 +58,9 @@ const ScheduleBackgroundEventsParametersStruct = object({
   date: refine(string(), 'date', (val) => {
     const date = DateTime.fromISO(val);
     if (date.isValid) {
-      if (!val.endsWith('Z') || date.offset === 0) {
+      // luxon doesn't have a reliable way to check if timezone info was not provided
+      const count = val.split('').filter((char) => char === '-').length;
+      if (count !== 3 && !val.includes('+') && !val.endsWith('Z')) {
         return 'ISO8601 string must have timezone information';
       }
       return true;