calling start on an started controller should be a no-op

andy-haynes · Nov 17, 2019 · 306aa94 · 306aa94
1 parent 30a5cfe
commit 306aa94
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/controller/controller.js b/src/controller/controller.js
@@ -1,5 +1,6 @@
 const Configure = require('@stamp/configure');
 const stampit = require('@stamp/it');
+const Promise = require('bluebird');
 const _ = require('lodash');
 
 const EventLogger = require('../composables/event_logger');
@@ -77,6 +78,10 @@ const Controller = stampit.compose(
       return this.pinController.disposeAll();
     },
     start() {
+      // already initialized, no-op
+      if (this.temperatureReadInterval) {
+        return Promise.resolve();
+      }
       return this.thermostat.initialize()
         .then(() => {
           this.temperatureReadInterval = setInterval(