Skip to content

Commit a4d5eaa

Browse files
committed
Fix tests
1 parent 099c13f commit a4d5eaa

2 files changed

Lines changed: 22 additions & 7 deletions

File tree

event/listener.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ public function validate_googleanalytics_id($event)
146146
$error = $event['error'];
147147

148148
// Add error message if the input is not a valid Google Analytics ID
149-
if (!preg_match('/^UA-\d{4,9}-\d{1,4}|G-[A-Z0-9]{10}$/', $input))
149+
if (!preg_match('/^UA-\d{4,9}-\d{1,4}$|^G-[A-Z0-9]{10}$/', $input))
150150
{
151151
$error[] = $this->user->lang('ACP_GOOGLEANALYTICS_ID_INVALID', $input);
152152
}

tests/event/listener_test.php

Lines changed: 21 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -172,32 +172,47 @@ public function validate_googleanalytics_id_data()
172172
return array(
173173
array(
174174
// valid code, no error
175-
array('googleanalytics_id' => 'UA-0000-00'),
175+
array('googleanalytics_id' => 'UA-0000-00', 'googleanalytics_tag' => 0),
176+
array(),
177+
),
178+
array(
179+
// valid code, no error
180+
array('googleanalytics_id' => 'UA-0000-00', 'googleanalytics_tag' => 1),
181+
array(),
182+
),
183+
array(
184+
// valid code, no error
185+
array('googleanalytics_id' => 'G-XXXXXXXXXX', 'googleanalytics_tag' => 1),
176186
array(),
177187
),
178188
array(
179189
// no code, no error
180-
array('googleanalytics_id' => ''),
190+
array('googleanalytics_id' => '', 'googleanalytics_tag' => 1),
181191
array(),
182192
),
193+
array(
194+
// invalid code, no error
195+
array('googleanalytics_id' => 'G-XXXXXXXXXX', 'googleanalytics_tag' => 0),
196+
array('ACP_GOOGLEANALYTICS_TAG_INVALID'),
197+
),
183198
array(
184199
// invalid code, error
185-
array('googleanalytics_id' => 'UA-00-00'),
200+
array('googleanalytics_id' => 'UA-00-00', 'googleanalytics_tag' => 1),
186201
array('ACP_GOOGLEANALYTICS_ID_INVALID'),
187202
),
188203
array(
189204
// invalid code, error
190-
array('googleanalytics_id' => 'UA-00000-00000'),
205+
array('googleanalytics_id' => 'UA-00000-00000', 'googleanalytics_tag' => 1),
191206
array('ACP_GOOGLEANALYTICS_ID_INVALID'),
192207
),
193208
array(
194209
// invalid code, error
195-
array('googleanalytics_id' => 'AU-0000-00'),
210+
array('googleanalytics_id' => 'AU-0000-00', 'googleanalytics_tag' => 1),
196211
array('ACP_GOOGLEANALYTICS_ID_INVALID'),
197212
),
198213
array(
199214
// invalid code, error
200-
array('googleanalytics_id' => 'foo-bar-foo'),
215+
array('googleanalytics_id' => 'foo-bar-foo', 'googleanalytics_tag' => 1),
201216
array('ACP_GOOGLEANALYTICS_ID_INVALID'),
202217
),
203218
array(

0 commit comments

Comments
 (0)