Default callbacks to noop

Author: erbridge

lib/Model.js

@@ -1,7 +1,7 @@
 import { ReplicateError, ReplicateResponseError } from "./errors.js";
 import Prediction from "./Prediction.js";
 import ReplicateObject from "./ReplicateObject.js";
-import { sleep } from "./utils.js";
+import { noop, sleep } from "./utils.js";
 
 export default class Model extends ReplicateObject {
   static propertyMap = {
@@ -48,7 +48,7 @@ export default class Model extends ReplicateObject {
 
   async predict(
     input,
-    { onUpdate, onTemporaryError } = {},
+    { onUpdate = noop, onTemporaryError = noop } = {},
     {
       defaultPollingInterval = 500,
       backoffFn = (errorCount) => Math.pow(2, errorCount) * 100,
@@ -60,7 +60,7 @@ export default class Model extends ReplicateObject {
 
     let prediction = await this.createPrediction(input);
 
-    onUpdate && onUpdate(prediction);
+    onUpdate(prediction);
 
     let pollingInterval = defaultPollingInterval;
     let errorCount = 0;
@@ -72,7 +72,7 @@ export default class Model extends ReplicateObject {
       try {
         prediction = await this.client.prediction(prediction.id).load();
 
-        onUpdate && onUpdate(prediction);
+        onUpdate(prediction);
 
         errorCount = 0; // Reset because we've had a non-error response.
       } catch (err) {
@@ -89,7 +89,7 @@ export default class Model extends ReplicateObject {
 
         errorCount += 1;
 
-        onTemporaryError && onTemporaryError(err);
+        onTemporaryError(err);
 
         pollingInterval = backoffFn(errorCount);
       }

lib/utils.js

@@ -1,3 +1,5 @@
+export function noop() {}
+
 export function sleep(ms) {
   return new Promise((resolve) => {
     setTimeout(resolve, ms);