From 94a594aee4b04497c9d7ba3a535737422c1cd423 Mon Sep 17 00:00:00 2001
From: Taranveer Virk <taranveer@virk.cc>
Date: Fri, 1 Sep 2017 17:03:30 -0400
Subject: [PATCH] Apply feedback

Move helper functions to bottom of file.
---
 .../repository-mixin/repository-mixin.test.ts | 90 +++++++++----------
 1 file changed, 45 insertions(+), 45 deletions(-)

diff --git a/packages/repository/test/unit/repository-mixin/repository-mixin.test.ts b/packages/repository/test/unit/repository-mixin/repository-mixin.test.ts
index fcdfe0cb7f64..6201d6e3cff8 100644
--- a/packages/repository/test/unit/repository-mixin/repository-mixin.test.ts
+++ b/packages/repository/test/unit/repository-mixin/repository-mixin.test.ts
@@ -15,51 +15,6 @@ import {Application, Component} from '@loopback/core';
 // tslint:disable:no-any
 
 describe('RepositoryMixin', () => {
-  const appWithRepoMixin = RepositoryMixin(Application);
-
-  class NoteRepo {
-    model: any;
-
-    constructor() {
-      const ds: juggler.DataSource = new DataSourceConstructor({
-        name: 'db',
-        connector: 'memory',
-      });
-
-      this.model = ds.createModel(
-        'note',
-        {title: 'string', content: 'string'},
-        {},
-      );
-    }
-  }
-
-  class TestComponent {
-    repositories = [NoteRepo];
-  }
-
-  function expectNoteRepoToBeBound(myApp: Application) {
-    const boundRepositories = myApp.find('repositories.*').map(b => b.key);
-    expect(boundRepositories).to.containEql('repositories.NoteRepo');
-    const repoInstance = myApp.getSync('repositories.NoteRepo');
-    expect(repoInstance).to.be.instanceOf(NoteRepo);
-  }
-
-  function expectNoteRepoToNotBeBound(myApp: Application) {
-    const boundRepos = myApp.find('repositories.*').map(b => b.key);
-    expect(boundRepos).to.be.empty();
-  }
-
-  function expectComponentToBeBound(
-    myApp: Application,
-    component: Class<Component>,
-  ) {
-    const boundComponents = myApp.find('components.*').map(b => b.key);
-    expect(boundComponents).to.containEql(`components.${component.name}`);
-    const componentInstance = myApp.getSync(`components.${component.name}`);
-    expect(componentInstance).to.be.instanceOf(component);
-  }
-
   it('mixed class has .repository()', () => {
     const myApp = new appWithRepoMixin();
     expect(typeof myApp.repository).to.be.eql('function');
@@ -111,4 +66,49 @@ describe('RepositoryMixin', () => {
     expectComponentToBeBound(myApp, TestComponent);
     expectNoteRepoToBeBound(myApp);
   });
+
+  const appWithRepoMixin = RepositoryMixin(Application);
+
+  class NoteRepo {
+    model: any;
+
+    constructor() {
+      const ds: juggler.DataSource = new DataSourceConstructor({
+        name: 'db',
+        connector: 'memory',
+      });
+
+      this.model = ds.createModel(
+        'note',
+        {title: 'string', content: 'string'},
+        {},
+      );
+    }
+  }
+
+  class TestComponent {
+    repositories = [NoteRepo];
+  }
+
+  function expectNoteRepoToBeBound(myApp: Application) {
+    const boundRepositories = myApp.find('repositories.*').map(b => b.key);
+    expect(boundRepositories).to.containEql('repositories.NoteRepo');
+    const repoInstance = myApp.getSync('repositories.NoteRepo');
+    expect(repoInstance).to.be.instanceOf(NoteRepo);
+  }
+
+  function expectNoteRepoToNotBeBound(myApp: Application) {
+    const boundRepos = myApp.find('repositories.*').map(b => b.key);
+    expect(boundRepos).to.be.empty();
+  }
+
+  function expectComponentToBeBound(
+    myApp: Application,
+    component: Class<Component>,
+  ) {
+    const boundComponents = myApp.find('components.*').map(b => b.key);
+    expect(boundComponents).to.containEql(`components.${component.name}`);
+    const componentInstance = myApp.getSync(`components.${component.name}`);
+    expect(componentInstance).to.be.instanceOf(component);
+  }
 });