dart-gde / chrome_gen.dart

Deprecated! See dart-gde/chrome.dart
https://github.com/dart-gde/chrome.dart
Other
6 stars 6 forks source link

Add chrome dart unit tests #127

Closed adam-singer closed 10 years ago

adam-singer commented 10 years ago

Just enough test harness in before adding all the unit tests.

bin/setup_app.dart will build apps in the app directory.

Usage:

–(~/dart/gen_tools.dart)–($ dart bin/setup_app.dart 
FormatException: This script requires at least 2 positional argument(s), but received 0.

Copies and builds a chrome application or extension

Usage:

dart setup_app.dart [options] <DIRECTORY> <MAIN>

Options:

-h, --help    Print this usage information.

Examples:

dart setup_app.dart app/test_ext harness.dart # Copies the packages folder to "app/test_ext" and builds "harness.dart"

Example:

dart setup_app.dart app/test_ext harness.dart

bin/load_app.sh will launch the apps in the app directory in either chrome/canary/dartium for mac only.

Usage:

NOTE: For extensions to be loaded by this script chrome//flags needs 'Experimental Extension APIs' enabled. 
Usage: ./bin/load_app.sh [chrome|canary|dartium] [app/extension directory]

Example:

./bin/load_app.sh dartium app/test_app/

Debated if the load_app.sh should just call the build everytime. Both the test_app and test_ext harness have a reload build it when the r key is pressed while open. Before going forward should we get this working in an automated or headless fashion like the spark project?

devoncarew commented 10 years ago

This looks great!

Before going forward should we get this working in an automated or headless fashion like the spark project?

+1 to headless eventually, but just getting tests running in any fashion is more important. I wouldn't want a perfect testing solution to delay a good one -

adam-singer commented 10 years ago

Cool, we'll merge this then we can start to adapt the rest of the tests. There was a fair amount of tests in the chrome.dart project and not sure how long it will take to port all of them in one shot.

adam-singer commented 10 years ago

heh, didnt know if you were still reviewing, totally overlooked the missing "lgtm". Might of shot the gun by accident.

devoncarew commented 10 years ago

Nope, no more comments. Thanks much for pushing on this!