Merge pull request #367 from ochafik/fix-future-ctors

fix: Future.{value,error} are named constructors, not statics

Author: ochafik

lib/facade_converter.ts

@@ -770,10 +770,12 @@ export class FacadeConverter extends base.TranspilerBase {
   };
   private es6PromisesProp: ts.Map<PropertyHandler> = {
     'resolve': (p: ts.PropertyAccessExpression) => {
+      this.emit('new ');
       this.visit(p.expression);
       this.emit('.value');
     },
     'reject': (p: ts.PropertyAccessExpression) => {
+      this.emit('new ');
       this.visit(p.expression);
       this.emit('.error');
     },

test/facade_converter_test.ts

@@ -254,10 +254,10 @@ var y = x.length;`);
     it('translates into Futures', () => {
       expectWithTypes('let x: Promise = Promise.resolve(1);').to.equal(`import "dart:async";
 
-Future x = Future.value(1);`);
+Future x = new Future.value(1);`);
       expectWithTypes('let x: Promise = Promise.reject(1);').to.equal(`import "dart:async";
 
-Future x = Future.error(1);`);
+Future x = new Future.error(1);`);
       expectWithTypes('let x: Promise = new Promise((resolve) => {resolve(1);});')
           .to.equal(`import "dart:async";