improve collecting csv if the last one fails

Author: archmoj

test/jasmine/performance_tests/all_test.js

@@ -8,7 +8,6 @@ var nSamples = require('./assets/constants').nSamples;
 var gd = createGraphDiv();
 
 const samples = Array.from({ length: nSamples }, (_, i) => i);
-const nTimes = samples.length - 1;
 
 var tests = [];
 
@@ -51,7 +50,7 @@ for(let traceType of ['box', 'violin']) {
 for(let traceType of ['scatter', 'scattergl', 'scattergeo']) {
     for(let mode of ['markers', 'lines', 'markers+lines']) {
         for(let nTraces of [1, 10, 100]) {
-            for(let n of [1000, 2000, 4000, 8000, 16000, 32000, 64000]) {
+            for(let n of [1000, 2000, 4000, 8000, 16000, 32000]) {
                 tests.push({
                     n:n,
                     nTraces: nTraces,
@@ -266,14 +265,16 @@ function makeScatterGeo(spec) {
 }
 
 
-tests.forEach(function(spec, index) {
-    describe('Performance test ' + spec.nTraces + ' ' + spec.traceType + (spec.mode ? ' | mode: ' + spec.mode : '') + ' | size:' + spec.n, function() {
-        'use strict';
+describe('Performance test various traces', function() {
+    'use strict';
 
-        afterEach(function(done) {
-            delay(100)().then(done);
-        });
+    afterAll(function(done) {
+        downloadCSV(tests);
+        // delay for the download to be completed
+        delay(1000)().then(done)
+    });
 
+    tests.forEach(function(spec, index) {
         samples.forEach(function(t) {
             it('turn: ' + t, function(done) {
                 var startTime, endTime;
@@ -295,10 +296,6 @@ tests.forEach(function(spec, index) {
                             expect(nodes.size()).toEqual(spec.nTraces);
                         }
 
-                        if(t === nTimes && index === tests.length - 1) {
-                            downloadCSV(tests);
-                        }
-
                         done();
                     });
                 });

test/jasmine/performance_tests/assets/post_process.js

@@ -6,16 +6,19 @@ exports.downloadCSV = function(allTests) {
         'run id',
         'rendering time(ms)'
     ].join(',') + '\n';
+
     for(var k = 0; k < allTests.length; k++) {
         var test = allTests[k];
 
-        for(var i = 0; i < test.raw.length; i++) {
+        var raw = test.raw || [];
+
+        for(var i = 0; i < raw.length; i++) {
             str += [
                 (test.nTraces || 1),
                 (test.traceType + (test.mode ? ' ' + test.mode : '')),
                 test.n,
                 i,
-                test.raw[i]
+                raw[i]
             ].join(',') + '\n';
         }
     }