Disable the routes by default (#336)

* Disable the routes by default

* Write the required tests

* Fix the style of codes

* Fix the style of codes

* Remove extra space

Author: NanoNulla

README.md

@@ -20,6 +20,7 @@ Create a Chat application for your multiple Models
 - [Installation](#installation)
 - [Usage](#usage)
   - [Adding the ability to participate to a Model](#Adding-the-ability-to-participate-to-a-Model)
+  - [Enable the routes](#enable-the-routes)
   - [Get participant details](#get-participant-details)
   - [Creating a conversation](#creating-a-conversation)
   - [Get a conversation by Id](#get-a-conversation-by-id)
@@ -101,6 +102,10 @@ class Bot extends Model
 }
 ```
 
+#### Enable the routes
+
+The package includes routes for conversations, conversation participants, and messaging. These routes are hidden by default. To enable them, change `should_load_routes` to `true`, but make sure to configure the middleware correctly. 
+
 #### Get participant details
 
 Since we allow Models with data that differ in structure to chat, we may want a uniform way to

config/musonza_chat.php

@@ -26,7 +26,7 @@
     /*
      * Whether to load the package routes file in your application.
      */
-    'should_load_routes' => true,
+    'should_load_routes' => false,
 
     /*
      * Routes configuration

tests/Feature/ConversationControllerTest.php

@@ -13,6 +13,13 @@
 
 class ConversationControllerTest extends TestCase
 {
+    public function setUp()
+    {
+        parent::setUp();
+
+        $this->app['config']->set('musonza_chat.should_load_routes', true);
+    }
+
     public function testStore()
     {
         $this->withoutExceptionHandling();

tests/Feature/ConversationMessageControllerTest.php

@@ -11,6 +11,13 @@
 
 class ConversationMessageControllerTest extends TestCase
 {
+    public function setUp()
+    {
+        parent::setUp();
+
+        $this->app['config']->set('musonza_chat.should_load_routes', true);
+    }
+
     public function testStore()
     {
         $conversation = factory(Conversation::class)->create();

tests/Feature/ConversationParticipationControllerTest.php

@@ -11,6 +11,13 @@
 
 class ConversationParticipationControllerTest extends TestCase
 {
+    public function setUp()
+    {
+        parent::setUp();
+
+        $this->app['config']->set('musonza_chat.should_load_routes', true);
+    }
+
     public function testStore()
     {
         $conversation = factory(Conversation::class)->create();

tests/Feature/DataTransformersTest.php

@@ -8,6 +8,13 @@
 
 class DataTransformersTest extends TestCase
 {
+    public function setUp()
+    {
+        parent::setUp();
+
+        $this->app['config']->set('musonza_chat.should_load_routes', true);
+    }
+
     public function testConversationWithoutTransformer()
     {
         $conversation = factory(Conversation::class)->create();

tests/Unit/RouteTest.php

@@ -0,0 +1,39 @@
+<?php
+
+namespace Musonza\Chat\Tests;
+
+use Illuminate\Support\Facades\Route;
+
+class RouteTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        // Disable route caching and refresh routes
+        Route::flushMiddlewareGroups();
+        Route::clearResolvedInstances();
+    }
+
+    /** @test */
+    public function it_can_disable_routes()
+    {
+        // Disable route loading
+        $this->app['config']->set('musonza_chat.should_load_routes', false);
+        $this->refreshApplication();
+
+        $response = $this->get('/conversations');
+        $response->assertStatus(404);
+    }
+
+    /** @test */
+    public function it_can_enable_routes()
+    {
+        // Enable route loading
+        $this->app['config']->set('musonza_chat.should_load_routes', true);
+        $this->refreshApplication();
+
+        $response = $this->get('/conversations');
+        $response->assertStatus(200);
+    }
+}