Feature/missing apis

[mubbi]

This pull request introduces several new features and improvements, primarily focusing on the addition of enums for statuses, new API controllers for managing articles and comments, and enhancements to the UserRole enum. The changes aim to improve code organization, readability, and functionality for admin-related operations.

Enum Enhancements

• ArticleStatus enum: Added REVIEW and TRASHED statuses, along with utility methods displayName, isPublished, and isDraft for better status handling.
• CommentStatus enum: Introduced a new enum with statuses (PENDING, APPROVED, REJECTED, SPAM) and utility methods displayName, isPublished, and isDraft.
• UserRole enum: Updated role values to lowercase and added a displayName method for consistent display formatting.

Article Management API Controllers

• ApproveArticleController: Enables approving and publishing articles, with error handling for not found and server issues.
• FeatureArticleController: Allows marking articles as featured, with structured responses and error handling.
• GetArticlesController: Provides a paginated list of articles with admin filters and metadata.
• ReportArticleController: Adds functionality to report articles with a reason.
• ShowArticleController: Retrieves a single article with detailed admin-related information.

Comment Management API Controllers

• ApproveCommentController: Approves pending comments for publication, with error handling for missing comments.
• DeleteCommentController: Permanently deletes comments and handles errors for non-existent comments.
• GetCommentsController: Retrieves a paginated list of comments with filters and metadata.

Newsletter Management

• DeleteSubscriberController: Adds functionality to remove a newsletter subscriber, with error handling for missing subscribers.

[Copilot]

Pull Request Overview

This pull request introduces comprehensive API controllers for admin-related functionality, including user management, comment moderation, newsletter subscriber management, notification handling, and article administration. The main purpose is to expand the admin API capabilities with proper authorization, validation, and error handling.

• Adds new API controllers for managing users (CRUD operations, ban/unban, block/unblock)
• Implements comment moderation features (approval, deletion with admin notes)
• Introduces newsletter subscriber management and notification system controllers
• Updates existing test files to use named routes instead of hardcoded URLs
• Adds comprehensive test coverage for all new admin functionality

Reviewed Changes

Copilot reviewed 123 out of 123 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/Feature/Providers/AuthServiceProviderTest.php	Updates error message to use localization function
tests/Feature/API/V1/User/UpdateProfileControllerTest.php	New comprehensive test suite for user profile updates
tests/Feature/API/V1/User/MeControllerTest.php	Enhanced test coverage with additional edge cases and route usage
tests/Feature/API/V1/Tag/GetTagsControllerTest.php	Updated to use named routes
tests/Feature/API/V1/Category/GetCategoriesControllerTest.php	Updated to use named routes
tests/Feature/API/V1/Auth/RefreshTokenControllerTest.php	Updates error message to use localization
tests/Feature/API/V1/Auth/LogoutControllerTest.php	Updates error message to use localization
tests/Feature/API/V1/Auth/LoginControllerTest.php	Updates error message to use localization
tests/Feature/API/V1/Article/ShowArticleControllerTest.php	Removes duplicate test line
tests/Feature/API/V1/Admin/User/*.php	New comprehensive admin user management test suites
tests/Feature/API/V1/Admin/Comment/*.php	New comment moderation test suites
tests/Feature/API/V1/Admin/Newsletter/*.php	New newsletter management test suites
tests/Feature/API/V1/Admin/Notification/*.php	New notification system test suites
tests/Feature/API/V1/Admin/Article/ShowArticleControllerTest.php	New admin article management test suite

Comments suppressed due to low confidence (2)

tests/Feature/API/V1/Admin/Comment/ApproveCommentControllerTest.php:225

• The test assumes admin_note validation will return 422, but the comment on line 211 suggests admin_note is optional and not strictly validated. This creates inconsistent test expectations.

        // Assert - admin_note should be validated and return 422 for exceeding max length

tests/Feature/API/V1/Admin/Comment/DeleteCommentControllerTest.php:210

• The test comment indicates admin_note is not validated, but then asserts a 200 response. This suggests the validation logic may not be properly tested or implemented consistently across comment controllers.

        // Assert - admin_note is optional and not strictly validated