Add a regression test for bug 2235.

Add a regression test for bug 2235.
Committed: https://code.google.com/p/dart/source/detail?r=8081

[turnidge, 2012-05-29 21:36:26]

TBR=iposva

[Ivan Posva, 2012-05-29 21:55:58]

The change LGTM as it mimics the other isolate tests. Adding Siggi as it bothers
me that we are depending on the system under test to run the tests.

-Ivan

https://chromiumcodereview.appspot.com/10443067/diff/1/tests/isolate/v2_spawn...
File tests/isolate/v2_spawn_function_custom_class_test.dart (right):

https://chromiumcodereview.appspot.com/10443067/diff/1/tests/isolate/v2_spawn...
tests/isolate/v2_spawn_function_custom_class_test.dart:28: test('message - reply
chain', () {
What bothers me here is that we are in a circular dependency. The test framework
itself depends on isolates to be working properly and it is used to test
isolates. Siggi, any suggestion to address this?

[Siggi Cherem (dart-lang), 2012-06-05 21:49:50]

https://chromiumcodereview.appspot.com/10443067/diff/1/tests/isolate/v2_spawn...
File tests/isolate/v2_spawn_function_custom_class_test.dart (right):

https://chromiumcodereview.appspot.com/10443067/diff/1/tests/isolate/v2_spawn...
tests/isolate/v2_spawn_function_custom_class_test.dart:28: test('message - reply
chain', () {
On 2012/05/29 21:55:58, Ivan Posva wrote:
> What bothers me here is that we are in a circular dependency. The test
framework
> itself depends on isolates to be working properly and it is used to test
> isolates. Siggi, any suggestion to address this?

It bothers me too.

The only reason we use dart:isolate in the unittest library is that we basically
need a platform independent way to run code at the end of the event loop. We use
messages within the same isolate to achieve this. That means that we are not
depending on 'spawn' for the unittest library, but we use port send/receive.
That's also why I didn't convert port_tests to use unittest.dart.

If we can provide a way to get a platform independent 'defer' in some 'dart:'
library, we could get rid of this dependence. Note, this platform independent
'defer' is needed also to fix some bugs in the 'future' code.