Fix rendering link attachment preview with other attachment types (#659)

* Fix rendering link attachment preview with other attachment types

* Update CHANGELOG.md

* Only evaluate the hasOnlyLinks if there is at least one link attachment

Author: nuno-vieira

CHANGELOG.md

@@ -7,6 +7,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - Make `VoiceRecordingButton` public [#658](https://github.com/GetStream/stream-chat-swiftui/pull/658)
 ### 🐞 Fixed
 - Fix message long press taking too much time to show actions [#648](https://github.com/GetStream/stream-chat-swiftui/pull/648)
+- Fix rendering link attachment preview with other attachment types [#659](https://github.com/GetStream/stream-chat-swiftui/pull/659)
 ### 🔄 Changed
 - Message composer now uses `.uploadFile` capability when showing attachment picker icon [#646](https://github.com/GetStream/stream-chat-swiftui/pull/646)
 - `ChannelInfoView` now uses `.updateChannelMembers` capability to show "Add Users" button [#651](https://github.com/GetStream/stream-chat-swiftui/pull/651)

Sources/StreamChatSwiftUI/ChatChannel/MessageList/MessageView.swift

@@ -44,7 +44,8 @@ public struct MessageView<Factory: ViewFactory>: View {
             } else if let poll = message.poll {
                 factory.makePollView(message: message, poll: poll, isFirst: isFirst)
             } else if !message.attachmentCounts.isEmpty {
-                if messageTypeResolver.hasLinkAttachment(message: message) {
+                let hasOnlyLinks = { message.attachmentCounts.keys.allSatisfy { $0 == .linkPreview } }
+                if messageTypeResolver.hasLinkAttachment(message: message) && hasOnlyLinks() {
                     factory.makeLinkAttachmentView(
                         for: message,
                         isFirst: isFirst,

StreamChatSwiftUITests/Tests/ChatChannel/MessageView_Tests.swift

@@ -388,6 +388,33 @@ class MessageView_Tests: StreamChatTestCase {
         assertSnapshot(matching: view, as: .image(perceptualPrecision: precision))
     }
 
+    func test_linkAttachmentView_shouldNotRenderLinkPreviewWithOtherAttachments() {
+        // Given
+        let messageWithLinkAndImages = ChatMessage.mock(
+            id: .unique,
+            cid: .unique,
+            text: "https://getstream.io",
+            author: .mock(id: .unique),
+            attachments: [
+                ChatChannelTestHelpers.imageAttachments[0],
+                ChatChannelTestHelpers.videoAttachments[0]
+            ]
+        )
+
+        // When
+        let view = MessageView(
+            factory: DefaultViewFactory.shared,
+            message: messageWithLinkAndImages,
+            contentWidth: defaultScreenSize.width,
+            isFirst: true,
+            scrolledId: .constant(nil)
+        )
+        .applyDefaultSize()
+
+        // Then
+        assertSnapshot(matching: view, as: .image(perceptualPrecision: precision))
+    }
+
     func test_deletedMessageView_snapshot() {
         // Given
         let message = ChatMessage.mock(