forcing textWidth input to string closes #3851

Author: Lauren McCarthy

src/typography/attributes.js

@@ -215,9 +215,10 @@ p5.prototype.textStyle = function(theStyle) {
  *Letter P and p5.js are displayed with vertical lines at end. P is wide
  *
  */
-p5.prototype.textWidth = function(theText) {
+p5.prototype.textWidth = function() {
+  arguments[0] += '';
   p5._validateParameters('textWidth', arguments);
-  if (theText.length === 0) {
+  if (arguments[0].length === 0) {
     return 0;
   }
   return this._renderer.textWidth.apply(this._renderer, arguments);

test/unit/typography/attributes.js

@@ -95,14 +95,8 @@ suite('Typography Attributes', function() {
       assert.isNumber(myp5.textWidth('p5.js'));
     });
     // Either should not throw error
-    test('wrong param type at #0.', function() {
-      assert.validationError(function() {
-        myp5.textWidth(9);
-      });
-    });
-    // Or should return NaN
-    test.skip('should return NaN for wrong param.', function() {
-      assert.isNumber(myp5.textWidth(100));
+    test('should return a number for number input', function() {
+      assert.isNumber(myp5.textWidth('p5.js'));
     });
   });