@wmfs/tymly-schedule-plugin 1.6.0 → 1.6.1

package/.version

@@ -1 +1 @@
-VERSION=1.6.0
+VERSION=1.6.1

package/CHANGELOG.md

@@ -1,3 +1,28 @@
+## [1.6.1](https://github.com/wmfs/tymly-schedule-plugin/compare/v1.6.0...v1.6.1) (2023-06-07)
+
+
+### 🐛 Bug Fixes
+
+* better utc handling with consequent test updates ([d15c042](https://github.com/wmfs/tymly-schedule-plugin/commit/d15c042987ebb61fb382eedb057413d718e825ea))
+
+
+### 🛠 Builds
+
+* **deps-dev:** update dependency [@wmfs](https://github.com/wmfs)/tymly-pg-plugin to v1.283.0 ([3322021](https://github.com/wmfs/tymly-schedule-plugin/commit/332202187f248defb6a68403dec428e9176efdbb))
+* **deps-dev:** update dependency semantic-release to v21.0.3 ([b2da7ba](https://github.com/wmfs/tymly-schedule-plugin/commit/b2da7babde8a45e0d899870ee5b4021c8a662d3b))
+* **deps-dev:** update dependency standard to v17.1.0 ([f7def04](https://github.com/wmfs/tymly-schedule-plugin/commit/f7def04f6abdb127fb0aa6ad02cc7b069480dee9))
+
+
+### 📦 Code Refactoring
+
+* renamed vars to improve clarity ([4ed331c](https://github.com/wmfs/tymly-schedule-plugin/commit/4ed331c0edbbd3740e72b18961122d2710ce4e39))
+
+
+### 🚨 Tests
+
+* minor change from let to const for standard ([e81e5de](https://github.com/wmfs/tymly-schedule-plugin/commit/e81e5dec130e9d35ad9460a316a3bce42ea9fcfc))
+* restored tests that were inadvertantly removed ([843e6d5](https://github.com/wmfs/tymly-schedule-plugin/commit/843e6d5e0ddd9e5046e39df8ff068e489e7fdfd7))
+
 # [1.6.0](https://github.com/wmfs/tymly-schedule-plugin/compare/v1.5.0...v1.6.0) (2023-06-06)
 
 

package/lib/components/services/schedule/index.js

@@ -1,8 +1,8 @@
 const INSTIGATING_CLIENT_DOMAIN = 'tymly-schedule-plugin'
 
 const nodeSchedule = require('node-schedule')
-const ruleToDatetime = require('./utils/rule-to-datetime')
-const datetimeToRule = require('./utils/datetime-to-rule')
+const ruleToUTCDateTime = require('./utils/rule-to-datetime')
+const localDateTimeToUTCRule = require('./utils/datetime-to-rule')
 const { v4: uuidv4 } = require('uuid')
 
 class CronService {
@@ -45,7 +45,7 @@ class CronService {
     if (scheduleType === 'interval') {
       jobOptions.rule = interval
     } else if (scheduleType === 'datetime') {
-      const rule = datetimeToRule(datetime)
+      const rule = localDateTimeToUTCRule(datetime)
 
       Object
         .entries(rule)
@@ -58,7 +58,7 @@ class CronService {
     task.job = nodeSchedule.scheduleJob(
       jobOptions,
       () => {
-        console.log(`Running scheduled state machine ${stateMachineName}`)
+        console.log(`Running scheduled state machine ${stateMachineName} at ${new Date()}`)
         const executionOptions = {
           instigatingClient: {
             appName: 'tymly',
@@ -145,7 +145,7 @@ class CronService {
         config.datetime = null
       } else if (typeof config.rule === 'object') {
         config.scheduleType = 'datetime'
-        config.datetime = ruleToDatetime(config.rule)
+        config.datetime = ruleToUTCDateTime(config.rule)
         config.interval = null
       }
 
@@ -219,7 +219,7 @@ class CronService {
       this.scheduledTasks[key].scheduleType = 'interval'
     } else if (typeof rule === 'object') {
       this.scheduledTasks[key].interval = null
-      this.scheduledTasks[key].datetime = ruleToDatetime(rule)
+      this.scheduledTasks[key].datetime = ruleToUTCDateTime(rule)
       this.scheduledTasks[key].scheduleType = 'datetime'
     }
 

package/lib/components/services/schedule/utils/datetime-to-rule.js

@@ -5,6 +5,9 @@ module.exports = function (datetime) {
     datetime = new Date()
   }
 
+  // convert local time back to UTC
+  datetime = moment(datetime).utc()
+
   return {
     year: moment(datetime).year(),
     month: moment(datetime).month(),

package/lib/components/services/schedule/utils/rule-to-datetime.js

@@ -12,7 +12,6 @@ module.exports = function (rule = {}) {
   )
   // note that rule does not specify a timezone, so we have to assume that
   // the timestamp provided is a local timestamp - as such, we have to convert
-  // to UTC.  note that in British Summer Time (for example, on the 5th June),
-  // now.getTimezoneOffset() will return -60.
-  return moment(date).add(now.getTimezoneOffset(), 'minutes').toDate()
+  // to UTC
+  return moment(date).utc().toDate()
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wmfs/tymly-schedule-plugin",
-  "version": "1.6.0",
+  "version": "1.6.1",
   "description": "Provides state machine scheduling for the Tymly framework",
   "homepage": "https://github.com/wmfs/tymly-schedule-plugin#readme",
   "author": "West Midlands Fire Service",
@@ -31,7 +31,7 @@
     "@semantic-release/exec": "6.0.3",
     "@semantic-release/git": "10.0.1",
     "@wmfs/tymly": "1.242.0",
-    "@wmfs/tymly-pg-plugin": "1.282.0",
+    "@wmfs/tymly-pg-plugin": "1.283.0",
     "@wmfs/tymly-test-helpers": "1.22.2",
     "chai": "4.3.7",
     "codecov": "3.8.3",
@@ -40,8 +40,8 @@
     "dirty-chai": "2.0.1",
     "mocha": "10.2.0",
     "nyc": "15.1.0",
-    "semantic-release": "21.0.0",
-    "standard": "17.0.0"
+    "semantic-release": "21.0.3",
+    "standard": "17.1.0"
   },
   "scripts": {
     "lint": "standard",

package/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "schedule-plugin",
-  "version": "1.6.0",
+  "version": "1.6.1",
   "label": "Tymly Schedule Plugin",
   "author": "WMFS",
   "organisation": "West Midlands Fire Service",

package/test/rule-to-datetime-tests.js

@@ -8,10 +8,9 @@ const datetimeToRule = require('../lib/components/services/schedule/utils/dateti
 
 describe('Rule to datetime tests', function () {
   it('No options, should default to now (UTC)', () => {
-    let now = new Date()
-    now = moment(now).add(now.getTimezoneOffset(), 'minutes').toDate()
-    const datetime = ruleToDatetime()
+    const now = moment().utc().toDate()
 
+    const datetime = ruleToDatetime()
     expect(datetime.getFullYear()).to.eql(now.getFullYear())
     expect(datetime.getMonth()).to.eql(now.getMonth())
     expect(datetime.getDate()).to.eql(now.getDate())
@@ -22,7 +21,7 @@ describe('Rule to datetime tests', function () {
 
   it('Year only, the rest defaults to now', () => {
     let now = new Date()
-    now = moment(now).add(now.getTimezoneOffset(), 'minutes').toDate()
+    now = moment(now).utc().toDate()
     const year = now.getFullYear() + 2
 
     const datetime = ruleToDatetime({ year })
@@ -48,7 +47,7 @@ describe('Rule to datetime tests', function () {
 
     const datetime = ruleToDatetime(rule)
 
-    const utcTimestamp = moment(now).add(now.getTimezoneOffset(), 'minutes').toDate()
+    const utcTimestamp = moment(now).utc().toDate()
 
     expect(datetime.getFullYear()).to.eql(utcTimestamp.getFullYear())
     expect(datetime.getMonth()).to.eql(utcTimestamp.getMonth())
@@ -71,7 +70,7 @@ describe('Rule to datetime tests', function () {
 
     const datetime = ruleToDatetime(rule)
 
-    const utcTimestamp = moment(now).add(now.getTimezoneOffset(), 'minutes').toDate()
+    const utcTimestamp = moment(now).utc().toDate()
 
     expect(datetime.getFullYear()).to.eql(utcTimestamp.getFullYear())
     expect(datetime.getMonth()).to.eql(utcTimestamp.getMonth())
@@ -83,8 +82,8 @@ describe('Rule to datetime tests', function () {
 })
 
 describe('Datetime to rule tests', function () {
-  it('No options, should default to now', () => {
-    const now = moment()
+  it('No options, should default to now (UTC)', () => {
+    const now = moment().utc()
     const rule = datetimeToRule()
 
     expect(rule.year).to.eql(now.year())
@@ -95,47 +94,35 @@ describe('Datetime to rule tests', function () {
     expect(rule.second).to.eql(now.second())
   })
 
-  it('Specific year, the rest should default to now', () => {
-    let datetime = new Date()
-    const year = datetime.getFullYear() + 2
-    datetime.setFullYear(year)
-
-    const rule = datetimeToRule(datetime)
-    datetime = moment(datetime)
-
-    expect(rule.year).to.eql(year)
-    expect(rule.month).to.eql(datetime.month())
-    expect(rule.date).to.eql(datetime.date())
-    expect(rule.hour).to.eql(datetime.hour())
-    expect(rule.minute).to.eql(datetime.minute())
-    expect(rule.second).to.eql(datetime.second())
-  })
-
   it('DST test, should change time by +1 hour', () => {
-    const currentDateTime = new Date('Sunday March 27 2022 00:59:00')
+    const lastMinuteOfGMT = new Date('Sunday March 27 2022 00:59:00')
+    console.log(`last minute of GMT: ${lastMinuteOfGMT}`)
 
-    // 2022-03-27T02:00:00+01:00
-    const startDSTDateTime = moment(currentDateTime).add(1, 'minutes')
+    let startOfBSTDateTime = moment(lastMinuteOfGMT).add(1, 'minutes').toDate()
+    console.log(`start of BST: ${startOfBSTDateTime}`)
 
     // adding 1 min and setting as Date object
-    const rule = datetimeToRule(new Date(currentDateTime.getTime() + (60 * 1000)))
+    const rule = datetimeToRule(new Date(lastMinuteOfGMT.getTime() + (60 * 1000)))
 
-    expect(rule.date).to.eql(startDSTDateTime.date())
-    expect(rule.hour).to.eql(startDSTDateTime.hour())
-    expect(rule.minute).to.eql(startDSTDateTime.minute())
+    startOfBSTDateTime = moment(startOfBSTDateTime).utc()
+    expect(rule.date).to.eql(startOfBSTDateTime.date())
+    expect(rule.hour).to.eql(startOfBSTDateTime.hour())
+    expect(rule.minute).to.eql(startOfBSTDateTime.minute())
   })
 
   it('DST test, should change time by -1 hour', () => {
-    const currentDateTime = new Date('Sunday October 30 2022 01:59:00')
+    const lastMinuteOfBST = new Date('Sunday October 30 2022 01:59:00')
+    console.log(`last minute of BST: ${lastMinuteOfBST}`)
 
-    // 2022-10-30T01:00:00+00:00
-    const endDSTDateTime = moment(currentDateTime).add(1, 'minutes')
+    let startOfGMTDateTime = moment(lastMinuteOfBST).add(1, 'minutes').toDate()
+    console.log(`start of GMT: ${startOfGMTDateTime}`)
 
     // adding 1 min and setting as Date object
-    const rule = datetimeToRule(new Date(currentDateTime.getTime() + (60 * 1000)))
+    const rule = datetimeToRule(new Date(lastMinuteOfBST.getTime() + (60 * 1000)))
 
-    expect(rule.date).to.eql(endDSTDateTime.date())
-    expect(rule.hour).to.eql(endDSTDateTime.hour())
-    expect(rule.minute).to.eql(endDSTDateTime.minute())
+    startOfGMTDateTime = moment(startOfGMTDateTime).utc()
+    expect(rule.date).to.eql(startOfGMTDateTime.date())
+    expect(rule.hour).to.eql(startOfGMTDateTime.hour())
+    expect(rule.minute).to.eql(startOfGMTDateTime.minute())
   })
 })

package/test/schedule-tests.js

@@ -3,6 +3,7 @@
 const expect = require('chai').expect
 const tymly = require('@wmfs/tymly')
 const path = require('path')
+const moment = require('moment')
 
 const scheduleKey = 'tymlyTest_twoSecondCatUpdates'
 
@@ -100,8 +101,8 @@ describe('Tymly schedule tests', function () {
   })
 
   it('update task to specific date/time', async () => {
-    const datetime = new Date()
-    datetime.setSeconds(datetime.getSeconds() + 10)
+    const datetime = moment().utc().add(10, 'seconds').toDate()
+    console.log(`scheduled: ${datetime}`)
 
     await scheduleService.stopAndUpdateTaskSchedule(
       scheduleKey,