Skip to content

Commit 85775b8

Browse files
authored
Merge pull request #6130 from Countly/QT-278-add-id
[QT-278] Added data test ids to content drawer elements
2 parents fa9b635 + fe2851a commit 85775b8

3 files changed

Lines changed: 13 additions & 3 deletions

File tree

frontend/express/public/javascripts/countly/vue/components/content.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,7 @@
378378
</el-collapse>
379379
</div>
380380
<div v-else>
381-
<div class="cly-vue-content-builder__layout-steps__header text-medium font-weight-bold">{{ header }}</div>
381+
<div class="cly-vue-content-builder__layout-steps__header text-medium font-weight-bold" :data-test-id="'content-drawer-sidebar-step-' + header.toLowerCase().replaceAll(' ', '-')">{{ header }}</div>
382382
<slot name="content-builder-layout-steps"></slot>
383383
</div>
384384
</div>
@@ -624,6 +624,11 @@
624624
value: {
625625
default: null,
626626
type: [String, Number]
627+
},
628+
testId: {
629+
type: String,
630+
default: 'cly-option-swapper-test-id',
631+
required: false
627632
}
628633
},
629634

frontend/express/public/javascripts/countly/vue/templates/UI/option-swapper.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
<div
88
v-for="(option, index) in options"
99
:key="`cly-option-swapper-option-${index}`"
10+
:data-test-id="`cly-option-swapper-option-${testId}-${option.value.toString().toLowerCase()}`"
1011
v-tooltip="option.tooltip"
1112
class="cly-option-swapper__option"
1213
:class="{
@@ -24,4 +25,4 @@
2425
/>
2526
<span v-else>{{ option.text }}</span>
2627
</div>
27-
</div>
28+
</div>

frontend/express/public/javascripts/countly/vue/templates/content/UI/content-sidebar-input.html

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
<div
33
v-if="subHeader"
44
class="cly-vue-content-builder-sidebar-input__sub-header"
5+
:data-test-id="'content-drawer-sidebar-step-' + subHeader.toLowerCase().replaceAll(' ', '-') + '-label'"
56
>
67
{{ subHeader }}
78
</div>
@@ -15,12 +16,14 @@
1516
<label
1617
v-if="label"
1718
class="cly-vue-content-builder-sidebar-input__label"
19+
:data-test-id="'content-drawer-sidebar-step-' + $attrs.id + '-' + label.toLowerCase().replaceAll(' ', '-') + '-label'"
1820
>
1921
{{ label }}
2022
<i
2123
v-if="isLabelTooltipVisible"
2224
v-tooltip.left="labelTooltip"
2325
:class="labelIcon"
26+
:data-test-id="'content-drawer-sidebar-step-' + $attrs.id + '-' + label.toLowerCase().replaceAll(' ', '-') + '-label-tooltip'"
2427
/>
2528
</label>
2629
<slot name="content-builder-layout-step">
@@ -38,6 +41,7 @@
3841
:controls="controlsProp"
3942
:disabled="disabled"
4043
:options="options"
44+
:test-id="'content-drawer-sidebar-step-' + $attrs.id"
4145
>
4246
<template
4347
v-if="isSuffixVisible"
@@ -88,4 +92,4 @@
8892
</div>
8993
</slot>
9094
</div>
91-
</div>
95+
</div>

0 commit comments

Comments
 (0)