feat(ui-scripts,ui-test-utils): move jsdom to ui-scripts

TEST PLAN:

yarn test:mocha should pass

Change-Id: Ib658810e7d251d5f5e28c0c81afd11929a2732e1
Reviewed-on: https://gerrit.instructure.com/184271
Tested-by: Jenkins
Reviewed-by: Chris Hart <chart@instructure.com>
Product-Review: Chris Hart <chart@instructure.com>
QA-Review: Jennifer Stern <jstern@instructure.com>

Author: junyper

packages/ui-scripts/lib/index.js

@@ -29,6 +29,7 @@ require('dotenv').config()
 
 const commands = [
   '--publish',
+  '--release',
   '--fix-publish',
   '--post-publish',
   '--bump',
@@ -56,7 +57,7 @@ if (process.argv.includes('--help')) {
   require('./post-publish')
 } else if (process.argv.includes('--bump')) {
   require('./bump')
-} else if (process.argv.includes('--publish')) {
+} else if (process.argv.includes('--publish') || process.argv.includes('--release')) {
   require('./publish')
 } else if (process.argv.includes('--fix-publish')) {
   require('./fix-publish')
@@ -87,4 +88,5 @@ if (process.argv.includes('--help')) {
 } else {
   error('[ui-scripts]: Invalid command!')
   listCommands()
+  process.exit(1)
 }

packages/ui-scripts/lib/test/mocha.js

@@ -32,6 +32,7 @@ let args = [
   '--colors',
   '--require', '@babel/register',
   '--require', '@babel/polyfill',
+  '--require', 'jsdom-global/register',
   '--exit'
 ]
 

packages/ui-scripts/package.json

@@ -35,6 +35,7 @@
     "gh-pages": "^2.0.1",
     "http-server": "^0.11.1",
     "jira-client": "^6.6.0",
+    "jsdom-global": "^3.0.2",
     "karma-cli": "^2.0.0",
     "lerna": "^3",
     "mocha": "^6.0.2",
@@ -47,6 +48,7 @@
     "webpack-dev-server": "^3.2.1"
   },
   "peerDependencies": {
+    "jsdom": ">=10.0.0",
     "react": "^0.14.9 || ^15 || ^16",
     "webpack": "^4"
   },

packages/ui-test-utils/package.json

@@ -33,7 +33,6 @@
     "chai-string": "^1.5.0",
     "dirty-chai": "^2.0.1",
     "fetch-mock": "^7.3.0",
-    "jsdom-global": "^3.0.2",
     "keycode": "^2",
     "pretty-format": "^24.0.0",
     "sinon": "^7.2.3",
@@ -42,7 +41,6 @@
     "wait-for-expect": "^1.1.0"
   },
   "peerDependencies": {
-    "jsdom": ">=10.0.0",
     "react": "^0.14.9 || ^15.0.0 || ^16.0.0",
     "react-dom": "^0.14.9 || ^15.0.0 || ^16.0.0"
   },

packages/ui-test-utils/src/utils/globals.js

@@ -21,7 +21,6 @@
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
  * SOFTWARE.
  */
-import jsdom from 'jsdom-global'
 import MutationObserverShim from '@sheerun/mutationobserver-shim'
 
 const noop = () => {}
@@ -30,7 +29,7 @@ let cleanupGlobals = noop
 
 const initGlobals = () => {
   if (typeof document === 'undefined') {
-    cleanupGlobals = jsdom()
+    console.error(`[ui-test-utils] requires access to the DOM API!`)
   }
   // jsdom doesn't provide Date on the window object and we need it for wait-for-expect
   global.window.Date = (global.window.Date && global.window.Date.now) ? global.window.Date : global.Date