agw/overleaf-cep
1
0
Fork 0
mirror of https://github.com/yu-i-i/overleaf-cep.git synced 2026-06-05 07:09:02 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #8895 from overleaf/jpa-allow-empty-segmentation-val

Browse Source 
[web] allow empty event segmentation values again

GitOrigin-RevId: a02fd5085697bf7e2f2d4a5c2dc93dfebcd6e140
This commit is contained in:
Jakob Ackermann
2022-07-18 09:03:59 +01:00
committed by Copybot
parent 3074c79780
commit c95aeae07d
2 changed files with 30 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
services/web/app/src/Features/Analytics/AnalyticsManager.js
+1 -1
View File
@@ -243,7 +243,7 @@ function _checkPropertyValue(propertyValue) {
}
function _isAttributeValid(attribute) {
return attribute && /^[a-zA-Z0-9-_.:;,/]+$/.test(attribute)
return !attribute || /^[a-zA-Z0-9-_.:;,/]+$/.test(attribute)
}
function _isSegmentationValid(segmentation) {
services/web/test/unit/src/Analytics/AnalyticsManagerTests.js
+29
View File
@@ -209,6 +209,35 @@ describe('AnalyticsManager', function () {
}
)
})
it('empty field in event segmentation', async function () {
const timings = null
await this.AnalyticsManager.recordEventForUser(
this.fakeUserId,
'an_event',
{ compileTime: timings?.compileE2E }
)
sinon.assert.calledWithMatch(this.analyticsEventsQueue.add, 'event', {
analyticsId: this.analyticsId,
event: 'an_event',
segmentation: { compileTime: undefined },
isLoggedIn: true,
})
})
it('boolean field in event segmentation', async function () {
await this.AnalyticsManager.recordEventForUser(
this.fakeUserId,
'an_event',
{ isAutoCompile: false }
)
sinon.assert.calledWithMatch(this.analyticsEventsQueue.add, 'event', {
analyticsId: this.analyticsId,
event: 'an_event',
segmentation: { isAutoCompile: false },
isLoggedIn: true,
})
})
})
describe('AnalyticsIdMiddleware', function () {