fix(projectAssignment): Added a default query value to me method on p…

…roject assignments. Fix #114
simplyspoke · Sep 5, 2018 · 378cacb · 378cacb
1 parent 44f0fd4
commit 378cacb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/api/projectAssignments.spec.ts b/src/api/projectAssignments.spec.ts
@@ -34,12 +34,11 @@ describe('ProjectAssignments test', () => {
   });
 
   it('should have a me method that calls the request method', () => {
-    const query = { is_active: true };
-    instance.me(query);
+    instance.me();
     expect(request).toBeCalledWith(
       'GET',
       '/v2/users/me/project_assignments',
-      query
+      {}
     );
   });
 });
diff --git a/src/api/projectAssignments.ts b/src/api/projectAssignments.ts
@@ -21,8 +21,7 @@ export class ProjectAssignmentsAPI {
   }
 
   // List all project assignments for the currently authenticated user
-  // TODO: COnsider handling defaults
-  public me(query: PagenationParameters) {
+  public me(query: PagenationParameters = {}) {
     return this.harvest.request(
       'GET',
       `${this.baseUrl}/me/project_assignments`,