runtime/vm/dart_api_impl_test.cc - Issue 10386107: Implement spawnUri from dart:isolate. This function allows us to

Unified Diff: runtime/vm/dart_api_impl_test.cc

Issue 10386107: Implement spawnUri from dart:isolate. This function allows us to (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/

Patch Set: Created 8 years, 7 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: runtime/vm/dart_api_impl_test.cc

===================================================================

--- runtime/vm/dart_api_impl_test.cc (revision 7517)

+++ runtime/vm/dart_api_impl_test.cc (working copy)

@@ -2331,8 +2331,7 @@

// Load imported lib.

Dart_Handle url = Dart_NewString("library_url");

Dart_Handle source = Dart_NewString(kImportedScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

- Dart_Handle imported_lib = Dart_LoadLibrary(url, source, import_map);

+ Dart_Handle imported_lib = Dart_LoadLibrary(url, source);

EXPECT_VALID(imported_lib);

Dart_Handle result = Dart_LibraryImportLibrary(lib, imported_lib);

EXPECT_VALID(result);

@@ -3294,14 +3293,13 @@

// Load lib1

Dart_Handle url = Dart_NewString("library1_url");

Dart_Handle source = Dart_NewString(kLibrary1Chars);

- Dart_Handle import_map = Dart_NewList(0);

- Dart_Handle lib1 = Dart_LoadLibrary(url, source, import_map);

+ Dart_Handle lib1 = Dart_LoadLibrary(url, source);

EXPECT_VALID(lib1);

// Load lib2

url = Dart_NewString("library2_url");

source = Dart_NewString(kLibrary2Chars);

- Dart_Handle lib2 = Dart_LoadLibrary(url, source, import_map);

+ Dart_Handle lib2 = Dart_LoadLibrary(url, source);

EXPECT_VALID(lib2);

// Import lib2 from lib1

@@ -3577,8 +3575,7 @@

static Dart_Handle library_handler(Dart_LibraryTag tag,

Dart_Handle library,

- Dart_Handle url,

- Dart_Handle import_url_map) {

+ Dart_Handle url) {

if (tag == kCanonicalizeUrl) {

return url;

}

@@ -3595,42 +3592,41 @@

Dart_Handle source = Dart_NewString(kScriptChars);

Dart_Handle error = Dart_Error("incoming error");

Dart_Handle result;

- Dart_Handle import_map = Dart_NewList(0);

result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(Dart_Null(), source, import_map);

+ result = Dart_LoadScript(Dart_Null(), source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("Dart_LoadScript expects argument 'url' to be non-null.",

Dart_GetError(result));

- result = Dart_LoadScript(Dart_True(), source, import_map);

+ result = Dart_LoadScript(Dart_True(), source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("Dart_LoadScript expects argument 'url' to be of type String.",

Dart_GetError(result));

- result = Dart_LoadScript(error, source, import_map);

+ result = Dart_LoadScript(error, source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("incoming error", Dart_GetError(result));

- result = Dart_LoadScript(url, Dart_Null(), import_map);

+ result = Dart_LoadScript(url, Dart_Null());

EXPECT(Dart_IsError(result));

EXPECT_STREQ("Dart_LoadScript expects argument 'source' to be non-null.",

Dart_GetError(result));

- result = Dart_LoadScript(url, Dart_True(), import_map);

+ result = Dart_LoadScript(url, Dart_True());

EXPECT(Dart_IsError(result));

EXPECT_STREQ(

"Dart_LoadScript expects argument 'source' to be of type String.",

Dart_GetError(result));

- result = Dart_LoadScript(url, error, import_map);

+ result = Dart_LoadScript(url, error);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("incoming error", Dart_GetError(result));

// Load a script successfully.

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

EXPECT_VALID(result);

result = Dart_Invoke(result, Dart_NewString("main"), 0, NULL);

@@ -3641,7 +3637,7 @@

EXPECT_EQ(12345, value);

// Further calls to LoadScript are errors.

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("Dart_LoadScript: "

"A script has already been loaded from 'dart:test-lib'.",

@@ -3660,8 +3656,7 @@

static Dart_Handle import_library_handler(Dart_LibraryTag tag,

Dart_Handle library,

- Dart_Handle url,

- Dart_Handle import_url_map) {

+ Dart_Handle url) {

if (tag == kCanonicalizeUrl) {

return url;

}

@@ -3712,7 +3707,9 @@

}

Dart_Handle result = Dart_SetLibraryTagHandler(import_library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_SetImportMap(import_map);

+ EXPECT_VALID(result);

+ result = Dart_LoadScript(url, source);

EXPECT(!Dart_IsError(result));

}

@@ -3725,10 +3722,9 @@

"}";

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

Dart_Handle result = Dart_SetLibraryTagHandler(import_library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

EXPECT(Dart_IsError(result));

EXPECT(strstr(Dart_GetError(result),

"import variable 'DEF' has not been defined"));

@@ -3750,7 +3746,9 @@

}

Dart_Handle result = Dart_SetLibraryTagHandler(import_library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_SetImportMap(import_map);

+ EXPECT_VALID(result);

+ result = Dart_LoadScript(url, source);

EXPECT(Dart_IsError(result));

EXPECT(strstr(Dart_GetError(result), "'}' expected"));

}

@@ -3761,10 +3759,9 @@

")";

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

Dart_Handle result = Dart_SetLibraryTagHandler(import_library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

EXPECT(Dart_IsError(result));

EXPECT(strstr(Dart_GetError(result), "unexpected token ')'"));

}

@@ -3781,15 +3778,14 @@

// Create a test library and Load up a test script in it.

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

Dart_Handle result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

EXPECT_VALID(result);

url = Dart_NewString("library1.dart");

source = Dart_NewString(kLibrary1Chars);

- result = Dart_LoadLibrary(url, source, import_map);

+ result = Dart_LoadLibrary(url, source);

EXPECT_VALID(result);

result = Dart_LookupLibrary(url);

@@ -3823,7 +3819,7 @@

"#library('library1_name');";

Dart_Handle url = Dart_NewString("library1_url");

Dart_Handle source = Dart_NewString(kLibrary1Chars);

- Dart_Handle lib = Dart_LoadLibrary(url, source, Dart_Null());

+ Dart_Handle lib = Dart_LoadLibrary(url, source);

Dart_Handle error = Dart_Error("incoming error");

EXPECT_VALID(lib);

@@ -3861,13 +3857,12 @@

Dart_Handle url = Dart_NewString("library1_url");

Dart_Handle source = Dart_NewString(kLibrary1Chars);

- Dart_Handle import_map = Dart_NewList(0);

- Dart_Handle lib1 = Dart_LoadLibrary(url, source, import_map);

+ Dart_Handle lib1 = Dart_LoadLibrary(url, source);

EXPECT_VALID(lib1);

url = Dart_NewString("library2_url");

source = Dart_NewString(kLibrary2Chars);

- Dart_Handle lib2 = Dart_LoadLibrary(url, source, import_map);

+ Dart_Handle lib2 = Dart_LoadLibrary(url, source);

EXPECT_VALID(lib2);

result = Dart_LibraryImportLibrary(Dart_Null(), lib2);

@@ -3916,44 +3911,43 @@

Dart_Handle url = Dart_NewString("library1_url");

Dart_Handle source = Dart_NewString(kLibrary1Chars);

- Dart_Handle import_map = Dart_NewList(0);

- result = Dart_LoadLibrary(Dart_Null(), source, import_map);

+ result = Dart_LoadLibrary(Dart_Null(), source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("Dart_LoadLibrary expects argument 'url' to be non-null.",

Dart_GetError(result));

- result = Dart_LoadLibrary(Dart_True(), source, import_map);

+ result = Dart_LoadLibrary(Dart_True(), source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("Dart_LoadLibrary expects argument 'url' to be of type String.",

Dart_GetError(result));

- result = Dart_LoadLibrary(error, source, import_map);

+ result = Dart_LoadLibrary(error, source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("incoming error", Dart_GetError(result));

- result = Dart_LoadLibrary(url, Dart_Null(), import_map);

+ result = Dart_LoadLibrary(url, Dart_Null());

EXPECT(Dart_IsError(result));

EXPECT_STREQ("Dart_LoadLibrary expects argument 'source' to be non-null.",

Dart_GetError(result));

- result = Dart_LoadLibrary(url, Dart_True(), import_map);

+ result = Dart_LoadLibrary(url, Dart_True());

EXPECT(Dart_IsError(result));

EXPECT_STREQ(

"Dart_LoadLibrary expects argument 'source' to be of type String.",

Dart_GetError(result));

- result = Dart_LoadLibrary(url, error, import_map);

+ result = Dart_LoadLibrary(url, error);

EXPECT(Dart_IsError(result));

EXPECT_STREQ("incoming error", Dart_GetError(result));

// Success.

- result = Dart_LoadLibrary(url, source, import_map);

+ result = Dart_LoadLibrary(url, source);

EXPECT_VALID(result);

EXPECT(Dart_IsLibrary(result));

// Duplicate library load fails.

- result = Dart_LoadLibrary(url, source, import_map);

+ result = Dart_LoadLibrary(url, source);

EXPECT(Dart_IsError(result));

EXPECT_STREQ(

"Dart_LoadLibrary: library 'library1_url' has already been loaded.",

@@ -3967,8 +3961,7 @@

")";

Dart_Handle url = Dart_NewString("library1_url");

Dart_Handle source = Dart_NewString(kLibrary1Chars);

- Dart_Handle import_map = Dart_NewList(0);

- Dart_Handle result = Dart_LoadLibrary(url, source, import_map);

+ Dart_Handle result = Dart_LoadLibrary(url, source);

EXPECT(Dart_IsError(result));

EXPECT(strstr(Dart_GetError(result), "unexpected token ')'"));

}

@@ -3987,8 +3980,7 @@

// Load up a library.

Dart_Handle url = Dart_NewString("library1_url");

Dart_Handle source = Dart_NewString(kLibrary1Chars);

- Dart_Handle import_map = Dart_NewList(0);

- Dart_Handle lib = Dart_LoadLibrary(url, source, import_map);

+ Dart_Handle lib = Dart_LoadLibrary(url, source);

EXPECT_VALID(lib);

EXPECT(Dart_IsLibrary(lib));

@@ -4101,10 +4093,9 @@

// Load a test script.

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- Dart_Handle lib = Dart_LoadScript(url, source, import_map);

+ Dart_Handle lib = Dart_LoadScript(url, source);

EXPECT_VALID(lib);

EXPECT(Dart_IsLibrary(lib));

Dart_Handle cls = Dart_GetClass(lib, Dart_NewString("Test"));

@@ -4185,15 +4176,15 @@

Dart_Handle source = Dart_NewString(kScriptChars);

result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, Dart_Null());

+ result = Dart_LoadScript(url, source);

url = Dart_NewString("library1.dart");

source = Dart_NewString(kLibrary1Chars);

- Dart_LoadLibrary(url, source, Dart_Null());

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("library2.dart");

source = Dart_NewString(kLibrary2Chars);

- Dart_LoadLibrary(url, source, Dart_Null());

+ Dart_LoadLibrary(url, source);

result = Dart_Invoke(result, Dart_NewString("main"), 0, NULL);

EXPECT(Dart_IsError(result));

@@ -4220,18 +4211,17 @@

// Create a test library and Load up a test script in it.

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

url = Dart_NewString("library1.dart");

source = Dart_NewString(kLibrary1Chars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("library2.dart");

source = Dart_NewString(kLibrary2Chars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

result = Dart_Invoke(result, Dart_NewString("main"), 0, NULL);

EXPECT_VALID(result);

@@ -4255,18 +4245,17 @@

// Create a test library and Load up a test script in it.

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

url = Dart_NewString("library2.dart");

source = Dart_NewString(kLibrary2Chars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("library1.dart");

source = Dart_NewString(kLibrary1Chars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

result = Dart_Invoke(result, Dart_NewString("main"), 0, NULL);

EXPECT(Dart_IsError(result));

@@ -4312,34 +4301,33 @@

// Create a test library and Load up a test script in it.

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

url = Dart_NewString("libraryA.dart");

source = Dart_NewString(kLibraryAChars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("libraryC.dart");

source = Dart_NewString(kLibraryCChars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("libraryB.dart");

source = Dart_NewString(kLibraryBChars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("libraryD.dart");

source = Dart_NewString(kLibraryDChars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("libraryF.dart");

source = Dart_NewString(kLibraryFChars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

url = Dart_NewString("libraryE.dart");

source = Dart_NewString(kLibraryEChars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

result = Dart_Invoke(result, Dart_NewString("main"), 0, NULL);

EXPECT(Dart_IsError(result));

@@ -4366,14 +4354,13 @@

// Create a test library and Load up a test script in it.

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

result = Dart_SetLibraryTagHandler(library_handler);

EXPECT_VALID(result);

- result = Dart_LoadScript(url, source, import_map);

+ result = Dart_LoadScript(url, source);

url = Dart_NewString("lib.dart");

source = Dart_NewString(kLibraryChars);

- Dart_LoadLibrary(url, source, import_map);

+ Dart_LoadLibrary(url, source);

result = Dart_Invoke(result, Dart_NewString("main"), 0, NULL);

EXPECT_VALID(result);

@@ -4469,8 +4456,10 @@

}

-static bool RunLoopTestCallback(const char* name_prefix,

- void* data, char** error) {

+static bool RunLoopTestCallback(const char* script_name,

+ const char* main,

+ void* data,

+ char** error) {

const char* kScriptChars =

"#import('builtin');\n"

"#import('dart:isolate');\n"

@@ -4505,10 +4494,9 @@

Dart_EnterScope();

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

Dart_Handle result = Dart_SetLibraryTagHandler(TestCase::library_handler);

EXPECT_VALID(result);

- Dart_Handle lib = Dart_LoadScript(url, source, import_map);

+ Dart_Handle lib = Dart_LoadScript(url, source);

EXPECT_VALID(lib);

Dart_ExitScope();

return true;

@@ -4520,7 +4508,7 @@

bool throw_exception_parent) {

Dart_IsolateCreateCallback saved = Isolate::CreateCallback();

Isolate::SetCreateCallback(RunLoopTestCallback);

- RunLoopTestCallback(NULL, NULL, NULL);

+ RunLoopTestCallback(NULL, NULL, NULL, NULL);

Dart_EnterScope();

Dart_Handle lib = Dart_LookupLibrary(Dart_NewString(TestCase::url()));

@@ -4604,15 +4592,14 @@

{

sync->Enter();

char* error = NULL;

- shared_isolate = Dart_CreateIsolate(NULL, NULL, NULL, &error);

+ shared_isolate = Dart_CreateIsolate(NULL, NULL, NULL, NULL, &error);

EXPECT(shared_isolate != NULL);

Dart_EnterScope();

Dart_Handle url = Dart_NewString(TestCase::url());

Dart_Handle source = Dart_NewString(kScriptChars);

- Dart_Handle import_map = Dart_NewList(0);

Dart_Handle result = Dart_SetLibraryTagHandler(TestCase::library_handler);

EXPECT_VALID(result);

- lib = Dart_LoadScript(url, source, import_map);

+ lib = Dart_LoadScript(url, source);

EXPECT_VALID(lib);

result = Dart_SetNativeResolver(lib, &IsolateInterruptTestNativeLookup);

DART_CHECK_VALID(result);

« runtime/lib/isolate.dart ('K') | « runtime/vm/dart_api_impl.cc ('k') | runtime/vm/object.h » ('j') | tests/isolate/isolate.status » ('J')