From 03159f1bca4c657bb82b612780c9d922ced4efcb Mon Sep 17 00:00:00 2001
From: shash256 <111925100+shash256@users.noreply.github.com>
Date: Thu, 24 Jul 2025 15:22:21 +0530
Subject: [PATCH] chore: address review comment

---
 sds/sds_test.go | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/sds/sds_test.go b/sds/sds_test.go
index 433cc59..5e80cf7 100644
--- a/sds/sds_test.go
+++ b/sds/sds_test.go
@@ -570,18 +570,16 @@ func TestMultiChannel_SingleRM(t *testing.T) {
 	require.Equal(t, msg1, *unwrapped1.Message)
 
 	// Verify channel ID is extracted correctly
-	if unwrapped1.ChannelId != nil {
-		require.Equal(t, channel1, *unwrapped1.ChannelId)
-	}
+	require.NotNil(t, unwrapped1.ChannelId, "Expected ChannelId to be not nil")
+	require.Equal(t, channel1, *unwrapped1.ChannelId)
 
 	unwrapped2, err := rm.UnwrapReceivedMessage(wrappedMsg2)
 	require.NoError(t, err)
 	require.Equal(t, msg2, *unwrapped2.Message)
 
 	// Verify channel ID is extracted correctly
-	if unwrapped2.ChannelId != nil {
-		require.Equal(t, channel2, *unwrapped2.ChannelId)
-	}
+	require.NotNil(t, unwrapped2.ChannelId, "Expected ChannelId to be not nil")
+	require.Equal(t, channel2, *unwrapped2.ChannelId)
 
 	// Test that the same RM can handle dependencies for different channels
 	err = rm.MarkDependenciesMet([]MessageID{msgID1}, channel1)