webhook working need to merge

alnutile · Jun 12, 2023 · 34e90f8 · 34e90f8
1 parent c3a1e0d
commit 34e90f8
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 10 deletions.
diff --git a/app/Listeners/RunTransformersListener.php b/app/Listeners/RunTransformersListener.php
@@ -3,8 +3,6 @@
 namespace App\Listeners;
 
 use App\Events\SourceRunCompleteEvent;
-use Illuminate\Contracts\Queue\ShouldQueue;
-use Illuminate\Queue\InteractsWithQueue;
 
 class RunTransformersListener
 {

diff --git a/app/Models/Source.php b/app/Models/Source.php
@@ -46,7 +46,7 @@ public function documents()
     /**
      * @throws SourceTypeMissingException
      */
-    public function run(array $payload = []) : Document
+    public function run(array $payload = []): Document
     {
         try {
             $sourceType = $this->getSourceTypeClass();

diff --git a/app/Providers/EventServiceProvider.php b/app/Providers/EventServiceProvider.php
@@ -21,8 +21,8 @@ class EventServiceProvider extends ServiceProvider
             SendEmailVerificationNotification::class,
         ],
         SourceRunCompleteEvent::class => [
-            RunTransformersListener::class
-        ]
+            RunTransformersListener::class,
+        ],
     ];
 
     /**

diff --git a/tests/Feature/Http/Controllers/WebHooksSourceControllerTest.php b/tests/Feature/Http/Controllers/WebHooksSourceControllerTest.php
@@ -2,11 +2,13 @@
 
 namespace Tests\Feature\Http\Controllers;
 
+use App\Events\SourceRunCompleteEvent;
 use App\Jobs\ProcessSourceJob;
 use App\Models\Project;
 use App\Models\Source;
 use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
+use Illuminate\Support\Facades\Event;
 use Illuminate\Support\Facades\Http;
 use Illuminate\Support\Facades\Queue;
 use Tests\TestCase;
@@ -120,6 +122,7 @@ public function testCreateWebHook()
     public function test_api()
     {
         Http::fake();
+        Event::fake();
         $data = get_fixture('github_webhook.json');
         $source = Source::factory()->webFileMetaData()->create();
         $this->post(route('api.sources.webhook', [
@@ -129,5 +132,6 @@ public function test_api()
             ->assertStatus(200);
 
         Http::assertSentCount(1);
+        Event::assertDispatched(SourceRunCompleteEvent::class);
     }
 }
diff --git a/tests/Feature/Listeners/RunTransformersListenerTest.php b/tests/Feature/Listeners/RunTransformersListenerTest.php
@@ -2,14 +2,12 @@
 
 namespace Tests\Feature\Listeners;
 
-use Illuminate\Foundation\Testing\RefreshDatabase;
-use Illuminate\Foundation\Testing\WithFaker;
 use Tests\TestCase;
 
 class RunTransformersListenerTest extends TestCase
 {
-
-    public function test_runs_transformers() {
-        $this->markTestSkipped("@TODO oops wrong branch");
+    public function test_runs_transformers()
+    {
+        $this->markTestSkipped('@TODO oops wrong branch');
     }
 }