BUG FIX: Argument 'workers' of type character was silently accepted [#34]

Author: HenrikBengtsson

NEWS

@@ -1,12 +1,15 @@
 Package: future.batchtools
 ==========================
 
-Version: 0.7.2-9000 [2018-12-22]
+Version: 0.7.2-9000 [2019-01-22]
 
 BUG FIXES:
 
  * Argument 'workers' could not be a function.
- 
+
+ * Argument 'workers' of type character was silently accepted and effectively
+   interpreted as workers = length(workers).
+
 
 Version: 0.7.2 [2018-12-03]
 

R/BatchtoolsFuture-class.R

@@ -64,9 +64,8 @@ BatchtoolsFuture <- function(expr = NULL, envir = parent.frame(),
     stop_if_not(length(workers) >= 1)
     if (is.numeric(workers)) {
       stop_if_not(!anyNA(workers), all(workers >= 1))
-    } else if (is.character(workers)) {
     } else {
-      stop("Argument 'workers' should be either numeric or character: ",
+      stop("Argument 'workers' should be either a numeric or a function: ",
            mode(workers))
     }
   }