internal_links: Add variable realmUrlWithSlash to add '/'

Updates the URL construction to include a trailing slash
before the fragment identifier. This ensures that the URL
is properly formatted and makes the url linkified.

Fixes: #845

Author: satyam372

lib/model/internal_link.dart

@@ -96,7 +96,11 @@ Uri narrowLink(PerAccountStore store, Narrow narrow, {int? nearMessageId}) {
     fragment.write('/near/$nearMessageId');
   }
 
-  return store.realmUrl.replace(fragment: fragment.toString());
+  final realmUrlWithSlash = store.realmUrl.path.endsWith('/')
+      ? store.realmUrl
+      : store.realmUrl.replace(path: '${store.realmUrl.path}/');
+
+  return realmUrlWithSlash.replace(fragment: fragment.toString());
 }
 
 /// A [Narrow] from a given URL, on `store`'s realm.

test/widgets/action_sheet_test.dart

@@ -38,11 +38,14 @@ late FakeApiConnection connection;
 Future<void> setupToMessageActionSheet(WidgetTester tester, {
   required Message message,
   required Narrow narrow,
+  Account? account,
 }) async {
   addTearDown(testBinding.reset);
 
-  await testBinding.globalStore.add(eg.selfAccount, eg.initialSnapshot());
-  final store = await testBinding.globalStore.perAccount(eg.selfAccount.id);
+  account ??= eg.selfAccount;
+  final initialSnapshot = eg.initialSnapshot(zulipFeatureLevel: account.zulipFeatureLevel);
+  await testBinding.globalStore.add(account, initialSnapshot);
+  final store = await testBinding.globalStore.perAccount(account.id);
   await store.addUser(eg.user(userId: message.senderId));
   if (message is StreamMessage) {
     final stream = eg.stream(streamId: message.streamId);
@@ -60,8 +63,8 @@ Future<void> setupToMessageActionSheet(WidgetTester tester, {
     historyLimited: false,
     messages: [message],
   ).toJson());
-
-  await tester.pumpWidget(TestZulipApp(accountId: eg.selfAccount.id,
+  await tester.pumpAndSettle();
+  await tester.pumpWidget(TestZulipApp(accountId: account.id,
     child: MessageListPage(initNarrow: narrow)));
 
   // global store, per-account store, and message list get loaded
@@ -551,6 +554,24 @@ void main() {
       final expectedLink = narrowLink(store, narrow, nearMessageId: message.id).toString();
       check(await Clipboard.getData('text/plain')).isNotNull().text.equals(expectedLink);
     });
+
+    testWidgets('Full Url Testing', (tester) async {
+      final message = eg.streamMessage();
+      final narrow = TopicNarrow.ofMessage(message);
+      await setupToMessageActionSheet(tester, message: message, narrow: narrow);
+      await tapCopyMessageLinkButton(tester);
+      await tester.pump(Duration.zero);
+
+      final customAccount = eg.selfAccount.copyWith(
+          id: 2, realmUrl: Uri.parse('https://chat.example'));
+      await setupToMessageActionSheet(
+          tester, message: message, narrow: narrow, account: customAccount);
+      final customAccountStore = await testBinding.globalStore.perAccount(customAccount.id);
+      final expectedLink = narrowLink(
+          customAccountStore, narrow, nearMessageId: message.id).toString();
+
+      check(await Clipboard.getData('text/plain')).isNotNull().text.equals(expectedLink);
+    });
   });
 
   group('ShareButton', () {
@@ -616,4 +637,4 @@ void main() {
       check(mockSharePlus.sharedString).isNull();
     });
   });
-}
+}