From bf2160617b28d7f8b1a9b8941a1158e2b6ea64c4 Mon Sep 17 00:00:00 2001 From: Igor Sirotin Date: Mon, 26 Aug 2024 11:35:31 +0100 Subject: [PATCH] temp_: try TestSendEmoji without transport layer --- protocol/messenger.go | 6 ++ protocol/messenger_emoji_test.go | 130 +++++++++++++++++++++++++------ 2 files changed, 112 insertions(+), 24 deletions(-) diff --git a/protocol/messenger.go b/protocol/messenger.go index a2987eb34..a7d8284a4 100644 --- a/protocol/messenger.go +++ b/protocol/messenger.go @@ -73,6 +73,7 @@ import ( "github.com/status-im/status-go/services/wallet/token" "github.com/status-im/status-go/signal" "github.com/status-im/status-go/telemetry" + "testing" ) const ( @@ -2101,6 +2102,11 @@ func (m *Messenger) dispatchPairInstallationMessage(ctx context.Context, spec co } func (m *Messenger) dispatchMessage(ctx context.Context, rawMessage common.RawMessage) (common.RawMessage, error) { + if testing.Testing() && m.dispatchMessageTestCallback != nil { + m.dispatchMessageTestCallback(rawMessage) + return rawMessage, nil + } + var err error var id []byte logger := m.logger.With(zap.String("site", "dispatchMessage"), zap.String("chatID", rawMessage.LocalChatID)) diff --git a/protocol/messenger_emoji_test.go b/protocol/messenger_emoji_test.go index ade494935..2c8c65922 100644 --- a/protocol/messenger_emoji_test.go +++ b/protocol/messenger_emoji_test.go @@ -13,6 +13,10 @@ import ( "github.com/status-im/status-go/multiaccounts" "github.com/status-im/status-go/protocol/common" "github.com/status-im/status-go/protocol/protobuf" + "github.com/status-im/status-go/protocol/transport" + "github.com/status-im/status-go/protocol/v1" + "go.uber.org/zap" + "time" ) func TestMessengerEmojiSuite(t *testing.T) { @@ -23,7 +27,46 @@ type MessengerEmojiSuite struct { MessengerBaseTestSuite } +func (s *MessengerEmojiSuite) handleRawMessage(m *Messenger, sender *Messenger, rawMessage *common.RawMessage) *MessengerResponse { + wrappedMessage, err := protocol.WrapMessageV1(rawMessage.Payload, rawMessage.MessageType, sender.identity) + s.Require().NoError(err) + + messageID := protocol.MessageID(sender.IdentityPublicKey(), wrappedMessage) + + statusMessage := &protocol.StatusMessage{ + ApplicationLayer: protocol.ApplicationLayer{ + Payload: rawMessage.Payload, + ID: protocol.MessageID(sender.IdentityPublicKey(), wrappedMessage), + SigPubKey: sender.IdentityPublicKey(), + Type: rawMessage.MessageType, + }, + TransportLayer: protocol.TransportLayer{ + Message: &types.Message{ + Timestamp: uint32(sender.getTimesource().GetCurrentTime()), + }, + }, + } + + messageState := m.buildMessageState() + messageState.CurrentMessageState = &CurrentMessageState{ + MessageID: messageID.String(), + WhisperTimestamp: sender.getTimesource().GetCurrentTime(), // uint64(statusMessage.TransportLayer.Message.Timestamp) * 1000, + Contact: sender.selfContact, + PublicKey: statusMessage.SigPubKey(), + StatusMessage: statusMessage, + } + err = m.dispatchToHandler(messageState, statusMessage.ApplicationLayer.Payload, statusMessage, transport.Filter{}, false) + s.Require().NoError(err) + + response, err := m.saveDataAndPrepareResponse(messageState) + s.Require().NoError(err) + + return response +} + func (s *MessengerEmojiSuite) TestSendEmoji() { + start := time.Now() + alice := s.m alice.account = &multiaccounts.Account{KeyUID: "0xdeadbeef"} key, err := crypto.GenerateKey() @@ -31,7 +74,13 @@ func (s *MessengerEmojiSuite) TestSendEmoji() { bob, err := newMessengerWithKey(s.shh, key, s.logger, nil) s.Require().NoError(err) - defer TearDownMessenger(&s.Suite, bob) + defer func() { + s.logger.Debug("<<< 5", zap.Duration("duration", time.Since(start))) + TearDownMessenger(&s.Suite, bob) + s.logger.Debug("<<< 6", zap.Duration("duration", time.Since(start))) + }() + + s.logger.Debug("<<< 1", zap.Duration("duration", time.Since(start))) chatID := statusChatID @@ -51,60 +100,93 @@ func (s *MessengerEmojiSuite) TestSendEmoji() { // Send chat message from alice to bob + aliceDispatchedMessages := make([]common.RawMessage, 0) + alice.dispatchMessageTestCallback = func(msg common.RawMessage) { + aliceDispatchedMessages = append(aliceDispatchedMessages, msg) + } + message := buildTestMessage(*chat) _, err = alice.SendChatMessage(context.Background(), message) s.NoError(err) - // Wait for message to arrive to bob - response, err := WaitOnMessengerResponse( - bob, - func(r *MessengerResponse) bool { return len(r.Messages()) > 0 }, - "no messages", - ) - s.Require().NoError(err) + //bob.handle + s.Require().Len(aliceDispatchedMessages, 1) + rawMessage := aliceDispatchedMessages[0] + response := s.handleRawMessage(bob, alice, &rawMessage) + s.logger.Debug("<<< 2", zap.Duration("duration", time.Since(start))) + + //// Wait for message to arrive to bob + //response, err := WaitOnMessengerResponse( + // bob, + // func(r *MessengerResponse) bool { return len(r.Messages()) > 0 }, + // "no messages", + //) + //s.Require().NoError(err) s.Require().Len(response.Messages(), 1) messageID := response.Messages()[0].ID // Respond with an emoji, donald trump style + bobDispatchedMessages := make([]common.RawMessage, 0) + bob.dispatchMessageTestCallback = func(msg common.RawMessage) { + bobDispatchedMessages = append(bobDispatchedMessages, msg) + } + response, err = bob.SendEmojiReaction(context.Background(), chat.ID, messageID, protobuf.EmojiReaction_SAD) s.Require().NoError(err) s.Require().Len(response.EmojiReactions(), 1) + s.Require().Len(bobDispatchedMessages, 1) + rawMessage = bobDispatchedMessages[0] + bobDispatchedMessages = make([]common.RawMessage, 0) + + s.Require().Equal(chatID, rawMessage.LocalChatID) + s.Require().Equal(common.ResendTypeNone, rawMessage.ResendType) + s.Require().Equal(protobuf.ApplicationMetadataMessage_EMOJI_REACTION, rawMessage.MessageType) + + response = s.handleRawMessage(alice, bob, &rawMessage) + s.Require().Len(response.EmojiReactions(), 1) + + //// Wait for the emoji to arrive to alice + //response, err = WaitOnMessengerResponse( + // alice, + // func(r *MessengerResponse) bool { return len(r.EmojiReactions()) == 1 }, + // "no emoji", + //) + //s.Require().NoError(err) + emojiID := response.EmojiReactions()[0].ID() - - // Wait for the emoji to arrive to alice - response, err = WaitOnMessengerResponse( - alice, - func(r *MessengerResponse) bool { return len(r.EmojiReactions()) == 1 }, - "no emoji", - ) - s.Require().NoError(err) - s.Require().Len(response.EmojiReactions(), 1) s.Require().Equal(response.EmojiReactions()[0].ID(), emojiID) s.Require().Equal(response.EmojiReactions()[0].Type, protobuf.EmojiReaction_SAD) + s.logger.Debug("<<< 3", zap.Duration("duration", time.Since(start))) + // Retract the emoji response, err = bob.SendEmojiReactionRetraction(context.Background(), emojiID) s.Require().NoError(err) s.Require().Len(response.EmojiReactions(), 1) s.Require().True(response.EmojiReactions()[0].Retracted) - // Wait for the emoji to arrive to alice - response, err = WaitOnMessengerResponse( - alice, - func(r *MessengerResponse) bool { return len(r.EmojiReactions()) == 1 }, - "no emoji", - ) - s.Require().NoError(err) + s.Require().Len(bobDispatchedMessages, 1) + rawMessage = bobDispatchedMessages[0] + response = s.handleRawMessage(alice, bob, &rawMessage) + //// Wait for the emoji to arrive to alice + //response, err = WaitOnMessengerResponse( + // alice, + // func(r *MessengerResponse) bool { return len(r.EmojiReactions()) == 1 }, + // "no emoji", + //) + //s.Require().NoError(err) s.Require().Len(response.EmojiReactions(), 1) s.Require().Equal(response.EmojiReactions()[0].ID(), emojiID) s.Require().Equal(response.EmojiReactions()[0].Type, protobuf.EmojiReaction_SAD) s.Require().True(response.EmojiReactions()[0].Retracted) + + s.logger.Debug("<<< 4", zap.Duration("duration", time.Since(start))) } func (s *MessengerEmojiSuite) TestEmojiPrivateGroup() {