Merge pull request #28 from austincarpenter/use-alpha-dash-validation-rule

Replace `regex` rule with `alpha_dash:ascii`

Author: nunomaduro

src/Adapters/Laravel/Http/Requests/CreateEventRequest.php

@@ -22,7 +22,7 @@ public function rules(): array
     {
         return [
             'events' => ['required', 'array'],
-            'events.*.name' => ['required', 'string', 'regex:/^[A-Za-z0-9-_]+$/'],
+            'events.*.name' => ['required', 'string', 'alpha_dash:ascii'],
             'events.*.type' => ['required', Rule::enum(EventType::class)],
         ];
     }

tests/Feature/Laravel/Http/EventsTest.php

@@ -79,6 +79,28 @@
     ]);
 });
 
+it('does not create an analytic event if the event name is invalid', function (string $name): void {
+    $response = $this->post('/pan/events', [
+        'events' => [[
+            'name' => $name,
+            'type' => 'impression',
+        ]],
+    ]);
+
+    $response->assertStatus(302)->assertSessionHasErrors([
+        'events.0.name' => 'The events.0.name field must only contain letters, numbers, dashes, and underscores.',
+    ]);
+
+    $analytics = array_map(fn (Analytic $analytic): array => $analytic->toArray(), app(AnalyticsRepository::class)->all());
+
+    expect($analytics)->toBe([]);
+})->with([
+    'help modal',
+    'help.modal',
+    'help/modal',
+    '🙋',
+]);
+
 it('does not create an analytic event if the event is invalid', function (): void {
     $response = $this->post('/pan/events', [
         'events' => [[