Apply fixes from StyleCI

This commit is contained in:
StyleCI Bot
2023-10-26 12:55:28 +00:00
committed by RafficMohammed
parent ccf417998c
commit 36396e6460
4 changed files with 181 additions and 216 deletions

View File

@@ -10,7 +10,6 @@ use App\Model\kb\Category;
use App\Model\kb\Relationship; use App\Model\kb\Relationship;
use App\User; use App\User;
use Faker\Factory as FakerFactory; use Faker\Factory as FakerFactory;
use Illuminate\Foundation\Testing\DatabaseTransactions;
use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Hash;
use Illuminate\Support\Facades\Lang; use Illuminate\Support\Facades\Lang;
use Illuminate\Support\Facades\Validator; use Illuminate\Support\Facades\Validator;
@@ -19,7 +18,6 @@ use Tests\TestCase;
class ArticleControllerTest extends TestCase class ArticleControllerTest extends TestCase
{ {
protected $user; // Declare a user property protected $user; // Declare a user property
// Set up the authenticated user before each test // Set up the authenticated user before each test
@@ -56,22 +54,18 @@ class ArticleControllerTest extends TestCase
$this->actingAs($user); $this->actingAs($user);
$this->assertAuthenticated(); $this->assertAuthenticated();
} }
/** @test */
public
function it_can_display_the_article_index_page()
{
/** @test */
public function it_can_display_the_article_index_page()
{
$response = $this->get(route('article.index')); $response = $this->get(route('article.index'));
$response->assertStatus(200); $response->assertStatus(200);
} }
public public function testStoreArticleWithCategories()
function testStoreArticleWithCategories()
{ {
// Create a Category model for testing // Create a Category model for testing
$data = [ $data = [
'name' => 'Test Category', 'name' => 'Test Category',
@@ -130,7 +124,7 @@ class ArticleControllerTest extends TestCase
$this->assertDatabaseHas('kb_article', [ $this->assertDatabaseHas('kb_article', [
'name' => $articleData['name'], 'name' => $articleData['name'],
'slug' => Str::slug($articleData['name'], '-'), 'slug' => Str::slug($articleData['name'], '-'),
'publish_time' => $articleData['year'] . '-' . $articleData['month'] . '-' . $articleData['day'] . ' ' . $articleData['hour'] . ':' . $articleData['minute'] . ':00', 'publish_time' => $articleData['year'].'-'.$articleData['month'].'-'.$articleData['day'].' '.$articleData['hour'].':'.$articleData['minute'].':00',
]); ]);
// Check if the category relationship exists // Check if the category relationship exists
@@ -139,14 +133,12 @@ class ArticleControllerTest extends TestCase
'article_id' => Article::latest()->first()->id, // Get the ID of the latest created article 'article_id' => Article::latest()->first()->id, // Get the ID of the latest created article
]); ]);
} else { } else {
$this->fail('Exception thrown: ' . $e->getMessage()); $this->fail('Exception thrown: '.$e->getMessage());
} }
} }
public public function testEditArticle()
function testEditArticle()
{ {
// Arrange // Arrange
$article = Article::latest()->first(); // Create a sample Article for testing $article = Article::latest()->first(); // Create a sample Article for testing
$relationship = Relationship::latest()->first(); // Create a sample Relationship for testing $relationship = Relationship::latest()->first(); // Create a sample Relationship for testing
@@ -165,10 +157,8 @@ class ArticleControllerTest extends TestCase
$response->assertStatus(200); $response->assertStatus(200);
} }
public public function testUpdateArticle()
function testUpdateArticle()
{ {
$article = Article::latest()->first(); $article = Article::latest()->first();
$category = Category::latest()->first(); $category = Category::latest()->first();
@@ -205,10 +195,8 @@ class ArticleControllerTest extends TestCase
} }
/** @test */ /** @test */
public public function it_can_delete_a_category()
function it_can_delete_a_category()
{ {
// Create a sample article, relationship // Create a sample article, relationship
$article = Article::latest()->first(); $article = Article::latest()->first();
$relationship = Relationship::find($article->id); $relationship = Relationship::find($article->id);
@@ -233,10 +221,8 @@ class ArticleControllerTest extends TestCase
$this->assertDatabaseMissing('kb_category', ['id' => $category->id]); $this->assertDatabaseMissing('kb_category', ['id' => $category->id]);
} }
public public function it_cannot_delete_a_article_if_related()
function it_cannot_delete_a_article_if_related()
{ {
// Create a category // Create a category
$article = Article::find(1); $article = Article::find(1);
@@ -252,5 +238,4 @@ class ArticleControllerTest extends TestCase
// Assert that the response has a failure message // Assert that the response has a failure message
$response->assertSessionHas('fails', Lang::get('lang.article_not_deleted')); $response->assertSessionHas('fails', Lang::get('lang.article_not_deleted'));
} }
} }

View File

