Bug Fixes (#382)

* fix: error messages and run all tests (#381)
* feat: Automatically open fold under cursor (#380)
* fix: discussion ID is not required (#383)

This is a #PATCH release.

---------

Co-authored-by: George Kontridze <george.kontridze@gmail.com>
Co-authored-by: Jakub F. Bortlík <jakub.bortlik@proton.me>

Author: 3 people

cmd/app/draft_notes.go

@@ -90,7 +90,7 @@ func (a draftNoteService) listDraftNotes(w http.ResponseWriter, r *http.Request)
 
 type PostDraftNoteRequest struct {
 	Comment      string `json:"comment" validate:"required"`
-	DiscussionId string `json:"discussion_id,omitempty" validate:"required"`
+	DiscussionId string `json:"discussion_id,omitempty"`
 	PositionData        // TODO: How to add validations to data from external package???
 }
 

cmd/app/git/git_test.go

@@ -198,7 +198,7 @@ func TestExtractGitInfo_FailToGetProjectRemoteUrl(t *testing.T) {
 	tC := FailTestCase{
 		desc:        "Error returned by function to get the project remote url",
 		errMsg:      "Some error",
-		expectedErr: "Could not get project Url: Some error",
+		expectedErr: "could not get project Url: Some error",
 	}
 	t.Run(tC.desc, func(t *testing.T) {
 		g := failingUrlManager{
@@ -227,7 +227,7 @@ func TestExtractGitInfo_FailToGetCurrentBranchName(t *testing.T) {
 	tC := FailTestCase{
 		desc:        "Error returned by function to get the project remote url",
 		errMsg:      "Some error",
-		expectedErr: "Failed to get current branch: Some error",
+		expectedErr: "failed to get current branch: Some error",
 	}
 	t.Run(tC.desc, func(t *testing.T) {
 		g := failingBranchManager{

lua/gitlab/reviewer/init.lua

@@ -145,6 +145,8 @@ M.jump = function(file_name, line_number, new_buffer)
     line_number = number_of_lines
   end
   vim.api.nvim_win_set_cursor(0, { line_number, 0 })
+  u.open_fold_under_cursor()
+  vim.cmd("normal! zz")
 end
 
 ---Get the data from diffview, such as line information and file name. May be used by

lua/gitlab/utils/init.lua

@@ -752,4 +752,10 @@ M.get_nested_field = function(table, field)
   end
 end
 
+M.open_fold_under_cursor = function()
+  if vim.fn.foldclosed(vim.fn.line(".")) > -1 then
+    vim.cmd("normal! zo")
+  end
+end
+
 return M

makefile

@@ -4,10 +4,10 @@ PROJECTNAME=$(shell basename "$(PWD)")
 
 ## compile: build golang project
 compile:
-	@cd cmd && go build -o bin && mv bin ../bin
+	@go build -o bin ./cmd
 ## test: run golang project tests
 test:
-	@cd cmd/app && go test
+	@go test -v ./...
 
 .PHONY: help
 all: help