Merge pull request #4696 from runspired/bench/query-materialization

fix(benchmarks): benchmarks for store.query no longer included record…

addon/-private/system/model/internal-model.js

@@ -263,6 +263,7 @@ export default class InternalModel {
   getRecord() {
     if (!this._record) {
       heimdall.increment(materializeRecord);
+      let token = heimdall.start('InternalModel.getRecord');
 
       // lookupFactory should really return an object that creates
       // instances with the injections applied
@@ -285,6 +286,7 @@ export default class InternalModel {
       this._record = this.modelClass._create(createOptions);
 
       this._triggerDeferredTriggers();
+      heimdall.stop(token);
     }
 
     return this._record;

tests/dummy/app/routes/query/route.js

@@ -27,6 +27,11 @@ export default Route.extend({
     let token = heimdall.start('ember-data');
     return this.get('store').query(modelName, params)
       .then((records) => {
+        // RecordArray lazily materializes the records
+        // We call toArray() to force materialization for benchmarking
+        // otherwise we would need to consume the RecordArray in our UI
+        // and clutter our benchmarks and make it harder to time.
+        records.toArray();
         heimdall.stop(token);
         window.result = heimdall.toString();