Merge pull request #44 from codyjdalton/compiler-scope

compiler-scope: update tests
codyjdalton · May 11, 2018 · 5221e11 · 5221e11
2 parents b71e642 + 9f0b1cf
commit 5221e11
Show file tree

Hide file tree

Showing 6 changed files with 197 additions and 390 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -4,7 +4,7 @@ node_js :
 install:
  - npm install
 script:
- - npm test
+ - npm run build
 
  # Send coverage data to Coveralls
 after_script: "cat coverage/lcov.info | node_modules/coveralls/bin/coveralls.js"
diff --git a/README.md b/README.md
@@ -194,7 +194,6 @@ export class PeopleComponent {
         produces: ResourceVersions.PEOPLE_V1 // Content-Type header
     })
     getPeople(req: HttpRequest, res: HttpResponse): void  {
-        // get the list of all people
         this.personService.fetchAll()
             .subscribe(
                 (people: Person[]) => res.success(people),
@@ -211,7 +210,6 @@ export class PeopleComponent {
         produces: ResourceVersions.PEOPLE_V1 // Content-Type header
     })
     getPerson(req: HttpRequest, res: HttpResponse): void  {
-        // get the list of all people
         this.personService.fetchById(req.params.id)
             .subscribe(
                 (people: Person[]) => res.success(people),
@@ -228,7 +226,6 @@ export class PeopleComponent {
         produces: ResourceVersions.PERSON_V1 // Content-Type header
     })
     updatePerson(req: HttpRequest, res: HttpResponse): void  {
-        // update person
         this.personService.update(req.params.id, req.body)
             .subscribe(
                 (person: Person) => res.success(person),