dart:io | Add automated testing for samples/sample_extension to buildbot.

dart:io | Add automated testing for samples/sample_extension to buildbot.

BUG=dartbug.com/3008
R=ajohnsen@google.com

Committed: https://code.google.com/p/dart/source/detail?r=25612

[Anders Johnsen, 2013-06-27 16:22:48]

+ricow, Can you check if this will work on buildbots?

What if we just renamed the files to *_test.dart, would they then run by
default?

https://codereview.chromium.org/18073003/diff/1/samples/tests/samples/standal...
File samples/tests/samples/standalone/sample_extension_test.dart (right):

https://codereview.chromium.org/18073003/diff/1/samples/tests/samples/standal...
samples/tests/samples/standalone/sample_extension_test.dart:20: return
Process.run('cmd.exe', ['/C', 'copy $src $dst']);
Use runInShell?

[Bill Hesse, 2013-06-27 16:37:27]

On 2013/06/27 16:22:48, Anders Johnsen wrote:
> +ricow, Can you check if this will work on buildbots?
> 
> What if we just renamed the files to *_test.dart, would they then run by
> default?
> 


We have to run the tests in a directory that contains the native library, or
rewrite the dart library file to have the path of the native library in it.  We
can't (shouldn't) modify the Dart checkout, only the build directory and temp
directories, so they are copied to a temp directory.  This is mostly copied from
tests/standalone/io/test_extension_test.dart.

I have tested this also with the -c dart2js and -c dartanalyzer options, and
runtime none and d8.
The status file is correctly excluding the browsers from running this test, so I
didn't need to change the status file at all - the default settings for
samples/tests/samples/standalone are correct.

[Anders Johnsen, 2013-06-27 16:46:51]

I see, thanks for explaining! LGTM

[ricow1, 2013-06-28 10:33:49]

[+ahe]

[Bill Hesse, 2013-07-04 14:11:37]

Added the samples/sample_extension target to the runtime build.  PTAL.

[Bill Hesse, 2013-07-30 12:16:24]

Committed patchset #2 manually as r25612 (presubmit successful).