Create the profile directories beforehand for unit test.

diff --git a/test/unit/Makefile b/test/unit/Makefile
index d2735b16a692cf297a66f972f44166cfb87b30f9..e7f157cd536b9f8d526ddc60fcec2e4b6115edb5 100644 (file)
--- a/test/unit/Makefile
+++ b/test/unit/Makefile
@@ -1,5 +1,5 @@
 # Create temporary profile directory name.
-TEMP_PROFILE_DIR=$(shell echo ./test_reports/temp`echo $$RANDOM$$RANDOM`profile)
+TEMP_PROFILE_DIR=./test_reports/temp_profile
 
 # These are the Firefox command line arguments.
 FIREFOX_ARGS=-no-remote -profile $(TEMP_PROFILE_DIR)
@@ -9,21 +9,41 @@ CHROME_ARGS=--user-data-dir=$(TEMP_PROFILE_DIR) --no-first-run --disable-sync
 
 # Unit test uses the manifest from ref test to determine which browsers will
 # be used for running the unit tests.
-MANIFEST:=../resources/browser_manifests/browser_manifest.json
+MANIFEST=../resources/browser_manifests/browser_manifest.json
 
 # Fetch the paths to browsers that are going to be used in testing.
-# For OS X the path to the binary needs to be added. Also the browser
-# arguments are added for each of the browser.
-BROWSERS=$(shell echo `\
+# For OS X the path to the binary needs to be added.
+# Add the browser arguments for each browser.
+# Create random profile directory for each browser.
+# Replace " with @@@@ so that echoing do not destroy the quotation marks.
+BROWSERS_WITH_UKNOWN_RANDOM_PROFILE_PATHS=$(shell echo `\
   sed -n 's|.*"path":\(.*\)|\1,|p' $(MANIFEST) | \
   sed 's|\(Aurora\.app\)|\1/Contents/MacOS/firefox-bin|' | \
   sed 's|\(Firefox.*\.app\)|\1/Contents/MacOS/firefox-bin|' | \
   sed 's|\(Google Chrome\.app\)|\1/Contents/MacOS/Google Chrome|' | \
   sed "s|\(irefox.*\)\",|\1;$(FIREFOX_ARGS)\",|" | \
   sed "s|\(hrome.*\)\",|\1;$(CHROME_ARGS)\",|" | \
+  sed 's|\(temp_profile\)|\1_$$(echo $$RANDOM$$RANDOM)|' | \
+  sed 's|"|@@@@|g' | \
   tr -d '\n' \
 `)
 
+# Echo the variable so that the unknown random directories become known.
+# Replace @@@@ with " so that jsTestDriver will work properly.
+BROWSERS:=$(shell echo "$(BROWSERS_WITH_UKNOWN_RANDOM_PROFILE_PATHS)" | \
+  sed 's|@@@@|"|g' \
+)
+
+# Split the browsers to separate lines.
+# Get the known random directories for browsers. This information will be used
+# to create the profile directories beforehand.
+# Join the separate lines back together.
+PROFILES:=$(shell echo $(BROWSERS) | \
+  sed 's|,|\n|g' | \
+  sed -n "s|.*\(\..*_[0-9]\+\).*|\1 |p" | \
+  tr -d '\n' \
+)
+
 # This is the command to invoke the unit test.
 PROG:=java \
 -Xms512m \
@@ -37,8 +57,9 @@ PROG:=java \
 --testOutput ./test_reports/
 
 test:
+       @mkdir -p $(PROFILES)
        $(PROG)
-       @rm -rf ./test_reports/temp*profile
+       @rm -rf $(PROFILES)
 
 .phony:: test