From 6edae51a239e12e53f9c650987a10b5702b09060 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lo=C3=AFc=20Dachary?= <loic@dachary.org>
Date: Sun, 12 Nov 2023 15:12:57 +0100
Subject: [PATCH] test POST
 /{username}/{reponame}/{type:issues|pulls}/{index}/content-history/soft-delete

---
 models/fixtures/issue_content_history.yml |  1 +
 tests/integration/issue_test.go           | 67 +++++++++++++++++++++++
 2 files changed, 68 insertions(+)
 create mode 100644 models/fixtures/issue_content_history.yml

diff --git a/models/fixtures/issue_content_history.yml b/models/fixtures/issue_content_history.yml
new file mode 100644
index 0000000000..ca780a73aa
--- /dev/null
+++ b/models/fixtures/issue_content_history.yml
@@ -0,0 +1 @@
+[] # empty
diff --git a/tests/integration/issue_test.go b/tests/integration/issue_test.go
index d5c206c77f..db978ba3cf 100644
--- a/tests/integration/issue_test.go
+++ b/tests/integration/issue_test.go
@@ -306,6 +306,73 @@ func TestIssueCommentUpdate(t *testing.T) {
 
 	comment = unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{ID: commentID})
 	assert.Equal(t, modifiedContent, comment.Content)
+
+	historyBefore := unittest.AssertExistsAndLoadBean(t, &issues_model.ContentHistory{CommentID: commentID})
+	assert.Equal(t, comment1, historyBefore.ContentText)
+	assert.False(t, historyBefore.IsDeleted)
+
+	softDelete := fmt.Sprintf("content-history/soft-delete?comment_id=%d&history_id=%d", commentID, historyBefore.ID)
+
+	// Using the ID of a comment that does not belong to the repository must fail
+	{
+		session5 := loginUser(t, "user5")
+		otherIssueURL := testNewIssue(t, session5, "user5", "repo4", "Other Title", "Other Description")
+
+		req = NewRequestWithValues(t, "POST", fmt.Sprintf("%s/%s", otherIssueURL, softDelete), map[string]string{
+			"_csrf": GetCSRF(t, session5, otherIssueURL),
+		})
+		session5.MakeRequest(t, req, http.StatusNotFound)
+	}
+
+	req = NewRequestWithValues(t, "POST", fmt.Sprintf("%s/%s", issueURL, softDelete), map[string]string{
+		"_csrf": GetCSRF(t, session, issueURL),
+	})
+	session.MakeRequest(t, req, http.StatusOK)
+
+	historyAfter := unittest.AssertExistsAndLoadBean(t, &issues_model.ContentHistory{ID: historyBefore.ID})
+	assert.True(t, historyAfter.IsDeleted)
+}
+
+func TestIssueContentUpdate(t *testing.T) {
+	defer tests.PrepareTestEnv(t)()
+	session := loginUser(t, "user2")
+	content1 := "Content one"
+	issueURL, issue := testIssueWithBean(t, "user2", 1, "Title", content1)
+	modifiedContent := content1 + "MODIFIED"
+
+	req := NewRequestWithValues(t, "POST", fmt.Sprintf("%s/content", issueURL), map[string]string{
+		"_csrf":   GetCSRF(t, session, issueURL),
+		"content": modifiedContent,
+	})
+	session.MakeRequest(t, req, http.StatusOK)
+
+	issue = unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: issue.ID})
+	assert.Equal(t, modifiedContent, issue.Content)
+
+	historyBefore := unittest.AssertExistsAndLoadBean(t, &issues_model.ContentHistory{IssueID: issue.ID})
+	assert.Equal(t, content1, historyBefore.ContentText)
+	assert.False(t, historyBefore.IsDeleted)
+
+	softDelete := fmt.Sprintf("content-history/soft-delete?history_id=%d", historyBefore.ID)
+
+	// Using the ID of a comment that does not belong to the repository must fail
+	session5 := loginUser(t, "user5")
+	otherIssueURL := testNewIssue(t, session5, "user5", "repo4", "Other Title", "Other Description")
+	req = NewRequestWithValues(t, "POST", fmt.Sprintf("%s/%s", otherIssueURL, softDelete), map[string]string{
+		"_csrf": GetCSRF(t, session5, otherIssueURL),
+	})
+	session5.MakeRequest(t, req, http.StatusNotFound)
+	historyIdentical := unittest.AssertExistsAndLoadBean(t, &issues_model.ContentHistory{ID: historyBefore.ID})
+	assert.Equal(t, content1, historyIdentical.ContentText)
+	assert.False(t, historyIdentical.IsDeleted)
+
+	req = NewRequestWithValues(t, "POST", fmt.Sprintf("%s/%s", issueURL, softDelete), map[string]string{
+		"_csrf": GetCSRF(t, session, issueURL),
+	})
+	session.MakeRequest(t, req, http.StatusOK)
+
+	historyAfter := unittest.AssertExistsAndLoadBean(t, &issues_model.ContentHistory{ID: historyBefore.ID})
+	assert.True(t, historyAfter.IsDeleted)
 }
 
 func TestIssueReaction(t *testing.T) {