Fix cctests using Sockets to be able to run in parallel

BUG=v8:945

Review URL: https://codereview.chromium.org/11015008

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12636 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 44967541
...@@ -213,6 +213,7 @@ static inline v8::Local<v8::Value> CompileRun(const char* source) { ...@@ -213,6 +213,7 @@ static inline v8::Local<v8::Value> CompileRun(const char* source) {
return v8::Script::Compile(v8::String::New(source))->Run(); return v8::Script::Compile(v8::String::New(source))->Run();
} }
// Helper function that compiles and runs the source with given origin. // Helper function that compiles and runs the source with given origin.
static inline v8::Local<v8::Value> CompileRunWithOrigin(const char* source, static inline v8::Local<v8::Value> CompileRunWithOrigin(const char* source,
const char* origin_url, const char* origin_url,
...@@ -225,4 +226,11 @@ static inline v8::Local<v8::Value> CompileRunWithOrigin(const char* source, ...@@ -225,4 +226,11 @@ static inline v8::Local<v8::Value> CompileRunWithOrigin(const char* source,
} }
// Pick a slightly different port to allow tests to be run in parallel.
static inline int FlagDependentPortOffset() {
return ::v8::internal::FLAG_crankshaft == false ? 100 :
::v8::internal::FLAG_always_opt ? 200 : 0;
}
#endif // ifndef CCTEST_H_ #endif // ifndef CCTEST_H_
...@@ -68,11 +68,6 @@ test-api/OutOfMemoryNested: SKIP ...@@ -68,11 +68,6 @@ test-api/OutOfMemoryNested: SKIP
# BUG(355): Test crashes on ARM. # BUG(355): Test crashes on ARM.
test-log/ProfLazyMode: SKIP test-log/ProfLazyMode: SKIP
# BUG(945): Tests using Socket cannot be run in parallel.
test-debug/DebuggerAgent: SKIP
test-debug/DebuggerAgentProtocolOverflowHeader: SKIP
test-sockets/Socket: SKIP
# BUG(1075): Unresolved crashes. # BUG(1075): Unresolved crashes.
test-serialize/Deserialize: SKIP test-serialize/Deserialize: SKIP
test-serialize/DeserializeFromSecondSerializationAndRunScript2: SKIP test-serialize/DeserializeFromSecondSerializationAndRunScript2: SKIP
...@@ -89,8 +84,3 @@ test-log/ProfLazyMode: SKIP ...@@ -89,8 +84,3 @@ test-log/ProfLazyMode: SKIP
# platform-tls.h does not contain an ANDROID-related header. # platform-tls.h does not contain an ANDROID-related header.
test-platform-tls/FastTLS: SKIP test-platform-tls/FastTLS: SKIP
# BUG(945): Tests using Socket cannot be run in parallel.
test-debug/DebuggerAgent: SKIP
test-debug/DebuggerAgentProtocolOverflowHeader: SKIP
test-sockets/Socket: SKIP
...@@ -5833,9 +5833,9 @@ TEST(DebuggerAgent) { ...@@ -5833,9 +5833,9 @@ TEST(DebuggerAgent) {
i::Debugger* debugger = i::Isolate::Current()->debugger(); i::Debugger* debugger = i::Isolate::Current()->debugger();
// Make sure these ports is not used by other tests to allow tests to run in // Make sure these ports is not used by other tests to allow tests to run in
// parallel. // parallel.
const int kPort1 = 5858; const int kPort1 = 5858 + FlagDependentPortOffset();
const int kPort2 = 5857; const int kPort2 = 5857 + FlagDependentPortOffset();
const int kPort3 = 5856; const int kPort3 = 5856 + FlagDependentPortOffset();
// Make a string with the port2 number. // Make a string with the port2 number.
const int kPortBufferLen = 6; const int kPortBufferLen = 6;
...@@ -5934,7 +5934,7 @@ void DebuggerAgentProtocolServerThread::Run() { ...@@ -5934,7 +5934,7 @@ void DebuggerAgentProtocolServerThread::Run() {
TEST(DebuggerAgentProtocolOverflowHeader) { TEST(DebuggerAgentProtocolOverflowHeader) {
// Make sure this port is not used by other tests to allow tests to run in // Make sure this port is not used by other tests to allow tests to run in
// parallel. // parallel.
const int kPort = 5860; const int kPort = 5860 + FlagDependentPortOffset();
static const char* kLocalhost = "localhost"; static const char* kLocalhost = "localhost";
// Make a string with the port number. // Make a string with the port number.
......
...@@ -124,7 +124,7 @@ static void SendAndReceive(int port, char *data, int len) { ...@@ -124,7 +124,7 @@ static void SendAndReceive(int port, char *data, int len) {
TEST(Socket) { TEST(Socket) {
// Make sure this port is not used by other tests to allow tests to run in // Make sure this port is not used by other tests to allow tests to run in
// parallel. // parallel.
static const int kPort = 5859; static const int kPort = 5859 + FlagDependentPortOffset();
bool ok; bool ok;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment