docs(testing): remove dis-used karma-htmlfile-reporter

also comments on the optional "testing" folder and corrects var names to match
also leaves Jasmine Spec Runner output visible when tests finish

Author: wardbell

public/docs/_examples/package.json

@@ -63,7 +63,6 @@
     "karma": "^1.3.0",
     "karma-chrome-launcher": "^2.0.0",
     "karma-cli": "^1.0.1",
-    "karma-htmlfile-reporter": "^0.3.4",
     "karma-jasmine": "^1.0.2",
     "karma-jasmine-html-reporter": "^0.2.2",
     "karma-phantomjs-launcher": "^1.0.2",

public/docs/_examples/testing/ts/app/banner.component.detect-changes.spec.ts

@@ -4,7 +4,7 @@
 import { async } from '@angular/core/testing';
 // #enddocregion import-async
 // #docregion import-ComponentFixtureAutoDetect
-import { ComponentFixtureAutoDetect} from '@angular/core/testing';
+import { ComponentFixtureAutoDetect } from '@angular/core/testing';
 // #enddocregion import-ComponentFixtureAutoDetect
 import { ComponentFixture, TestBed } from '@angular/core/testing';
 import { By }              from '@angular/platform-browser';

public/docs/_examples/testing/ts/karma-test-shim.js

@@ -36,7 +36,7 @@ var allSpecFiles = Object.keys(window.__karma__.files)
 
 System.config({
   baseURL: 'base',
-  // Extend usual application package list with test folder
+  // Extend usual application package list with testing folder
   packages: { 'testing': { main: 'index.js', defaultExtension: 'js' } },
 
   // Assume npm: is set in `paths` in systemjs.config

public/docs/_examples/testing/ts/karma.conf.js

@@ -5,6 +5,7 @@ module.exports = function(config) {
   var appSrcBase = 'app/';      // app source TS files
   var appAssets  = 'base/app/'; // component assets fetched by Angular's compiler
 
+  // Testing helpers (optional) are conventionally in a folder called `testing`
   var testingBase    = 'testing/'; // transpiled test JS and map files
   var testingSrcBase = 'testing/'; // test source TS files