Merge pull request #2651 from angular-fullstack/fix/canary-tests

Fix/canary tests

Author: Awk34

src/generators/app/index.js

@@ -542,6 +542,14 @@ export class Generator extends Base {
         // Convert HTML into Pug
         if(this.filters.pug) {
           let pugFilter = filter(['**/*.html', '!client/_index.html'], {restore: true});
+
+          function pugReplacer(contents) {
+            return contents
+              .replace('ngif', 'ngIf')
+              .replace('ngfor', 'ngFor')
+              .replace('ngmodel', 'ngModel');
+          }
+
           this.registerTransformStream([
             pugFilter,
             html2jade({
@@ -552,6 +560,11 @@ export class Generator extends Base {
             rename(path => {
               path.extname = '.pug';
             }),
+            tap(function(file, t) {
+              var contents = file.contents.toString();
+              contents = pugReplacer(contents);
+              file.contents = new Buffer(contents);
+            }),
             pugFilter.restore
           ]);
         }

src/test/get-expected-files.js

@@ -212,6 +212,7 @@ export function app(options) {
   /* WebSockets */
   if (options.ws) {
     files = files.concat([
+      'client/components/socket/primus.mock.' + script,
       'client/components/socket/socket.service.' + script,
       'client/components/socket/socket.mock.' + script,
       'server/api/thing/thing.socket.js',

src/test/main.test.js

@@ -255,7 +255,7 @@ describe('angular-fullstack:app', function() {
     }
   });
 
-  describe('with sequelize models, auth', function() {
+  describe.only('with sequelize models, auth', function() {
     var dir;
     var lintResult;
     var clientTestResult;

templates/app/_package.json

@@ -214,6 +214,7 @@
   },
   "scripts": {
     "test": "gulp test",
+    "test:client": "karma start ./karma.conf.js --single-run",
     <%_ if(filters.flow) { -%>
     "flow": "flow",
     <%_ } -%>

templates/app/client/app/main/main.component.js

@@ -24,11 +24,9 @@ export class MainComponent implements OnInit<% if(filters.ws) { %>, OnDestroy<%
   }
 
   ngOnInit() {
-    this.Http.get('/api/things')
-      .map(res => {
-        return res.json();
-      })
-      .catch(err => Observable.throw(err.json().error || 'Server error'))
+    return this.Http.get('/api/things')
+      .map(res => res.json())
+      // .catch(err => Observable.throw(err.json().error || 'Server error'))
       .subscribe(things => {
         this.awesomeThings = things;
         <%_ if(filters.ws) { -%>

templates/app/client/app/main/main.component.spec.js

@@ -1,8 +1,75 @@
 'use strict';
-
-// import main from './main.component';
-// import {MainController} from './main.component';
+import {
+  async,
+  ComponentFixture,
+  inject,
+  TestBed,
+} from '@angular/core/testing';
+import {
+  BaseRequestOptions,
+  ConnectionBackend,
+  Http,
+  HttpModule,
+  Response,
+  ResponseOptions,
+} from '@angular/http';
+import { MockBackend } from '@angular/http/testing';
+<%_ if(filters.mocha && filters.expect) { -%>
+import { expect } from 'chai';<% } %><% if(filters.uibootstrap) { %>
+import { TooltipModule } from 'ngx-bootstrap';<% } %>
+import { FormsModule } from '@angular/forms';
+import { SocketService } from '../../components/socket/socket.service';
+import { SocketServiceStub } from '../../components/socket/socket.mock';
+import { MainComponent } from './main.component';
 
 describe('Component: MainComponent', function() {
-  it('should attach a list of things to the controller', function() {});
+  let comp: MainComponent;
+  let fixture: ComponentFixture<MainComponent>;
+
+  beforeEach(async(() => {
+    TestBed.configureTestingModule({
+      imports: [
+        FormsModule,<% if(filters.uibootstrap) { %>
+        TooltipModule.forRoot(),<% } %>
+        HttpModule,
+      ],
+      declarations: [ MainComponent ], // declare the test component
+      providers: [
+        BaseRequestOptions,
+        MockBackend,
+        {
+          provide: Http,
+          useFactory: (backend: ConnectionBackend, defaultOptions: BaseRequestOptions) => {
+            return new Http(backend, defaultOptions);
+          },<%_ if(filters.ws) { %>
+          deps: [MockBackend, BaseRequestOptions]<% } %>
+        },
+        { provide: SocketService, useClass: SocketServiceStub },
+      ],
+    }).compileComponents();
+  }));
+  <%_ if(filters.ws) { %>
+  beforeEach(async(inject([MockBackend], (mockBackend) => {
+    mockBackend.connections.subscribe(conn => {
+      conn.mockRespond(new Response(new ResponseOptions({
+        body: JSON.stringify(['HTML5 Boilerplate', 'AngularJS', 'Karma', 'Express'])
+      })));
+    });
+  })));<% } %>
+
+  beforeEach(async(() => {
+    fixture = TestBed.createComponent(MainComponent);
+    // MainComponent test instance
+    comp = fixture.componentInstance;
+
+    /**
+     * Trigger initial data binding.
+     */
+    fixture.detectChanges();
+  }));
+
+  it('should attach a list of things to the controller', () => {<% if(filters.jasmine) { %>
+    expect(comp.awesomeThings.length).toEqual(4);<% } else if(filters.mocha) { -%>
+    <%= expect() %>comp.awesomeThings.length<%= to() %>.equal(4);<% } %>
+  });
 });

templates/app/client/components/oauth-buttons(oauth)/oauth-buttons.component.spec.js

@@ -2,7 +2,7 @@
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { By } from '@angular/platform-browser';
 import { DebugElement } from '@angular/core';
-<%_ if(filters.expect) { -%>
+<%_ if(filters.mocha && filters.expect) { -%>
 import { expect } from 'chai';<% } %>
 
 import { OauthButtonsComponent } from './oauth-buttons.component';

templates/app/client/components/socket(ws)/primus.mock.js

@@ -0,0 +1 @@
+export default class Primus {}

templates/app/client/components/socket(ws)/socket.mock.js

@@ -1,13 +1,7 @@
 'use strict';
-import { noop } from 'lodash-es';
 
-export class SocketServiceMock {
-    socket = {
-        connect: noop,
-        on: noop,
-        emit: noop,
-        receive: noop
-    };
-    syncUpdates = noop;
-    unsyncUpdates = noop;
+export class SocketServiceStub {
+  constructor() {}
+  syncUpdates() {}
+  unsyncUpdates() {}
 }

templates/app/client/components/socket(ws)/socket.service.js

@@ -1,5 +1,5 @@
 'use strict';
-import Primus from './primus';
+import Primus from 'primus';
 import primusEmit from 'primus-emit';
 import { Injectable } from '@angular/core';
 import { noop, find, remove } from 'lodash';