From 34a2f3b32bde01c847683fa3c8faea7874487680 Mon Sep 17 00:00:00 2001 From: Javier Goizueta Date: Mon, 28 May 2018 16:50:53 +0200 Subject: [PATCH] Tests were missing in previous commit --- .../stats/mapnik_stats_layergroup.js | 27 ++++++++++++++++++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/test/acceptance/stats/mapnik_stats_layergroup.js b/test/acceptance/stats/mapnik_stats_layergroup.js index 46c4368a..4e8413ae 100644 --- a/test/acceptance/stats/mapnik_stats_layergroup.js +++ b/test/acceptance/stats/mapnik_stats_layergroup.js @@ -513,7 +513,7 @@ describe('Create mapnik layergroup', function() { version: '1.4.0', layers: [ layerWithMetadata(mapnikLayer4, { - sample: 3 + sample: { num_rows: 3 } }) ] }); @@ -529,6 +529,31 @@ describe('Create mapnik layergroup', function() { }); }); + it('can specify sample columns', function(done) { + var testClient = new TestClient({ + version: '1.4.0', + layers: [ + layerWithMetadata(mapnikLayer4, { + sample: { + num_rows: 3, + include_columns: [ 'cartodb_id', 'address', 'the_geom' ] + } + }) + ] + }); + + testClient.getLayergroup(function(err, layergroup) { + assert.ifError(err); + assert.equal(layergroup.metadata.layers[0].id, mapnikBasicLayerId(0)); + assert.equal(layergroup.metadata.layers[0].meta.stats.estimatedFeatureCount, 5); + assert(layergroup.metadata.layers[0].meta.stats.sample.length > 0); + const expectedCols = [ 'cartodb_id', 'address', 'the_geom' ].sort(); + assert.deepEqual(Object.keys(layergroup.metadata.layers[0].meta.stats.sample[0]).sort(), expectedCols); + testClient.drain(done); + }); + }); + + it('should only provide requested optional metadata', function(done) { var testClient = new TestClient({ version: '1.4.0',