mojo: Add python bindings tests to mojob.sh

mojo/tools/mojob.sh

Index: mojo/tools/mojob.sh
diff --git a/mojo/tools/mojob.sh b/mojo/tools/mojob.sh
index e0804d1ef656f98b6bd27c800c66d5a204c0f763..1d2a86752b456ffd39560aa4e10cc48cbc9776d0 100755
--- a/mojo/tools/mojob.sh
+++ b/mojo/tools/mojob.sh
@@ -19,7 +19,7 @@ command should be one of:
   build - Build.
   test - Run unit tests (does not build).
   perftest - Run perf tests (does not build).
-  pytest - Run Python unit tests.
+  pytest - Run Python unit tests (does not build).
   gyp - Run gyp for mojo (does not sync).
   gypall - Run gyp for all of chromium (does not sync).
   sync - Sync using gclient (does not run gyp).
@@ -67,6 +67,10 @@ do_perftests() {
 
 do_pytests() {
   python mojo/tools/run_mojo_python_tests.py || exit 1
+  if [ "$COMPONENT" = "static" ]; then

[viettrungluu, 2014/10/08 16:06:32]

This is rather dubious, since the component flags typically only affect
gyp/gypall, so one wouldn't usually specify it when running tests.

[qsr, 2014/10/08 16:16:07]

 Unfortunately, I do not have a good solution for this :( The generated python
modules do not work in component builds...

 Do you have a suggestion?

+    python mojo/tools/run_mojo_python_bindings_tests.py \
+        "--build-dir=out/$1" || exit 1

[viettrungluu, 2014/10/08 16:06:33]

And if it runs things out of a particular build directory, it should tell you
which one.

[qsr, 2014/10/08 16:16:08]

Done.

+  fi
 }
 
 do_gyp() {
@@ -180,7 +184,8 @@ for arg in "$@"; do
       should_do_Release && do_perftests Release
       ;;
     pytest)
-      do_pytests
+      should_do_Debug && do_pytests Debug
+      should_do_Release && do_pytests Release
       ;;
     gyp)
       set_goma_dir_if_necessary
@@ -236,3 +241,5 @@ for arg in "$@"; do
       ;;
   esac
 done
+
+exit 0

[viettrungluu, 2014/10/08 16:06:32]

Is there a reason for adding this?

[qsr, 2014/10/08 16:16:08]

 The script exit with 1 when called for debug.

 My guess is that it doesn't like to have the last called instruction be a false
|| 

[viettrungluu, 2014/10/09 22:49:52]

Sorry, what was the case (e.g., command-line) when it exited with 1?
But which one would that be (given that it's always "|| exit 1"?