Apply feedback

Author: virkt25

.vscode/tasks.json

@@ -5,8 +5,8 @@
   "tasks": [
     {
       "taskName": "Watch and compile TypeScript",
-      "command": "tsc",
-      "args": ["--watch"],
+      "command": "npm",
+      "args": ["--silent", "run", "build:watch"],
       "type": "process",
       "isBackground": true,
       "problemMatcher": "$tsc-watch",
@@ -24,11 +24,7 @@
     {
       "taskName": "Test and lint",
       "command": "npm",
-      "args": [
-        "--silent",
-        "run",
-        "vscode-test"
-      ],
+      "args": ["--silent", "run", "test:dev"],
       "type": "process",
       "group": {
         "kind": "test",

package.json

@@ -10,14 +10,15 @@
     "build": "npm run build:lib && npm run build:lib6",
     "build:lib": "tsc --target es2017 --outDir dist",
     "build:lib6": "tsc --target es2015 --outDir dist6",
+    "build:watch": "tsc --watch",
     "clean": "rm -rf dist dist6",
     "lint": "tslint -c tslint.full.json --project tsconfig.json --type-check",
     "lint:fix": "npm run lint -- --fix",
     "prepublish": "npm run build",
     "pretest": "npm run clean && npm run build",
     "test": "mocha",
     "posttest": "npm run lint",
-    "vscode-test": "mocha && npm run lint"
+    "test:dev": "mocha && npm run lint"
   },
   "repository": {
     "type": "git",
@@ -41,7 +42,6 @@
     "@loopback/testlab": "^4.0.0-alpha.7",
     "@types/mocha": "^2.2.43",
     "mocha": "^3.5.3",
-    "ts-node": "^3.3.0",
     "tslint": "^5.7.0",
     "typescript": "^2.5.2"
   }

src/controllers/README.md

@@ -0,0 +1,3 @@
+# Controllers
+
+This directory contains source files for the controllers exported by this extension.

src/index.ts

@@ -3,4 +3,4 @@
 // This file is licensed under the MIT License.
 // License text available at https://opensource.org/licenses/MIT
 
-export * from './logger-mixin';
+export * from './mixins/logger.mixin';

src/logger-mixin/index.ts

@@ -22,7 +22,7 @@ cosnt newClass = MixinClass1(MixinClass2(BaseClass))
 
 ### LoggerMixin
 
-LoggerMixin adds capabilities to a LoopBack Next Application by adding a `.logger()` function that allows used to bind a Logger class to `Context` automatically. The binding key will be `loggers.${Class.name}` where `Class.name` is the name of the Logger class being bound. The Mixin also overrides existing `.component()` function so that components are also capable of providing Logger's to be bound automatically. 
+LoggerMixin adds capabilities to a LoopBack Next Application by adding a `.logger()` function that is used to bind a Logger class to `Context` automatically. The binding key will be `loggers.${Class.name}` where `Class.name` is the name of the Logger class being bound. Components are also able to provide their own Logger implementation which will be bound via `.component()` automatically when using this Mixin.
 
 **Example**
 ```
@@ -57,7 +57,7 @@ class ColorLogger {
 
 Once a Logger has been bound, you can retrieve it by using [Dependency Inject](http://loopback.io/doc/en/lb4/Dependency-injection.html)
 
-**More Examples for binding a Logger**
+#### More Examples for binding a Logger**
 ```
 // Using the app's .logger() function. 
 class LoggingApplication extends LoggerMixin(Application) {
@@ -80,20 +80,3 @@ const app = new LoggingApplication({
   components: [MyComponent] // Logger from MyComponent will be bound to loggers.ColorLogger
 });
 ```
-
-## Contributions
-
-- [Guidelines](http://loopback.io/doc/en/contrib/index.html)
-- [Join the team](https://github.com/strongloop/loopback-next/issues/110)
-
-## Tests
-
-Run `npm test` from the root folder.
-
-## Contributors
-
-See [all contributors](https://github.com/strongloop/loopback-next-extension-starter/graphs/contributors).
-
-## License
-
-MIT

src/logger-mixin/mixins/README.md renamed to src/mixins/README.md

@@ -29,13 +29,6 @@ export function LoggerMixin<T extends Constructor<any>>(superClass: T) {
           this.logger(logger);
         }
       }
-
-      if (this.options.components) {
-        // Super would have already mounted the component
-        for (const component of this.options.components) {
-          this.component(component);
-        }
-      }
     }
 
     /**
@@ -61,18 +54,17 @@ export function LoggerMixin<T extends Constructor<any>>(superClass: T) {
 
     /**
     * Add a component to this application. Also mounts
-    * all the components repositories.
+    * all the components Loggers.
     *
     * @param component The component to add.
     *
     * ```ts
     *
     * export class ProductComponent {
     *   controllers = [ProductController];
-    *   loggers = [Logger];
+    *   loggers = [ProductLogger];
     *   providers = {
-    *     [AUTHENTICATION_STRATEGY]: AuthStrategy,
-    *     [AUTHORIZATION_ROLE]: Role,
+    *     [PRODUCT_PROVIDER]: ProductProvider,
     *   };
     * };
     *
@@ -89,7 +81,7 @@ export function LoggerMixin<T extends Constructor<any>>(superClass: T) {
     * loggers. This function is intended to be used internally
     * by component()
     *
-    * @param component The component to mount repositories of
+    * @param component The component to mount Logger's of
     */
     mountComponentLogger(component: Constructor<any>) {
       const componentKey = `components.${component.name}`;

src/logger-mixin/mixins/logger-mixin.ts renamed to src/mixins/logger.mixin.ts

@@ -1,3 +1,3 @@
---compilers ts:ts-node/register
 --recursive
 --reporter dot
+dist/test

test/mocha.opts

@@ -30,16 +30,6 @@ describe('LoggerMixin', () => {
     expectLoggerToBeBound(myApp);
   });
 
-  it('binds user defined component without Logger', () => {
-    class EmptyTestComponent {}
-
-    const myApp = new AppWithLogger({
-      components: [EmptyTestComponent],
-    });
-
-    expectComponentToBeBound(myApp, EmptyTestComponent);
-  });
-
   it('binds user defined component with Logger in constructor', () => {
     const myApp = new AppWithLogger({
       loggers: [MyLogger],
@@ -66,10 +56,7 @@ describe('LoggerMixin', () => {
     });
 
     const logger = myApp.getSync('loggers.MyLogger');
-    expect(typeof logger.log).to.be.eql('function');
-    expect(typeof logger.info).to.be.eql('function');
-    expect(typeof logger.warn).to.be.eql('function');
-    expect(typeof logger.error).to.be.eql('function');
+    expect(logger).to.be.instanceOf(MyLogger);
   });
 
   class AppWithLogger extends LoggerMixin(Application) {}

test/unit/logger-mixin.test.ts

@@ -9,7 +9,7 @@
     "lib": ["es2017", "dom"],
     "module": "commonjs",
     "moduleResolution": "node",
-    "target": "es6",
+    "target": "es2017",
     "outDir": "dist",
     "sourceMap": true,
     "declaration": true