feat(): Dartanalyzer running on travis.

Author: hansl

.gitignore

@@ -8,6 +8,13 @@
 /node_modules
 /bower_components
 
+# Dart
+/.pub
+/.packages
+/packages
+/pubspec.lock
+
+
 # IDEs
 /.idea
 

.travis.yml

@@ -10,6 +10,7 @@ node_js:
 cache:
   directories:
     - node_modules
+    - $HOME/.pub-cache
 
 env:
   global:
@@ -19,6 +20,8 @@ env:
   - BROWSER_STACK_USERNAME=angularteam1
   - BROWSER_STACK_ACCESS_KEY=BWCd4SynLzdDcv8xtzsB
   - ARCH=linux-x64
+  - DART_DEV_VERSION=latest
+  - DART_STABLE_VERSION=latest
   # Token for tsd to increase github rate limit
   # See https://github.com/DefinitelyTyped/tsd#tsdrc
   # This does not use http://docs.travis-ci.com/user/environment-variables/#Secure-Variables
@@ -32,6 +35,7 @@ env:
     # Order: a slower build first, so that we don't occupy an idle travis worker waiting for others to complete.
     - MODE=saucelabs_required
     - MODE=browserstack_required
+    - MODE=dart_required DART_CHANNEL=stable DART_VERSION=$DART_STABLE_VERSION
     - MODE=saucelabs_optional
     - MODE=browserstack_optional
 
@@ -40,6 +44,10 @@ matrix:
   - env: "MODE=saucelabs_optional"
   - env: "MODE=browserstack_optional"
 
+before_install:
+  - ./scripts/ci/install_dart.sh
+
+
 install:
   - npm install
 
@@ -52,7 +60,7 @@ before_script:
 
 
 script:
-- ./scripts/ci/build-and-test.sh ${MODE}
+- ./scripts/ci/build-and-test.sh
 
 cache:
   directories:

docs/dart/writing-compatible-typescript.md

@@ -0,0 +1,40 @@
+# Writing Dart Compatible TypeScript
+
+When writing TypeScript files in Material 2, it is important to keep in mind that these files will be automatically compiled and verified in Dart. There is valid TypeScript that will work in Angular 2, but will be transpiled to invalid Dart code.
+
+Because of this, there's certain things to keep in mind.
+
+## Requirements
+
+This document refers to the following concepts, but do not explain them:
+
+* The Dart language: https://www.dartlang.org
+* The `pub` Dart packages management: https://pub.dartlang.org/
+* The `angular-cli` toolchain: https://github.com/angular/angular-cli
+
+## Dart Toolchain
+
+When running `ng build` (or `ng server` which also builds), a Broccoli tree will automatically compile and format your TypeScript code into Dart.  These files live along the JavaScript output of the build process, and also are moved into a special directory structure that is compatible with `pub`.
+
+For example, let say you have a `md-awesome` component for Material, which contains the `awesome.ts` implementation. This file is contained in the source code as `src/components/awesome/awesome.ts`. Building this component (assuming no specs or `scss`) will result in the following files:
+
+* `dist/components/awesome/awesome.ts`. The original copy.
+* `dist/components/awesome/awesome.js`. The Javascript code.
+* `dist/components/awesome/awesome.js.map`. Sourcemap.
+* `dist/components/awesome/awesome.dart`. The Dart code, transpiled by `ts2dart` and formatted by `dartfmt`.
+* `dist/dart/lib/components/awesome/awesome.dart`. The same file, copied from above into a directory structure compatible with `pub`.
+
+If you have transpilation errors (ie. your TypeScript is invalid in Dart), these will fail the `ng build` step now and will output the errors in the console.
+
+## Gotchas
+
+Here's a list of gotchas to keep in mind when writing TypeScript code that will be compiled to Dart:
+
+* **Unused imports are enforced in Dart.** Importing a file but not using it is an error, not a warning.
+* **`void` is not a proper type for templates.** This is important when creating `EventEmitter<>` or other templates. Instead of `void`, use `Object` and pass in `null`.
+* **There's no global execution.** Every executed code need to be in a function.
+* **Unit tests need to live inside a `main()` function.** Because of the point above, `describe()` is technically executed code and has to live in a function. Additionally, spec files are programs in Dart and as such much have a `void main() {}` function containing the test code.
+* **Boolean expressions are required to be boolean.** There's no type coercion or truthiness concept in Dart. Code like this: `if (!aNumber) {}` must be refactored to be `if (aNumber != 0) {}`.
+* **Accessing any platform primitive must be done through a Facade.** For example, Promises or DOM manipulation. Facades are going to be provided on a need-to-have basis.
+* **Union types are a no-go.** Dart has them on its roadmap, but for now we must avoid them.
+* **Dart files cannot have the same name as a reserved keyword.** For example, `for.dart`, `switch.dart` or `class.dart` are all invalid names. Since the TypeScript files have the same name when transpiled to Dart, they also have the same restriction.

