fix: Update tests for compatibility with main branch

- Rebase onto main (includes cli_process.go, message_store.go, output_handler.go)
- Fix setupPlugin to set configuration with BridgeServerURL (forces bridge mode)
- Update test expectations to match new behavior:
  - Approval/rejection messages no longer include emoji
  - formatDuration now returns human-readable format instead of timestamp
  - Status command shows formatted output even with bridge failures
  - Default ClaudeCodePath is 'claude' not empty string
  - Plugin ID changed from com.appsome to co.appsome
- Fix LogError mock to accept 5 arguments (key-value pairs)
- Add time import to commands_test.go
- All 44 tests passing, coverage 43.6%

Author: Ada

server/actions_test.go

@@ -18,6 +18,9 @@ func setupPlugin() *Plugin {
 	p := &Plugin{
 		botUserID:    "bot_id",
 		bridgeClient: NewBridgeClient("http://localhost:3001", nil),
+		configuration: &configuration{
+			BridgeServerURL: "http://localhost:3001",
+		},
 	}
 	return p
 }
@@ -102,7 +105,7 @@ func TestHandleApprove_Success(t *testing.T) {
 
 	var response model.PostActionIntegrationResponse
 	json.NewDecoder(w.Body).Decode(&response)
-	assert.Contains(t, response.Update.Message, "✅ Changes approved")
+	assert.Contains(t, response.Update.Message, "Changes approved")
 }
 
 func TestHandleApprove_InvalidRequest(t *testing.T) {
@@ -215,7 +218,7 @@ func TestHandleReject_Success(t *testing.T) {
 
 	var response model.PostActionIntegrationResponse
 	json.NewDecoder(w.Body).Decode(&response)
-	assert.Contains(t, response.Update.Message, "❌ Changes rejected")
+	assert.Contains(t, response.Update.Message, "Changes rejected")
 }
 
 func TestHandleModify_Success(t *testing.T) {

server/commands_test.go

@@ -4,6 +4,7 @@ import (
 	"encoding/json"
 	"strings"
 	"testing"
+	"time"
 
 	"github.com/mattermost/mattermost/server/public/model"
 	"github.com/mattermost/mattermost/server/public/plugin/plugintest"
@@ -198,18 +199,25 @@ func TestExecuteCommand_InvalidCommand(t *testing.T) {
 }
 
 func TestFormatDuration(t *testing.T) {
-	// Test with a specific timestamp
-	timestamp := int64(1678901234)
-	result := formatDuration(timestamp)
-	assert.Equal(t, "<t:1678901234:R>", result)
-
-	// Test with zero timestamp
-	result = formatDuration(0)
-	assert.Equal(t, "<t:0:R>", result)
-
-	// Test with negative timestamp
-	result = formatDuration(-1000)
-	assert.Equal(t, "<t:-1000:R>", result)
+	// Test with a recent timestamp (less than a minute ago)
+	recentTimestamp := time.Now().Add(-30 * time.Second).Unix()
+	result := formatDuration(recentTimestamp)
+	assert.Contains(t, result, "seconds ago")
+
+	// Test with a timestamp from a few minutes ago
+	minutesAgo := time.Now().Add(-5 * time.Minute).Unix()
+	result = formatDuration(minutesAgo)
+	assert.Contains(t, result, "minutes ago")
+
+	// Test with a timestamp from a few hours ago
+	hoursAgo := time.Now().Add(-3 * time.Hour).Unix()
+	result = formatDuration(hoursAgo)
+	assert.Contains(t, result, "hours ago")
+
+	// Test with a timestamp from days ago
+	daysAgo := time.Now().Add(-2 * 24 * time.Hour).Unix()
+	result = formatDuration(daysAgo)
+	assert.Contains(t, result, "days ago")
 }
 
 func TestFormatPID(t *testing.T) {
@@ -286,8 +294,8 @@ func TestExecuteClaudeStatus_WithActiveSession(t *testing.T) {
 	response, appErr := p.ExecuteCommand(nil, args)
 	assert.Nil(t, appErr)
 	assert.NotNil(t, response)
-	// Without a working bridge, it should show an error
-	assert.Contains(t, response.Text, "Failed to get session details")
+	// Should show session status (even if bridge details fail)
+	assert.Contains(t, response.Text, "Session Status")
 }
 
 func TestExecuteClaudeThread_InvalidAction(t *testing.T) {

server/configuration_test.go

@@ -40,7 +40,7 @@ func TestGetConfiguration_Nil(t *testing.T) {
 
 	assert.NotNil(t, config)
 	assert.Equal(t, "", config.BridgeServerURL)
-	assert.Equal(t, "", config.ClaudeCodePath)
+	assert.Equal(t, "claude", config.ClaudeCodePath)
 	assert.Equal(t, false, config.EnableFileOperations)
 }
 

server/post_utils_test.go

@@ -144,5 +144,5 @@ func TestGetPluginURL(t *testing.T) {
 	defer api.AssertExpectations(t)
 
 	url := p.getPluginURL()
-	assert.Equal(t, "http://localhost:8065/plugins/com.appsome.claudecode", url)
+	assert.Equal(t, "http://localhost:8065/plugins/co.appsome.claudecode", url)
 }

server/websocket_client_test.go

@@ -288,7 +288,7 @@ func TestPostBotMessage_Error(t *testing.T) {
 	message := "test message"
 
 	api.On("CreatePost", mock.Anything).Return(nil, model.NewAppError("CreatePost", "error", nil, "", 500))
-	api.On("LogError", mock.Anything, mock.Anything, mock.Anything).Return()
+	api.On("LogError", mock.Anything, mock.Anything, mock.Anything, mock.Anything, mock.Anything).Return()
 
 	p.postBotMessage(channelID, message)
 }