add assortions and check for nil response

Author: tommaso-moro

pkg/github/minimal_types.go

@@ -354,6 +354,10 @@ func convertToMinimalIssueComment(comment *github.IssueComment) MinimalIssueComm
 func convertToMinimalFileContentResponse(resp *github.RepositoryContentResponse) MinimalFileContentResponse {
 	m := MinimalFileContentResponse{}
 
+	if resp == nil {
+		return m
+	}
+
 	if c := resp.Content; c != nil {
 		m.Content = &MinimalFileContent{
 			Name:    c.GetName(),

pkg/github/repositories_test.go

@@ -1442,10 +1442,19 @@ func Test_CreateOrUpdateFile(t *testing.T) {
 			assert.Equal(t, tc.expectedContent.Content.GetName(), returnedContent.Content.Name)
 			assert.Equal(t, tc.expectedContent.Content.GetPath(), returnedContent.Content.Path)
 			assert.Equal(t, tc.expectedContent.Content.GetSHA(), returnedContent.Content.SHA)
+			assert.Equal(t, tc.expectedContent.Content.GetSize(), returnedContent.Content.Size)
+			assert.Equal(t, tc.expectedContent.Content.GetHTMLURL(), returnedContent.Content.HTMLURL)
 
 			// Verify commit
 			assert.Equal(t, tc.expectedContent.Commit.GetSHA(), returnedContent.Commit.SHA)
 			assert.Equal(t, tc.expectedContent.Commit.GetMessage(), returnedContent.Commit.Message)
+			assert.Equal(t, tc.expectedContent.Commit.GetHTMLURL(), returnedContent.Commit.HTMLURL)
+
+			// Verify commit author
+			require.NotNil(t, returnedContent.Commit.Author)
+			assert.Equal(t, tc.expectedContent.Commit.Author.GetName(), returnedContent.Commit.Author.Name)
+			assert.Equal(t, tc.expectedContent.Commit.Author.GetEmail(), returnedContent.Commit.Author.Email)
+			assert.NotEmpty(t, returnedContent.Commit.Author.Date)
 		})
 	}
 }