diff --git a/test/integration/metrics-test.js b/test/integration/metrics-test.js index 52d345a7..c1b75eec 100644 --- a/test/integration/metrics-test.js +++ b/test/integration/metrics-test.js @@ -18,12 +18,27 @@ const mapConfig = { ] }; +const mapConfigWithTable = { + version: '1.8.0', + layers: [ + { + options: { + sql: 'select * from test_table', + cartocss: TestClient.CARTOCSS.POINTS, + cartocss_version: '2.3.0' + } + } + ] +}; + function templateBuilder ({ name }) { + const templateName = `metrics-template-${name}-${Date.now()}`; + return { version: '0.0.1', - name: `metrics-template-${name}`, + name: templateName, layergroup: { - stat_tag: `stat-tag-${name}`, + stat_tag: `stat-tag-${templateName}`, version: '1.8.0', layers: [ { @@ -37,514 +52,532 @@ function templateBuilder ({ name }) { ] } }; -}; +} -describe('metrics', function () { - beforeEach(function () { - this.originalMetricsBackendSendMethod = MetricsBackend.prototype.send; - this.pubSubMetricsBackendSendMethodCalled = false; - MetricsBackend.prototype.send = (event, attributes) => { - this.pubSubMetricsBackendSendMethodCalled = true; - this.pubSubMetricsBackendSendMethodCalledWith = { event, attributes }; - return Promise.resolve(); - }; - }); +function templateMissingCartoCSSVersionBuilder () { + const templateName = `missing-cartocss-version-${Date.now()}`; - afterEach(function (done) { - MetricsBackend.prototype.send = this.originalMetricsBackendSendMethod; - return this.testClient.drain(done); - }); + return { + version: '0.0.1', + name: templateName, + layergroup: { + stat_tag: `stat-tag-${templateName}`, + version: '1.8.0', + layers: [ + { + type: 'cartodb', + options: { + sql: TestClient.SQL.ONE_POINT, + cartocss: TestClient.CARTOCSS.POINTS + } + } + ] + } + }; +} + +const suites = [ + { + desc: 'map config with live query', + mapConfig + }, + { + desc: 'map config with query against table', + mapConfig: mapConfigWithTable + } +]; + +suites.forEach(function ({ desc, mapConfig }) { + describe(`metrics: ${desc}`, function () { + beforeEach(function () { + this.originalMetricsBackendSendMethod = MetricsBackend.prototype.send; + this.pubSubMetricsBackendSendMethodCalled = false; + MetricsBackend.prototype.send = (event, attributes) => { + this.pubSubMetricsBackendSendMethodCalled = true; + this.pubSubMetricsBackendSendMethodCalledWith = { event, attributes }; + return Promise.resolve(); + }; + }); - it('should not send event if not enabled', function (done) { - const extraHeaders = { - 'Carto-Event': 'test-event', - 'Carto-Event-Source': 'test', - 'Carto-Event-Group-Id': '1' - }; - const overrideServerOptions = { pubSubMetrics: { enabled: false } }; + afterEach(function (done) { + MetricsBackend.prototype.send = this.originalMetricsBackendSendMethod; + return this.testClient.drain(done); + }); - this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); + it('should not send event if not enabled', function (done) { + const extraHeaders = { + 'Carto-Event': 'test-event', + 'Carto-Event-Source': 'test', + 'Carto-Event-Group-Id': '1' + }; + const overrideServerOptions = { pubSubMetrics: { enabled: false } }; - this.testClient.getLayergroup((err, body) => { - if (err) { - return done(err); - } + this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); + + this.testClient.getLayergroup((err, body) => { + if (err) { + return done(err); + } - assert.strictEqual(typeof body.layergroupid, 'string'); - assert.ok(!this.pubSubMetricsBackendSendMethodCalled); + assert.strictEqual(typeof body.layergroupid, 'string'); + assert.ok(!this.pubSubMetricsBackendSendMethodCalled); - return done(); + return done(); + }); }); - }); - it('should not send event if headers not present', function (done) { - const extraHeaders = {}; - const overrideServerOptions = { pubSubMetrics: { enabled: false } }; + it('should not send event if headers not present', function (done) { + const extraHeaders = {}; + const overrideServerOptions = { pubSubMetrics: { enabled: false } }; - this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); + this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); - this.testClient.getLayergroup((err, body) => { - if (err) { - return done(err); - } + this.testClient.getLayergroup((err, body) => { + if (err) { + return done(err); + } - assert.strictEqual(typeof body.layergroupid, 'string'); - assert.ok(!this.pubSubMetricsBackendSendMethodCalled); + assert.strictEqual(typeof body.layergroupid, 'string'); + assert.ok(!this.pubSubMetricsBackendSendMethodCalled); - return done(); + return done(); + }); }); - }); - it('should send event for map requests', function (done) { - const expectedEvent = 'map_view'; - const expectedMetricsEvent = 'event-test'; - const expectedEventSource = 'event-source-test'; - const expectedEventGroupId = '1'; - const expectedResponseCode = '200'; - const expectedMapType = 'anonymous'; - const extraHeaders = { - 'Carto-Event': expectedMetricsEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - - this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); - - this.testClient.getLayergroup((err, body) => { - if (err) { - return done(err); - } + it('should send event for map requests', function (done) { + const expectedEvent = 'map_view'; + const expectedMetricsEvent = 'event-test'; + const expectedEventSource = 'event-source-test'; + const expectedEventGroupId = '1'; + const expectedResponseCode = '200'; + const expectedMapType = 'anonymous'; + const extraHeaders = { + 'Carto-Event': expectedMetricsEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + + this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); + + this.testClient.getLayergroup((err, body) => { + if (err) { + return done(err); + } - assert.strictEqual(typeof body.layergroupid, 'string'); + assert.strictEqual(typeof body.layergroupid, 'string'); - const { token, cacheBuster } = LayergroupToken.parse(body.layergroupid); + const { token, cacheBuster } = LayergroupToken.parse(body.layergroupid); - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.client_event, expectedMetricsEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(attributes.map_type, expectedMapType); - assert.strictEqual(attributes.map_id, token); - assert.strictEqual(attributes.cache_buster, cacheBuster); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.client_event, expectedMetricsEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(attributes.map_type, expectedMapType); + assert.strictEqual(attributes.map_id, token); + assert.strictEqual(attributes.cache_buster, cacheBuster); - return done(); + return done(); + }); }); - }); - it('should normalized headers type and length', function (done) { - const expectedEvent = 'map_view'; - const eventLong = 'If you are sending a text this long in a header you kind of deserve the worst, honestly. I mean this is not a header, it is almost a novel, and you do not see any Novel cookie here, right?'; - const expectedMetricsEvent = eventLong.trim().substr(0, 100); - const expectedEventGroupId = '1'; - const expectedEventSource = 'test'; - const expectedResponseCode = '200'; - const expectedMapType = 'anonymous'; - const extraHeaders = { - 'Carto-Event': eventLong, - 'Carto-Event-Source': 'test', - 'Carto-Event-Group-Id': 1 - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - - this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); - - this.testClient.getLayergroup((err, body) => { - if (err) { - return done(err); - } + it('should normalized headers type and length', function (done) { + const expectedEvent = 'map_view'; + const eventLong = 'If you are sending a text this long in a header you kind of deserve the worst, honestly. I mean this is not a header, it is almost a novel, and you do not see any Novel cookie here, right?'; + const expectedMetricsEvent = eventLong.trim().substr(0, 100); + const expectedEventGroupId = '1'; + const expectedEventSource = 'test'; + const expectedResponseCode = '200'; + const expectedMapType = 'anonymous'; + const extraHeaders = { + 'Carto-Event': eventLong, + 'Carto-Event-Source': 'test', + 'Carto-Event-Group-Id': 1 + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + + this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); + + this.testClient.getLayergroup((err, body) => { + if (err) { + return done(err); + } - assert.strictEqual(typeof body.layergroupid, 'string'); + assert.strictEqual(typeof body.layergroupid, 'string'); - const { token, cacheBuster } = LayergroupToken.parse(body.layergroupid); + const { token, cacheBuster } = LayergroupToken.parse(body.layergroupid); - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.client_event, expectedMetricsEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(attributes.map_type, expectedMapType); - assert.strictEqual(attributes.map_id, token); - assert.strictEqual(attributes.cache_buster, cacheBuster); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.client_event, expectedMetricsEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(attributes.map_type, expectedMapType); + assert.strictEqual(attributes.map_id, token); + assert.strictEqual(attributes.cache_buster, cacheBuster); - return done(); + return done(); + }); }); - }); - it('should send event when error', function (done) { - const expectedEvent = 'map_view'; - const expectedMetricsEvent = 'event-test'; - const expectedEventSource = 'event-source-test'; - const expectedEventGroupId = '1'; - const expectedResponseCode = '400'; - const expectedMapType = 'anonymous'; - const extraHeaders = { - 'Carto-Event': expectedMetricsEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - const mapConfigMissingCartoCSS = { - version: '1.8.0', - layers: [ - { - options: { - sql: TestClient.SQL.ONE_POINT, - cartocss: TestClient.CARTOCSS.POINTS + it('should send event when error', function (done) { + const expectedEvent = 'map_view'; + const expectedMetricsEvent = 'event-test'; + const expectedEventSource = 'event-source-test'; + const expectedEventGroupId = '1'; + const expectedResponseCode = '400'; + const expectedMapType = 'anonymous'; + const extraHeaders = { + 'Carto-Event': expectedMetricsEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + const mapConfigMissingCartoCSS = { + version: '1.8.0', + layers: [ + { + options: { + sql: TestClient.SQL.ONE_POINT, + cartocss: TestClient.CARTOCSS.POINTS + } } - } - ] - }; + ] + }; - this.testClient = new TestClient(mapConfigMissingCartoCSS, apikey, extraHeaders, overrideServerOptions); + this.testClient = new TestClient(mapConfigMissingCartoCSS, apikey, extraHeaders, overrideServerOptions); - const params = { response: { status: 400 } }; + const params = { response: { status: 400 } }; - this.testClient.getLayergroup(params, (err, body) => { - if (err) { - return done(err); - } + this.testClient.getLayergroup(params, (err, body) => { + if (err) { + return done(err); + } - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.client_event, expectedMetricsEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(attributes.map_type, expectedMapType); - assert.strictEqual(typeof attributes.map_id, 'string'); - assert.strictEqual(typeof attributes.cache_buster, 'string'); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.client_event, expectedMetricsEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(attributes.map_type, expectedMapType); + assert.strictEqual(typeof attributes.map_id, 'string'); + assert.strictEqual(typeof attributes.cache_buster, 'string'); - return done(); + return done(); + }); }); - }); - it.skip('should send event for tile requests', function (done) { - const expectedEvent = 'event-tile-test'; - const expectedEventSource = 'event-source-tile-test'; - const expectedEventGroupId = '12345'; - const expectedResponseCode = '200'; - const extraHeaders = { - 'Carto-Event': expectedEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - - this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); - - this.testClient.getTile(0, 0, 0, (err, res, tile) => { - if (err) { - return done(err); - } + it.skip('should send event for tile requests', function (done) { + const expectedEvent = 'event-tile-test'; + const expectedEventSource = 'event-source-tile-test'; + const expectedEventGroupId = '12345'; + const expectedResponseCode = '200'; + const extraHeaders = { + 'Carto-Event': expectedEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + + this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); + + this.testClient.getTile(0, 0, 0, (err, res, tile) => { + if (err) { + return done(err); + } - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(typeof attributes.map_id, 'string'); - assert.strictEqual(typeof attributes.cache_buster, 'string'); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(typeof attributes.map_id, 'string'); + assert.strictEqual(typeof attributes.cache_buster, 'string'); - return done(); + return done(); + }); }); - }); - it.skip('should send event for errored tile requests', function (done) { - const expectedEvent = 'event-tile-test'; - const expectedEventSource = 'event-source-tile-test'; - const expectedEventGroupId = '12345'; - const expectedResponseCode = '400'; - const extraHeaders = { - 'Carto-Event': expectedEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - - this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); - - const params = { - response: { - status: 400, - headers: { - 'Content-Type': 'application/json; charset=utf-8' + it.skip('should send event for errored tile requests', function (done) { + const expectedEvent = 'event-tile-test'; + const expectedEventSource = 'event-source-tile-test'; + const expectedEventGroupId = '12345'; + const expectedResponseCode = '400'; + const extraHeaders = { + 'Carto-Event': expectedEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + + this.testClient = new TestClient(mapConfig, apikey, extraHeaders, overrideServerOptions); + + const params = { + response: { + status: 400, + headers: { + 'Content-Type': 'application/json; charset=utf-8' + } } - } - }; + }; - this.testClient.getTile(0, 0, 2, params, (err, res, tile) => { - if (err) { - return done(err); - } + this.testClient.getTile(0, 0, 2, params, (err, res, tile) => { + if (err) { + return done(err); + } - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(typeof attributes.map_id, 'string'); - assert.strictEqual(typeof attributes.cache_buster, 'string'); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(typeof attributes.map_id, 'string'); + assert.strictEqual(typeof attributes.cache_buster, 'string'); - return done(); + return done(); + }); }); - }); - it('should send event for named map requests', function (done) { - const expectedEvent = 'map_view'; - const expectedMetricsEvent = 'event-test'; - const expectedEventSource = 'event-source-test'; - const expectedEventGroupId = '1'; - const expectedResponseCode = '200'; - const expectedMapType = 'named'; - const extraHeaders = { - 'Carto-Event': expectedMetricsEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - const template = templateBuilder({ name: 'map' }); - - this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); - - this.testClient.getLayergroup((err, body) => { - if (err) { - return done(err); - } + it('should send event for named map requests', function (done) { + const expectedEvent = 'map_view'; + const expectedMetricsEvent = 'event-test'; + const expectedEventSource = 'event-source-test'; + const expectedEventGroupId = '1'; + const expectedResponseCode = '200'; + const expectedMapType = 'named'; + const extraHeaders = { + 'Carto-Event': expectedMetricsEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + const template = templateBuilder({ name: 'map' }); + + this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); + + this.testClient.getLayergroup((err, body) => { + if (err) { + return done(err); + } - assert.strictEqual(typeof body.layergroupid, 'string'); + assert.strictEqual(typeof body.layergroupid, 'string'); - const { token, cacheBuster, templateHash } = LayergroupToken.parse(body.layergroupid); + const { token, cacheBuster, templateHash } = LayergroupToken.parse(body.layergroupid); - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.client_event, expectedMetricsEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(attributes.map_type, expectedMapType); - assert.strictEqual(attributes.map_id, token); - assert.strictEqual(attributes.cache_buster, cacheBuster); - assert.strictEqual(attributes.template_hash, templateHash); - assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.client_event, expectedMetricsEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(attributes.map_type, expectedMapType); + assert.strictEqual(attributes.map_id, token); + assert.strictEqual(attributes.cache_buster, cacheBuster); + assert.strictEqual(attributes.template_hash, templateHash); + assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); - return done(); + return done(); + }); }); - }); - it('should send event for errored named map requests', function (done) { - const expectedEvent = 'map_view'; - const expectedMetricsEvent = 'event-test'; - const expectedEventSource = 'event-source-test'; - const expectedEventGroupId = '1'; - const expectedResponseCode = '400'; - const expectedMapType = 'named'; - const extraHeaders = { - 'Carto-Event': expectedMetricsEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - const templateMissingCartoCSSVersion = { - version: '0.0.1', - name: 'metrics-template-missing-cartocss-version', - layergroup: { - stat_tag: 'stat-tag-missing-cartocss-version', - version: '1.8.0', - layers: [ - { - type: 'cartodb', - options: { - sql: TestClient.SQL.ONE_POINT, - cartocss: TestClient.CARTOCSS.POINTS - } + it('should send event for errored named map requests', function (done) { + const expectedEvent = 'map_view'; + const expectedMetricsEvent = 'event-test'; + const expectedEventSource = 'event-source-test'; + const expectedEventGroupId = '1'; + const expectedResponseCode = '400'; + const expectedMapType = 'named'; + const extraHeaders = { + 'Carto-Event': expectedMetricsEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + const templateMissingCartoCSSVersion = templateMissingCartoCSSVersionBuilder(); + this.testClient = new TestClient(templateMissingCartoCSSVersion, apikey, extraHeaders, overrideServerOptions); + + const params = { + response: { + status: 400, + headers: { + 'Content-Type': 'application/json; charset=utf-8' } - ] - } - }; - - this.testClient = new TestClient(templateMissingCartoCSSVersion, apikey, extraHeaders, overrideServerOptions); - - const params = { - response: { - status: 400, - headers: { - 'Content-Type': 'application/json; charset=utf-8' } - } - }; + }; - this.testClient.getLayergroup(params, (err, body) => { - if (err) { - return done(err); - } + this.testClient.getLayergroup(params, (err, body) => { + if (err) { + return done(err); + } - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.client_event, expectedMetricsEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(attributes.map_type, expectedMapType); - assert.strictEqual(typeof attributes.map_id, 'string'); - assert.strictEqual(typeof attributes.cache_buster, 'string'); - assert.strictEqual(typeof attributes.template_hash, 'string'); - assert.strictEqual(attributes.stat_tag, templateMissingCartoCSSVersion.layergroup.stat_tag); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.client_event, expectedMetricsEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(attributes.map_type, expectedMapType); + assert.strictEqual(typeof attributes.map_id, 'string'); + assert.strictEqual(typeof attributes.cache_buster, 'string'); + assert.strictEqual(typeof attributes.template_hash, 'string'); + assert.strictEqual(attributes.stat_tag, templateMissingCartoCSSVersion.layergroup.stat_tag); - return done(); + return done(); + }); }); - }); - it.skip('should send event for named map tile requests', function (done) { - const expectedEvent = 'event-named-map-tile-test'; - const expectedEventSource = 'event-source-named-map-tile-test'; - const expectedEventGroupId = '1'; - const expectedResponseCode = '200'; - const extraHeaders = { - 'Carto-Event': expectedEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - const template = templateBuilder({ name: 'tile' }); - - this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); - - this.testClient.getTile(0, 0, 0, (err, body) => { - if (err) { - return done(err); - } + it.skip('should send event for named map tile requests', function (done) { + const expectedEvent = 'event-named-map-tile-test'; + const expectedEventSource = 'event-source-named-map-tile-test'; + const expectedEventGroupId = '1'; + const expectedResponseCode = '200'; + const extraHeaders = { + 'Carto-Event': expectedEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + const template = templateBuilder({ name: 'tile' }); + + this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); + + this.testClient.getTile(0, 0, 0, (err, body) => { + if (err) { + return done(err); + } - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(typeof attributes.map_id, 'string'); - assert.strictEqual(typeof attributes.cache_buster, 'string'); - assert.strictEqual(typeof attributes.template_hash, 'string'); - assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(typeof attributes.map_id, 'string'); + assert.strictEqual(typeof attributes.cache_buster, 'string'); + assert.strictEqual(typeof attributes.template_hash, 'string'); + assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); - return done(); + return done(); + }); }); - }); - it('should send event for static named map requests', function (done) { - const expectedEvent = 'map_view'; - const expectedMetricsEvent = 'event-test'; - const expectedEventSource = 'event-source-test'; - const expectedEventGroupId = '1'; - const expectedResponseCode = '200'; - const expectedMapType = 'static'; - const extraHeaders = { - 'Carto-Event': expectedMetricsEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - const template = templateBuilder({ name: 'preview' }); - - this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); - - this.testClient.getPreview(640, 480, {}, (err, res, body) => { - if (err) { - return done(err); - } + it('should send event for static named map requests', function (done) { + const expectedEvent = 'map_view'; + const expectedMetricsEvent = 'event-test'; + const expectedEventSource = 'event-source-test'; + const expectedEventGroupId = '1'; + const expectedResponseCode = '200'; + const expectedMapType = 'static'; + const extraHeaders = { + 'Carto-Event': expectedMetricsEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + const template = templateBuilder({ name: 'preview' }); + + this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); + + this.testClient.getPreview(640, 480, {}, (err, res, body) => { + if (err) { + return done(err); + } - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.client_event, expectedMetricsEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(attributes.map_type, expectedMapType); - assert.strictEqual(typeof attributes.map_id, 'string'); - assert.strictEqual(typeof attributes.cache_buster, 'string'); - assert.strictEqual(typeof attributes.template_hash, 'string'); - assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.client_event, expectedMetricsEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(attributes.map_type, expectedMapType); + assert.strictEqual(typeof attributes.map_id, 'string'); + assert.strictEqual(typeof attributes.cache_buster, 'string'); + assert.strictEqual(typeof attributes.template_hash, 'string'); + assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); - return done(); + return done(); + }); }); - }); - it('should send event for errored static named map requests', function (done) { - const expectedEvent = 'map_view'; - const expectedMetricsEvent = 'event-test'; - const expectedEventSource = 'event-source-test'; - const expectedEventGroupId = '1'; - const expectedResponseCode = '400'; - const expectedMapType = 'static'; - const extraHeaders = { - 'Carto-Event': expectedMetricsEvent, - 'Carto-Event-Source': expectedEventSource, - 'Carto-Event-Group-Id': expectedEventGroupId - }; - const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; - const template = templateBuilder({ name: 'preview-errored' }); - - this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); - - const widthTooLarge = 8193; - const params = { - response: { - status: 400, - headers: { - 'Content-Type': 'application/json; charset=utf-8' + it('should send event for errored static named map requests', function (done) { + const expectedEvent = 'map_view'; + const expectedMetricsEvent = 'event-test'; + const expectedEventSource = 'event-source-test'; + const expectedEventGroupId = '1'; + const expectedResponseCode = '400'; + const expectedMapType = 'static'; + const extraHeaders = { + 'Carto-Event': expectedMetricsEvent, + 'Carto-Event-Source': expectedEventSource, + 'Carto-Event-Group-Id': expectedEventGroupId + }; + const overrideServerOptions = { pubSubMetrics: { enabled: true, topic: 'topic-test' } }; + const template = templateBuilder({ name: 'preview-errored' }); + + this.testClient = new TestClient(template, apikey, extraHeaders, overrideServerOptions); + + const widthTooLarge = 8193; + const params = { + response: { + status: 400, + headers: { + 'Content-Type': 'application/json; charset=utf-8' + } } - } - }; + }; - this.testClient.getPreview(widthTooLarge, 480, params, (err, res, body) => { - if (err) { - return done(err); - } + this.testClient.getPreview(widthTooLarge, 480, params, (err, res, body) => { + if (err) { + return done(err); + } - assert.ok(this.pubSubMetricsBackendSendMethodCalled); + assert.ok(this.pubSubMetricsBackendSendMethodCalled); - const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; + const { event, attributes } = this.pubSubMetricsBackendSendMethodCalledWith; - assert.strictEqual(event, expectedEvent); - assert.strictEqual(attributes.client_event, expectedMetricsEvent); - assert.strictEqual(attributes.event_source, expectedEventSource); - assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); - assert.strictEqual(attributes.response_code, expectedResponseCode); - assert.strictEqual(attributes.map_type, expectedMapType); - assert.strictEqual(typeof attributes.map_id, 'string'); - assert.strictEqual(typeof attributes.cache_buster, 'string'); - assert.strictEqual(typeof attributes.template_hash, 'string'); - assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); + assert.strictEqual(event, expectedEvent); + assert.strictEqual(attributes.client_event, expectedMetricsEvent); + assert.strictEqual(attributes.event_source, expectedEventSource); + assert.strictEqual(attributes.client_event_group_id, expectedEventGroupId); + assert.strictEqual(attributes.response_code, expectedResponseCode); + assert.strictEqual(attributes.map_type, expectedMapType); + assert.strictEqual(typeof attributes.map_id, 'string'); + assert.strictEqual(typeof attributes.cache_buster, 'string'); + assert.strictEqual(typeof attributes.template_hash, 'string'); + assert.strictEqual(attributes.stat_tag, template.layergroup.stat_tag); - return done(); + return done(); + }); }); }); });