fix(build.py): move submodule checkout to the commonjs testing script

Author: Xmader

build.py

@@ -52,7 +52,6 @@ def copy_artifacts():
         execute("cp ./_spidermonkey_install/lib/libmozjs* ./python/pythonmonkey/", cwd=TOP_DIR)
 
 def build():
-    execute("git submodule update --init --recursive || echo 'Skipping. We are installing from an sdist tarball.'", cwd=TOP_DIR)
     ensure_spidermonkey()
     run_cmake_build()
     copy_artifacts()

tests/js/commonjs-modules.bash

@@ -11,7 +11,11 @@ panic()
   echo "$*" >&2
   exit 2
 }
-cd `dirname "$0"`/../commonjs-official/tests/modules/1.0 || panic "could not change to test directory"
+
+cd `dirname "$0"`
+git submodule update --init --recursive || panic "could not checkout the required git submodule"
+
+cd ../commonjs-official/tests/modules/1.0 || panic "could not change to test directory"
 
 runTest()
 {