Version 1.2 Release

.github/workflows/coverage_check.yml

@@ -0,0 +1,70 @@
+name: Test and Coverage Check
+
+on:
+  push:
+    branches:
+      - main
+      - development
+
+jobs:
+  test-and-check-coverage:
+    runs-on: ubuntu-latest
+
+    steps:
+    - name: Checkout repository
+      uses: actions/checkout@v2
+
+    - name: Set up Go
+      uses: actions/setup-go@v2
+      with:
+        go-version: '1.21.7'
+
+    - name: Install dependencies and update go.mod/go.sum
+      run: |
+        go get -t -v ./...
+        go mod tidy
+
+    - name: Run tests
+      run: |
+        go test -cover ./usecases/admin/...
+        go test -cover ./usecases/category/...
+        go test -cover ./usecases/chatbot/...
+        go test -cover ./usecases/complaint/...
+        go test -cover ./usecases/complaint_activity/...
+        go test -cover ./usecases/complaint_file/...
+        go test -cover ./usecases/complaint_like/...
+        go test -cover ./usecases/complaint_process/...
+        go test -cover ./usecases/dashboard/...
+        go test -cover ./usecases/discussion/...
+        go test -cover ./usecases/news/...
+        go test -cover ./usecases/news_comment/...
+        go test -cover ./usecases/news_file/...
+        go test -cover ./usecases/news_like/...
+        go test -cover ./usecases/regency/...
+        go test -cover ./usecases/user/...
+
+    - name: Check coverage
+      run: |
+        admin_coverage=$(go test -cover ./usecases/admin/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        category_coverage=$(go test -cover ./usecases/category/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        chatbot_coverage=$(go test -cover ./usecases/chatbot/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        complaint_coverage=$(go test -cover ./usecases/complaint/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        complaint_activity_coverage=$(go test -cover ./usecases/complaint_activity/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        complaint_file_coverage=$(go test -cover ./usecases/complaint_file/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        complaint_like_coverage=$(go test -cover ./usecases/complaint_like/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        complaint_process_coverage=$(go test -cover ./usecases/complaint_process/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        dashboard_coverage=$(go test -cover ./usecases/dashboard/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        discussion_coverage=$(go test -cover ./usecases/discussion/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        news_coverage=$(go test -cover ./usecases/news/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        news_comment_coverage=$(go test -cover ./usecases/news_comment/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        news_file_coverage=$(go test -cover ./usecases/news_file/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        news_like_coverage=$(go test -cover ./usecases/news_like/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        regency_coverage=$(go test -cover ./usecases/regency/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+        user_coverage=$(go test -cover ./usecases/user/... | grep -o '[0-9.]\+%' | cut -d'.' -f1 | tr -d '%')
+
+        if [ $admin_coverage -ge 90 ] && [ $category_coverage -ge 90 ] && [ $chatbot_coverage -ge 90 ] && [ $complaint_coverage -ge 90 ] && [ $complaint_activity_coverage -ge 90 ] && [ $complaint_file_coverage -ge 90 ] && [ $complaint_like_coverage -ge 90 ] && [ $complaint_process_coverage -ge 90 ] && [ $dashboard_coverage -ge 90 ] && [ $discussion_coverage -ge 90 ] && [ $news_coverage -ge 90 ] && [ $news_comment_coverage -ge 90 ] && [ $news_file_coverage -ge 90 ] && [ $news_like_coverage -ge 90 ] && [ $regency_coverage -ge 90 ] && [ $user_coverage -ge 90 ]; then
+          echo "All services have coverage above 90%"
+        else
+          echo "Some services have coverage below 90%"
+          exit 1
+        fi

constants/errror.go

@@ -47,4 +47,10 @@ var (
 	ErrInvalidFileFormat                = errors.New("invalid file format")
 	ErrForgotPasswordOTPNotVerified     = errors.New("forgot password otp not verified")
 	ErrConfirmPasswordDoesntMatch       = errors.New("confirm password doesn't match")
+	ErrColumnsDoesntMatch               = errors.New("columns doesn't match")
+	ErrInvalidCategoryIDFormat          = errors.New("invalid category id format")
+	ErrEmailNotRegistered               = errors.New("email not registered")
+	ErrDiscussionNotFound               = errors.New("discussion not found")
+	ErrPasswordMustBeAtLeast8Characters = errors.New("password must be at least 8 characters")
+	ErrCategoryHasBeenUsed              = errors.New("category has been used")
 )

controllers/category/category.go

@@ -28,7 +28,12 @@ func (cc *CategoryController) GetAll(c echo.Context) error {
 		return c.JSON(utils.ConvertResponseCode(err), base.NewErrorResponse(err.Error()))
 	}
 
-	return c.JSON(http.StatusOK, base.NewSuccessResponse("Success get all categories", categories))
+	responseCategories := make([]*response.Get, len(categories))
+	for i, category := range categories {
+		responseCategories[i] = response.GetFromEntitiesToResponse(&category)
+	}
+
+	return c.JSON(http.StatusOK, base.NewSuccessResponse("Success get all categories", responseCategories))
 }
 
 func (cc *CategoryController) GetByID(c echo.Context) error {
@@ -42,10 +47,6 @@ func (cc *CategoryController) GetByID(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, base.NewErrorResponse(err.Error()))
 	}
 
-	if err != nil {
-		return c.JSON(http.StatusBadRequest, base.NewErrorResponse(err.Error()))
-	}
-
 	category, err := cc.useCase.GetByID(id)
 	if err != nil {
 		if errors.Is(err, constants.ErrCategoryNotFound) {
@@ -54,7 +55,9 @@ func (cc *CategoryController) GetByID(c echo.Context) error {
 		return c.JSON(utils.ConvertResponseCode(err), base.NewErrorResponse(err.Error()))
 	}
 
-	return c.JSON(http.StatusOK, base.NewSuccessResponse("Success get category by ID", category))
+	responseCategory := response.GetFromEntitiesToResponse(&category)
+
+	return c.JSON(http.StatusOK, base.NewSuccessResponse("Success get category by ID", responseCategory))
 }
 
 func (cc *CategoryController) CreateCategory(c echo.Context) error {

controllers/dashboard/dashboard.go

@@ -6,7 +6,6 @@ import (
 	"e-complaint-api/usecases/dashboard"
 	"github.com/labstack/echo/v4"
 	"net/http"
-	"time"
 )
 
 type DashboardController struct {
@@ -31,7 +30,7 @@ func (ctrl *DashboardController) GetDashboardData(c echo.Context) error {
 				Name: complaint.User.Name,
 			},
 			Complaint: response.Complaint{
-				Date:   complaint.Date.Format(time.RFC3339),
+				Date:   complaint.CreatedAt.Format("2 January 2006 15:04:05"),
 				Status: complaint.Status,
 			},
 			Category: response.Category{

controllers/user/request/update_password.go

@@ -1,10 +1,5 @@
 package request
 
 type UpdatePassword struct {
-	NewPassword        string `json:"new_password" form:"new_password"`
-	ConfirmNewPassword string `json:"confirm_new_password" form:"confirm_new_password"`
-}
-
-func (up *UpdatePassword) ToEntities() (string, string) {
-	return up.ConfirmNewPassword, up.NewPassword
+	NewPassword string `json:"new_password" form:"new_password"`
 }

controllers/user/user.go

@@ -66,18 +66,9 @@ func (uc *UserController) GetUserByID(c echo.Context) error {
 		return c.JSON(http.StatusBadRequest, base.NewErrorResponse(constants.ErrInvalidIDFormat.Error()))
 	}
 
-	jwtID, err := utils.GetIDFromJWT(c)
-	if err != nil {
-		return c.JSON(http.StatusInternalServerError, base.NewErrorResponse(err.Error()))
-	}
-
-	if id != jwtID {
-		return c.JSON(http.StatusUnauthorized, base.NewErrorResponse(constants.ErrUnauthorized.Error()))
-	}
-
 	user, err := uc.userUseCase.GetUserByID(id)
 	if err != nil {
-		return c.JSON(http.StatusInternalServerError, base.NewErrorResponse(err.Error()))
+		return c.JSON(utils.ConvertResponseCode(err), base.NewErrorResponse(err.Error()))
 	}
 
 	userResponse := response.GetUsersFromEntitiesToResponse(user)
@@ -159,7 +150,7 @@ func (uc *UserController) DeleteUser(c echo.Context) error {
 
 	err = uc.userUseCase.Delete(id)
 	if err != nil {
-		return c.JSON(http.StatusInternalServerError, base.NewErrorResponse(err.Error()))
+		return c.JSON(utils.ConvertResponseCode(err), base.NewErrorResponse(err.Error()))
 	}
 
 	return c.JSON(http.StatusOK, base.NewSuccessResponse("Success Delete User", nil))
@@ -171,22 +162,12 @@ func (uc *UserController) UpdatePassword(c echo.Context) error {
 		return c.JSON(http.StatusInternalServerError, base.NewErrorResponse(err.Error()))
 	}
 
-	userRole, err := utils.GetRoleFromJWT(c)
-	if err != nil {
-		return c.JSON(http.StatusInternalServerError, base.NewErrorResponse(err.Error()))
-	}
-
-	if userRole != "user" {
-		return c.JSON(http.StatusUnauthorized, base.NewErrorResponse(constants.ErrUnauthorized.Error()))
-	}
-
 	var passwordRequest request.UpdatePassword
 	if err := c.Bind(&passwordRequest); err != nil {
 		return c.JSON(http.StatusBadRequest, base.NewErrorResponse(err.Error()))
 	}
 
-	newPassword, confirmPassword := passwordRequest.ToEntities()
-	err = uc.userUseCase.UpdatePassword(jwtID, newPassword, confirmPassword)
+	err = uc.userUseCase.UpdatePassword(jwtID, passwordRequest.NewPassword)
 	if err != nil {
 		return c.JSON(http.StatusBadRequest, base.NewErrorResponse(err.Error()))
 	}