Merge pull request #798 from derrabauke/fix-customer-statistics-criteria

fix(customer-statistics): require fromDate param

Author: derrabauke

app/statistics/controller.js

@@ -20,7 +20,7 @@ const deserializeMoment = (momentString) =>
 const TYPES = {
   year: { include: "", requiredParams: [] },
   month: { include: "", requiredParams: [] },
-  customer: { include: "", requiredParams: [] },
+  customer: { include: "", requiredParams: ["fromDate"] },
   project: {
     include: "customer",
     requiredParams: ["customer"],

tests/acceptance/statistics-test.js

@@ -40,7 +40,7 @@ module("Acceptance | statistics", function (hooks) {
   });
 
   test("can view statistics by customer", async function (assert) {
-    await visit("/statistics?type=customer");
+    await visit("/statistics?type=customer&fromDate=1900-01-01");
 
     assert.dom("thead > tr > th").exists({ count: 3 });
     assert.dom("tbody > tr").exists({ count: 5 });