ci: disable tests on platforms that dont support the test

Author: hansl

packages/angular_devkit/build_angular/test/browser/rebuild_spec_large.ts

@@ -12,6 +12,13 @@ import { TestLogger, Timeout, browserTargetSpec, host, runTargetSpec } from '../
 import { lazyModuleFiles, lazyModuleImport } from './lazy-module_spec_large';
 
 
+// TODO: replace this with an "it()" macro that's reusable globally.
+let linuxOnlyIt: typeof it = it;
+if (process.platform.startsWith('win')) {
+  linuxOnlyIt = xit;
+}
+
+
 describe('Browser Builder rebuilds', () => {
   const outputPath = normalize('dist');
 
@@ -201,7 +208,7 @@ describe('Browser Builder rebuilds', () => {
   }, Timeout.Basic);
 
 
-  it('rebuilds after errors in AOT', (done) => {
+  linuxOnlyIt('rebuilds after errors in AOT', (done) => {
     // Save the original contents of `./src/app/app.component.ts`.
     const origContent = virtualFs.fileBufferToString(
       host.scopedSync().read(normalize('src/app/app.component.ts')));
@@ -263,7 +270,7 @@ describe('Browser Builder rebuilds', () => {
   }, Timeout.Complex);
 
 
-  it('rebuilds AOT factories', (done) => {
+  linuxOnlyIt('rebuilds AOT factories', (done) => {
 
     host.writeMultipleFiles({
       'src/app/app.component.css': `

packages/angular_devkit/build_angular/test/protractor/works_spec_large.ts

@@ -11,25 +11,32 @@ import { retry } from 'rxjs/operators';
 import { host, protractorTargetSpec, runTargetSpec } from '../utils';
 
 
+// TODO: replace this with an "it()" macro that's reusable globally.
+let linuxOnlyIt: typeof it = it;
+if (process.platform.startsWith('win')) {
+  linuxOnlyIt = xit;
+}
+
+
 describe('Protractor Builder', () => {
   beforeEach(done => host.initialize().subscribe(undefined, done.fail, done));
   afterEach(done => host.restore().subscribe(undefined, done.fail, done));
 
-  it('works', (done) => {
+  linuxOnlyIt('works', (done) => {
     runTargetSpec(host, protractorTargetSpec).pipe(
       retry(3),
     ).subscribe(undefined, done.fail, done);
   }, 30000);
 
-  it('works with no devServerTarget', (done) => {
+  linuxOnlyIt('works with no devServerTarget', (done) => {
     const overrides = { devServerTarget: undefined };
 
     runTargetSpec(host, protractorTargetSpec, overrides).pipe(
       // This should fail because no server is available for connection.
     ).subscribe(undefined, done, done.fail);
   }, 30000);
 
-  it('overrides protractor specs', (done) => {
+  linuxOnlyIt('overrides protractor specs', (done) => {
     host.scopedSync().rename(normalize('./e2e/app.e2e-spec.ts'),
       normalize('./e2e/renamed-app.e2e-spec.ts'));
 
@@ -40,7 +47,7 @@ describe('Protractor Builder', () => {
     ).subscribe(undefined, done.fail, done);
   }, 60000);
 
-  it('overrides protractor suites', (done) => {
+  linuxOnlyIt('overrides protractor suites', (done) => {
     host.scopedSync().rename(normalize('./e2e/app.e2e-spec.ts'),
       normalize('./e2e/renamed-app.e2e-spec.ts'));
 

packages/angular_devkit/build_ng_packagr/src/build/index_spec_large.ts

@@ -12,6 +12,13 @@ import { NodeJsSyncHost } from '@angular-devkit/core/node';
 import { concatMap, tap } from 'rxjs/operators';
 
 
+// TODO: replace this with an "it()" macro that's reusable globally.
+let linuxOnlyIt: typeof it = it;
+if (process.platform.startsWith('win')) {
+  linuxOnlyIt = xit;
+}
+
+
 describe('NgPackagr Builder', () => {
   const workspaceFile = normalize('angular.json');
   const devkitRoot = normalize((global as any)._DevKitRoot); // tslint:disable-line:no-any
@@ -33,7 +40,7 @@ describe('NgPackagr Builder', () => {
     ).subscribe(undefined, done.fail, done);
   }, 30000);
 
-  it('tests works', (done) => {
+  linuxOnlyIt('tests works', (done) => {
     const targetSpec: TargetSpecifier = { project: 'lib', target: 'test' };
 
     return workspace.loadWorkspaceFromHost(workspaceFile).pipe(

packages/angular_devkit/core/node/host_spec.ts

@@ -16,6 +16,13 @@ import { Observable, Subscription } from 'rxjs';
 const temp = require('temp');
 
 
+// TODO: replace this with an "it()" macro that's reusable globally.
+let linuxOnlyIt: typeof it = it;
+if (process.platform.startsWith('win') || process.platform.startsWith('darwin')) {
+  linuxOnlyIt = xit;
+}
+
+
 describe('NodeJsAsyncHost', () => {
   let root: string;
   let host: virtualFs.Host<fs.Stats>;
@@ -29,7 +36,7 @@ describe('NodeJsAsyncHost', () => {
       .subscribe({ complete() { done(); } });
   });
 
-  it('can watch', done => {
+  linuxOnlyIt('can watch', done => {
     let obs: Observable<virtualFs.HostWatchEvent>;
     let subscription: Subscription;
     const content = virtualFs.stringToFileBuffer('hello world');
@@ -56,7 +63,7 @@ describe('NodeJsAsyncHost', () => {
         subscription.unsubscribe();
       })
       .then(done, done.fail);
-  }, 10000000);
+  }, 30000);
 });
 
 
@@ -73,7 +80,7 @@ describe('NodeJsSyncHost', () => {
     host.delete(normalize('/'));
   });
 
-  it('can watch', done => {
+  linuxOnlyIt('can watch', done => {
     let obs: Observable<virtualFs.HostWatchEvent>;
     let subscription: Subscription;
     const content = virtualFs.stringToFileBuffer('hello world');
@@ -102,6 +109,6 @@ describe('NodeJsSyncHost', () => {
         subscription.unsubscribe();
       })
       .then(done, done.fail);
-  }, 10000000);
+  }, 30000);
 
 });