Skip to content

Commit 7e4efba

Browse files
author
Richard Tan
committed
feat: proper test field initialisation
1 parent 99b7ae6 commit 7e4efba

File tree

1 file changed

+58
-52
lines changed

1 file changed

+58
-52
lines changed

packages/drafts-realm/campaign-form.gts

+58-52
Original file line numberDiff line numberDiff line change
@@ -79,75 +79,43 @@ const formatNumberWithSeparator = (
7979
class Isolated extends Component<typeof CampaignForm> {
8080
<template>
8181
<div class='campaign-form-isolated'>
82-
<FieldContainer @label='Name' data-test-field='active' class='field'>
82+
<FieldContainer @label='Name' class='field'>
8383
{{@model.name}}
8484
</FieldContainer>
85-
<FieldContainer @label='Status' data-test-field='active' class='field'>
85+
<FieldContainer @label='Status' class='field'>
8686
{{@model.status}}
8787
</FieldContainer>
88-
<FieldContainer @label='Active' data-test-field='active' class='field'>
88+
<FieldContainer @label='Active' class='field'>
8989
{{if @model.active 'Yes' 'No'}}
9090
</FieldContainer>
91-
<FieldContainer @label='Type' data-test-field='active' class='field'>
91+
<FieldContainer @label='Type' class='field'>
9292
{{@model.type}}
9393
</FieldContainer>
94-
<FieldContainer
95-
@label='Parent Campaign'
96-
data-test-field='active'
97-
class='field'
98-
>
94+
<FieldContainer @label='Parent Campaign' class='field'>
9995
<@fields.parent_campaign />
10096
</FieldContainer>
101-
<FieldContainer
102-
@label='Description'
103-
data-test-field='active'
104-
class='field'
105-
>
97+
<FieldContainer @label='Description' class='field'>
10698
{{@model.description}}
10799
</FieldContainer>
108-
<FieldContainer
109-
@label='Start Date'
110-
data-test-field='active'
111-
class='field'
112-
>
100+
<FieldContainer @label='Start Date' class='field'>
113101
<@fields.start_date />
114102
</FieldContainer>
115-
<FieldContainer @label='End Date' data-test-field='active' class='field'>
103+
<FieldContainer @label='End Date' class='field'>
116104
<@fields.end_date />
117105
</FieldContainer>
118-
<FieldContainer
119-
@label='Num Sent in Campaign'
120-
data-test-field='active'
121-
class='field'
122-
>
106+
<FieldContainer @label='Num Sent in Campaign' class='field'>
123107
{{formatNumberWithSeparator @model.number_sent}}
124108
</FieldContainer>
125-
<FieldContainer
126-
@label='Expected Response (%)'
127-
data-test-field='active'
128-
class='field'
129-
>
109+
<FieldContainer @label='Expected Response (%)' class='field'>
130110
{{formatNumberWithSeparator @model.expected_response_percentage true}}
131111
</FieldContainer>
132-
<FieldContainer
133-
@label='Expected Revenue in Campaign'
134-
data-test-field='active'
135-
class='field'
136-
>
112+
<FieldContainer @label='Expected Revenue in Campaign' class='field'>
137113
{{formatCurrency @model.expected_revenue}}
138114
</FieldContainer>
139-
<FieldContainer
140-
@label='Budgeted Cost in Campaign'
141-
data-test-field='active'
142-
class='field'
143-
>
115+
<FieldContainer @label='Budgeted Cost in Campaign' class='field'>
144116
{{formatCurrency @model.budgeted_cost}}
145117
</FieldContainer>
146-
<FieldContainer
147-
@label='Actual Cost in Campaign'
148-
data-test-field='active'
149-
class='field'
150-
>
118+
<FieldContainer @label='Actual Cost in Campaign' class='field'>
151119
{{formatCurrency @model.actual_cost}}
152120
</FieldContainer>
153121
</div>
@@ -335,7 +303,12 @@ class Edit extends Component<typeof CampaignForm> {
335303

336304
<template>
337305
<div class='campaign-form-edit'>
338-
<FieldContainer @label='Campaign Name' @tag='label' class='field'>
306+
<FieldContainer
307+
@label='Campaign Name'
308+
data-test-field='name'
309+
@tag='label'
310+
class='field'
311+
>
339312
<BoxelInput
340313
@value={{this.name}}
341314
@onInput={{this.updateName}}
@@ -374,41 +347,71 @@ class Edit extends Component<typeof CampaignForm> {
374347
<div>{{item.name}}</div>
375348
</BoxelSelect>
376349
</FieldContainer>
377-
<FieldContainer @label='Parent Campaign' @tag='label' class='field'>
350+
<FieldContainer
351+
@label='Parent Campaign'
352+
data-test-field='parent_campaign'
353+
@tag='label'
354+
class='field'
355+
>
378356
<@fields.parent_campaign />
379357
</FieldContainer>
380-
<FieldContainer @label='Description' @tag='label' class='field'>
358+
<FieldContainer
359+
@label='Description'
360+
data-test-field='description'
361+
@tag='label'
362+
class='field'
363+
>
381364
<BoxelInput
382365
@type='textarea'
383366
@value={{this.description}}
384367
@onInput={{this.updateDescription}}
385368
/>
386369
</FieldContainer>
387-
<FieldContainer @label='Start Date' @tag='label' class='field'>
370+
<FieldContainer
371+
@label='Start Date'
372+
data-test-field='start_date'
373+
@tag='label'
374+
class='field'
375+
>
388376
<BoxelInput
389377
type='date'
390378
@value={{this.startDateString}}
391379
@onInput={{fn this.parseDateInput 'start_date'}}
392380
@max='9999-12-31'
393381
/>
394382
</FieldContainer>
395-
<FieldContainer @label='End Date' @tag='label' class='field'>
383+
<FieldContainer
384+
@label='End Date'
385+
data-test-field='end_date'
386+
@tag='label'
387+
class='field'
388+
>
396389
<BoxelInput
397390
type='date'
398391
@value={{this.endDateString}}
399392
@onInput={{fn this.parseDateInput 'end_date'}}
400393
@max='9999-12-31'
401394
/>
402395
</FieldContainer>
403-
<FieldContainer @label='Num Sent in Campaign' @tag='label' class='field'>
396+
<FieldContainer
397+
@label='Num Sent in Campaign'
398+
data-test-field='number_sent'
399+
@tag='label'
400+
class='field'
401+
>
404402
<BoxelInput
405403
@value={{this.numberSentInputValue}}
406404
@onKeyPress={{this.validateOnKeyPress}}
407405
@onInput={{fn this.updateCustomNumberInput 'numberSentInputValue'}}
408406
@onBlur={{this.onBlurNumberSent}}
409407
/>
410408
</FieldContainer>
411-
<FieldContainer @label='Expected Response (%)' @tag='label' class='field'>
409+
<FieldContainer
410+
@label='Expected Response (%)'
411+
data-test-field='expected_response_percentage'
412+
@tag='label'
413+
class='field'
414+
>
412415
<BoxelInput
413416
@value={{this.expectedResponseInputValue}}
414417
@onKeyPress={{this.validateOnKeyPress}}
@@ -421,6 +424,7 @@ class Edit extends Component<typeof CampaignForm> {
421424
</FieldContainer>
422425
<FieldContainer
423426
@label='Expected Revenue in Campaign'
427+
data-test-field='expected_revenue'
424428
@tag='label'
425429
class='field'
426430
>
@@ -441,6 +445,7 @@ class Edit extends Component<typeof CampaignForm> {
441445
</FieldContainer>
442446
<FieldContainer
443447
@label='Budgeted Cost in Campaign'
448+
data-test-field='budgeted_cost'
444449
@tag='label'
445450
class='field'
446451
>
@@ -458,6 +463,7 @@ class Edit extends Component<typeof CampaignForm> {
458463
</FieldContainer>
459464
<FieldContainer
460465
@label='Actual Cost in Campaign'
466+
data-test-field='actual_cost'
461467
@tag='label'
462468
class='field'
463469
>

0 commit comments

Comments
 (0)