@@ -7,7 +7,6 @@ use App\Model\kb\Category;
use App\Model\kb\Relationship; use App\Model\kb\Relationship;
use App\User; use App\User;
use Faker\Factory as FakerFactory; use Faker\Factory as FakerFactory;
use Illuminate\Foundation\Testing\DatabaseTransactions;
use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Hash;
use Illuminate\Support\Facades\Lang; use Illuminate\Support\Facades\Lang;
use Illuminate\Support\Facades\Validator; use Illuminate\Support\Facades\Validator;
@@ -57,7 +56,6 @@ class CategoryControllerTest extends TestCase
/** @test */ /** @test */
public function it_can_display_the_category_index_page() public function it_can_display_the_category_index_page()
{ {
$response = $this->get(route('category.index')); $response = $this->get(route('category.index'));
$response->assertStatus(200); $response->assertStatus(200);
@@ -65,7 +63,6 @@ class CategoryControllerTest extends TestCase
public function testValidationPasses() public function testValidationPasses()
{ {
$data = [ $data = [
'name' => 'New Category', 'name' => 'New Category',
'description' => 'Category Description', 'description' => 'Category Description',
@@ -83,7 +80,6 @@ class CategoryControllerTest extends TestCase
public function testValidationFailsWhenNameMissing() public function testValidationFailsWhenNameMissing()
{ {
$data = [ $data = [
'description' => 'Category Description', 'description' => 'Category Description',
]; ];
@@ -97,7 +93,6 @@ class CategoryControllerTest extends TestCase
public function testValidationFailsWhenNameExceedsMaxLength() public function testValidationFailsWhenNameExceedsMaxLength()
{ {
$data = [ $data = [
'name' => str_repeat('A', 251), 'name' => str_repeat('A', 251),
'description' => 'Category Description', 'description' => 'Category Description',
@@ -112,7 +107,6 @@ class CategoryControllerTest extends TestCase
public function testValidationFailsWhenNameNotUnique() public function testValidationFailsWhenNameNotUnique()
{ {
$data = [ $data = [
'name' => 'New Category', 'name' => 'New Category',
'description' => 'Category Description', 'description' => 'Category Description',
@@ -127,7 +121,6 @@ class CategoryControllerTest extends TestCase
public function testValidationFailsWhenDescriptionMissing() public function testValidationFailsWhenDescriptionMissing()
{ {
$data = [ $data = [
'name' => 'New Category', 'name' => 'New Category',
]; ];
@@ -141,7 +134,6 @@ class CategoryControllerTest extends TestCase
public function testEditCategory() public function testEditCategory()
{ {
$category = Category::latest()->first(); $category = Category::latest()->first();
$categories = Category::pluck('name', 'id')->toArray(); $categories = Category::pluck('name', 'id')->toArray();
$response = $this->get( $response = $this->get(
@@ -156,7 +148,6 @@ class CategoryControllerTest extends TestCase
/** @test */ /** @test */
public function it_can_update_an_existing_category() public function it_can_update_an_existing_category()
{ {
// Retrieve an existing category from the database // Retrieve an existing category from the database
$category = Category::latest()->first(); $category = Category::latest()->first();
@@ -179,7 +170,6 @@ class CategoryControllerTest extends TestCase
/** @test */ /** @test */
public function it_cannot_update_an_existing_category() public function it_cannot_update_an_existing_category()
{ {
// Retrieve an existing category from the database // Retrieve an existing category from the database
$category = Category::latest()->first(); $category = Category::latest()->first();
@@ -200,7 +190,6 @@ class CategoryControllerTest extends TestCase
/** @test */ /** @test */
public function it_can_delete_a_category() public function it_can_delete_a_category()
{ {
// Create a category // Create a category
$category = Category::latest()->first(); $category = Category::latest()->first();
@@ -219,5 +208,4 @@ class CategoryControllerTest extends TestCase
// Assert that the response has a success message // Assert that the response has a success message
$response->assertSessionHas('success', Lang::get('lang.category_deleted_successfully')); $response->assertSessionHas('success', Lang::get('lang.category_deleted_successfully'));
} }
} }

View File

@@ -6,8 +6,6 @@ use App\Http\Requests\kb\PageRequest;
use App\Model\kb\Page; use App\Model\kb\Page;
use App\User; use App\User;
use Faker\Factory as FakerFactory; use Faker\Factory as FakerFactory;
use Illuminate\Foundation\Testing\DatabaseTransactions;
use Illuminate\Foundation\Testing\RefreshDatabase;
use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Hash;
use Illuminate\Support\Facades\Validator; use Illuminate\Support\Facades\Validator;
use Tests\TestCase; use Tests\TestCase;

View File

@@ -5,14 +5,11 @@ namespace Tests\Unit;
use App\Model\helpdesk\Ticket\Ticket_Thread; use App\Model\helpdesk\Ticket\Ticket_Thread;
use App\Model\helpdesk\Ticket\Tickets; use App\Model\helpdesk\Ticket\Tickets;
use App\User; use App\User;
use DateTimeZone;
use Faker\Factory as FakerFactory; use Faker\Factory as FakerFactory;
use Illuminate\Foundation\Testing\DatabaseTransactions;
use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Hash;
use Illuminate\Support\Facades\Lang; use Illuminate\Support\Facades\Lang;
use Illuminate\Support\Str; use Illuminate\Support\Str;
use Tests\TestCase; use Tests\TestCase;
use UTC;
class TicketControllerTest extends TestCase class TicketControllerTest extends TestCase
{ {
@@ -102,7 +99,6 @@ class TicketControllerTest extends TestCase
// Assert that the response status is 200 (OK). // Assert that the response status is 200 (OK).
$response->assertStatus(200); $response->assertStatus(200);
} }
//Testing Reply Alert and Last Activity filed //Testing Reply Alert and Last Activity filed
@@ -144,7 +140,5 @@ class TicketControllerTest extends TestCase
$response3 = $this->post(route('ticket.reply', ['id' => $tickets->id]), $replyData); $response3 = $this->post(route('ticket.reply', ['id' => $tickets->id]), $replyData);
$response3->assertStatus(200); $response3->assertStatus(200);
$response3->assertSee(Lang::get('lang.you_have_successfully_replied_to_your_ticket')); $response3->assertSee(Lang::get('lang.you_have_successfully_replied_to_your_ticket'));
} }
} }