ember-cli-build.js

@@ -4,11 +4,13 @@ var path = require('path');
 // Import the require hook. Enables us to require TS files natively.
 require('ts-node/register');
 
+const detect = require('./tools/build/dart').detect;
 var mergeTrees = require('broccoli-merge-trees');
 var Angular2App = require('angular-cli/lib/broccoli/angular2-app');
 var BroccoliSass = require('broccoli-sass');
 var broccoliAutoprefixer = require('broccoli-autoprefixer');
 var BroccoliTs2Dart = require('./tools/broccoli/broccoli-ts2dart').default;
+var dartfmt = require('./tools/broccoli/broccoli-dartfmt').default;
 
 var autoprefixerOptions = require('./build/autoprefixer-options');
 
@@ -17,21 +19,39 @@ module.exports = function(defaults) {
   var demoCssTree = getCssTree('demo-app');
   var componentCssTree = getCssTree('components');
   var angularAppTree = new Angular2App(defaults);
-  var dartAppTree = new BroccoliTs2Dart('src/', {
-    generateLibraryName: true,
-    generateSourceMap: false,
-    translateBuiltins: true,
-  });
+  var dartAppTree = getDartTree('src/');
 
   return mergeTrees([
     angularAppTree.toTree(),
     componentCssTree,
     demoAppCssTree,
     demoCssTree,
-    dartAppTree,
-  ]);
+  ].concat(dartAppTree || []));
 };
 
+/** Gets the Dart tree - Transpile Dart files and format them afterward. */
+function getDartTree(root) {
+  const ts2dart = new BroccoliTs2Dart('src/', {
+    generateLibraryName: true,
+    generateSourceMap: false,
+    translateBuiltins: true,
+  });
+
+  const dartSDK = detect();
+  if (dartSDK) {
+    // If Dart isn't found, detect() will throw an error.
+    return dartfmt(ts2dart, { dartSDK });
+  } else {
+    console.warn('---------------------------------------');
+    console.warn('You do not have the Dart SDK installed.');
+    console.warn('In order to contribute to this repo, please refer to');
+    console.warn('https://github.com/angular/material2/blob/master/CONTRIBUTING.md');
+    console.warn('');
+    console.warn('You can still build and serve the demo app without dart support.');
+    return null;
+  }
+}
+
 /** Gets the tree for all of the components' CSS. */
 function getCssTree(folder) {
   var srcPath = `src/${folder}/`;

package.json

@@ -9,6 +9,7 @@
   },
   "scripts": {
     "build": "ng build",
+    "dartanalyzer": "ts-node scripts/ci/dart_analyzer",
     "demo-app": "cd src && ng serve",
     "test": "karma start test/karma.conf.js"
   },
@@ -34,17 +35,23 @@
     "broccoli-sass": "^0.7.0",
     "browserstacktunnel-wrapper": "^1.4.2",
     "ember-cli-inject-live-reload": "^1.3.0",
+    "fs-extra": "^0.26.5",
+    "glob": "^6.0.4",
     "jasmine-core": "^2.3.4",
+    "js-yaml": "^3.5.2",
     "karma": "^0.13.15",
     "karma-browserstack-launcher": "^0.1.7",
     "karma-chrome-launcher": "^0.2.1",
+    "karma-dart": "^0.3.0",
     "karma-firefox-launcher": "^0.1.7",
     "karma-jasmine": "^0.3.6",
     "karma-sauce-launcher": "^0.2.14",
+    "strip-ansi": "^3.0.0",
     "symlink-or-copy": "^1.0.1",
     "ts-node": "^0.5.5",
     "ts2dart": "^0.7.20",
     "tslint": "^3.2.2",
-    "typescript": "^1.7.5"
+    "typescript": "^1.7.5",
+    "which": "^1.2.4"
   }
 }

pubspec.yaml

@@ -0,0 +1,16 @@
+name: material2
+environment:
+  sdk: '>=1.12.0-dev.5.10 <2.0.0'
+dependencies:
+  observe: '^0.13.1'
+  angular2: '2.0.0-beta.0'
+dev_dependencies:
+  guinness: '^0.1.18'
+  intl: '^0.12.4'
+  unittest: '^0.11.5+4'
+  quiver: '^0.21.4'
+transformers:
+- angular2:
+    platform_directives: 'package:angular2/src/common/directives.dart#CORE_DIRECTIVES'
+    entry_points:
+        - web/src/demo-app/demo-app.dart

