search

processing / p5.js

p5.js is a client-side JS platform that empowers artists, designers, students, and anyone to learn to code and express themselves creatively on the web. It is based on the core principles of Processing. http://twitter.com/p5xjs —
http://p5js.org/
GNU Lesser General Public License v2.1
21.09k stars 3.22k forks source link

Fix broken test #7010

Closed davepagurek closed 1 month ago

davepagurek commented 1 month ago

Resolves https://github.com/processing/p5.js/issues/7008

Removes a focused test case, and fixes a broken test that got merged in because of that.

I looked into adding mocha.setup({ ui: 'tdd', forbidOnly: true }) into mocha_setup.js to see if we could catch these in CI, but I couldn't get it to actually fail on usage of .only locally. Any idea what I might be missing @limzykenneth?

limzykenneth commented 1 month ago

Not sure about whether mocha's forbidOnly works or not, I can test but I'll merge this one first to fix unit tests.