From 1479de9050ed4ecad3bb8f8d74b169635b1c26e1 Mon Sep 17 00:00:00 2001
From: David Zhang <crispgm@gmail.com>
Date: Wed, 29 Nov 2023 12:20:31 +0800
Subject: [PATCH] chore(test): drop message id constant

---
 api_message_test.go | 25 ++++++++++++-------------
 lark_test.go        |  5 +----
 msg_buf_test.go     |  4 ++--
 3 files changed, 15 insertions(+), 19 deletions(-)

diff --git a/api_message_test.go b/api_message_test.go
index 0a529ec..8c65ca6 100644
--- a/api_message_test.go
+++ b/api_message_test.go
@@ -409,13 +409,6 @@ func TestMessageCRUD(t *testing.T) {
 	if assert.NoError(t, err) {
 		t.Log(receipt.Data.ReadUsers)
 	}
-	// receipt read
-	receiptOld, err := bot.MessageReadReceipt(testMessageID)
-	if assert.NoError(t, err) {
-		// failed because the message ID will be outdated after tens of days
-		// assert.NotEmpty(t, receiptOld.Data.ReadUsers)
-		t.Log(receiptOld.Data.ReadUsers)
-	}
 }
 
 func TestIdempotentMessage(t *testing.T) {
@@ -437,12 +430,18 @@ func TestIdempotentMessage(t *testing.T) {
 }
 
 func TestPinMessages(t *testing.T) {
-	resp, err := bot.PinMessage(testMessageID)
+	msg := NewMsgBuffer(MsgText)
+	om := msg.BindEmail(testUserEmail).Text("hello, world").Build()
+	resp, err := bot.PostMessage(om)
 	if assert.NoError(t, err) {
-		assert.Equal(t, 0, resp.Code)
-		assert.Equal(t, testMessageID, resp.Data.Pin.MessageID)
-		unpinResp, err := bot.UnpinMessage(testMessageID)
-		assert.NoError(t, err)
-		assert.Equal(t, 0, unpinResp.Code)
+		messageID := resp.Data.MessageID
+		resp, err := bot.PinMessage(messageID)
+		if assert.NoError(t, err) {
+			assert.Equal(t, 0, resp.Code)
+			assert.Equal(t, messageID, resp.Data.Pin.MessageID)
+			unpinResp, err := bot.UnpinMessage(messageID)
+			assert.NoError(t, err)
+			assert.Equal(t, 0, unpinResp.Code)
+		}
 	}
 }
diff --git a/lark_test.go b/lark_test.go
index 8502574..d9cebfc 100644
--- a/lark_test.go
+++ b/lark_test.go
@@ -24,7 +24,6 @@ var (
 	testUserID          string
 	testUserUnionID     string
 	testGroupChatID     string
-	testMessageID       string
 	testWebhookV1       string
 	testWebhookV2       string
 	testWebhookV2Signed string
@@ -48,7 +47,6 @@ func newTestBot() *Bot {
 	testUserUnionID = os.Getenv("LARK_UNION_ID")
 	testUserOpenID = os.Getenv("LARK_OPEN_ID")
 	testGroupChatID = os.Getenv("LARK_CHAT_ID")
-	testMessageID = os.Getenv("LARK_MESSAGE_ID")
 	testWebhookV1 = os.Getenv("LARK_WEBHOOK_V1")
 	testWebhookV2 = os.Getenv("LARK_WEBHOOK_V2")
 	testWebhookV2Signed = os.Getenv("LARK_WEBHOOK_V2_SIGNED")
@@ -58,8 +56,7 @@ func newTestBot() *Bot {
 		len(testUserID) == 0 ||
 		len(testUserUnionID) == 0 ||
 		len(testUserOpenID) == 0 ||
-		len(testGroupChatID) == 0 ||
-		len(testMessageID) == 0 {
+		len(testGroupChatID) == 0 {
 		panic("insufficient test environment")
 	}
 	return NewChatBot(testAppID, testAppSecret)
diff --git a/msg_buf_test.go b/msg_buf_test.go
index 1af4820..9f54e65 100644
--- a/msg_buf_test.go
+++ b/msg_buf_test.go
@@ -39,8 +39,8 @@ func TestBindingUserIDs(t *testing.T) {
 	assert.Equal(t, "333444", msgUserID.UserID)
 
 	mb.Clear()
-	msgReplyID := mb.BindReply(testMessageID).Build()
-	assert.Equal(t, testMessageID, msgReplyID.RootID)
+	msgReplyID := mb.BindReply("om_f779ffe0ffa3d1b94fc1ef5fcb6f1063").Build()
+	assert.Equal(t, "om_f779ffe0ffa3d1b94fc1ef5fcb6f1063", msgReplyID.RootID)
 }
 
 func TestMsgShareChat(t *testing.T) {