scripts/ci/README.md

@@ -0,0 +1,12 @@
+# Continuous Integration Scripts
+
+This directory contains scripts that are related to CI only.
+
+### `build-and-test.sh`
+
+The main script. Setup the tests environment, build the files using the `angular-cli` tool and run the tests.
+
+### `install_dart.sh`
+
+Only run in Dart environments. Install Dart locally on the CI.
+

scripts/ci/build-and-test.sh

@@ -1,57 +1,27 @@
 #!/usr/bin/env bash
-set -ex
 
 echo "=======  Starting build-and-test.sh  ========================================"
 
 # Go to project dir
-SCRIPT_DIR=$(dirname $0)
-cd ${SCRIPT_DIR}/../..
+cd $(dirname $0)/../..
 
+# Include sources.
+source scripts/ci/sources/mode.sh
+source scripts/ci/sources/tunnel.sh
 
-start_tunnel() {
-  case "$MODE" in
-    saucelabs*)
-      ./scripts/sauce/sauce_connect_setup.sh
-      ;;
-    browserstack*)
-      ./scripts/browserstack/start_tunnel.sh
-      ;;
-    *)
-      ;;
-  esac
-}
-
-wait_for_tunnel() {
-  case "$MODE" in
-    saucelabs*)
-      ./scripts/sauce/sauce_connect_block.sh
-      ;;
-    browserstack*)
-      ./scripts/browserstack/waitfor_tunnel.sh
-      ;;
-    *)
-      ;;
-  esac
-  sleep 10
-}
-
-teardown_tunnel() {
-  case "$MODE" in
-    saucelabs*)
-      ./scripts/sauce/sauce_connect_teardown.sh
-      ;;
-    browserstack*)
-      # ./scripts/browserstack/teardown_tunnel.sh
-      ;;
-    *)
-      ;;
-  esac
-}
-
+# Setup environment.
+is_dart && source scripts/ci/sources/env_dart.sh
 
 start_tunnel
-wait_for_tunnel
 npm run build
-karma start test/karma.conf.js --single-run --no-auto-watch --reporters='dots'
+is_dart && pub install
+
+wait_for_tunnel
+if is_dart; then
+  # npm run dartanalyzer
+  echo 'TODO(hans): Implement dartanalyzer'
+else
+  karma start test/karma.conf.js --single-run --no-auto-watch --reporters='dots'
+fi
 teardown_tunnel
 

scripts/ci/dart_analyzer.ts

@@ -0,0 +1,12 @@
+import {analyze} from '../../tools/build/dart_analyzer';
+import {detect} from '../../tools/build/dart';
+
+
+const command = detect()['ANALYZER'];
+
+analyze('dist/dart', command, true)
+  .then(() => {
+    console.log('Done.');
+  }, (err) => {
+    console.error('Error:', err);
+  });

scripts/ci/install_dart.sh

@@ -0,0 +1,33 @@
+#!/bin/bash
+
+set -e -x
+
+
+if [ -z "${DART_CHANNEL}" -o -z "${DART_VERSION}" -o -z "${ARCH}" ]; then
+  echo "No Dart necessary. Not installing."
+  exit 0
+fi
+
+AVAILABLE_DART_VERSION=$(curl "https://storage.googleapis.com/dart-archive/channels/${DART_CHANNEL}/release/${DART_VERSION}/VERSION" | python -c \
+    'import sys, json; print(json.loads(sys.stdin.read())["version"])')
+
+echo Fetch Dart channel: ${DART_CHANNEL}
+
+URL_PREFIX=https://storage.googleapis.com/dart-archive/channels/${DART_CHANNEL}/release/${DART_VERSION}
+DART_SDK_URL="$URL_PREFIX/sdk/dartsdk-$ARCH-release.zip"
+DARTIUM_URL="$URL_PREFIX/dartium/dartium-$ARCH-release.zip"
+
+download_and_unzip() {
+  ZIPFILE=${1/*\//}
+  curl -O -L $1 && unzip -q $ZIPFILE && rm $ZIPFILE
+}
+
+# TODO: do these downloads in parallel
+download_and_unzip $DART_SDK_URL
+download_and_unzip $DARTIUM_URL
+
+echo Fetched new dart version $(<dart-sdk/version)
+
+if [[ -n $DARTIUM_URL ]]; then
+  mv dartium-* chromium
+fi