feat(sdk): Add federation version mock endpoint and log server version in FFI

Co-authored-by: pixlwave <6060466+pixlwave@users.noreply.github.com>

Author: Copilot

bindings/matrix-sdk-ffi/src/client.rs

@@ -1608,15 +1608,6 @@ impl Client {
         Ok(self.inner.server_versions().await?.contains(&ruma::api::MatrixVersion::V1_13))
     }
 
-    /// Get the server version information from the federation API.
-    ///
-    /// This method calls the `/_matrix/federation/v1/version` endpoint to get
-    /// both the server name and version.
-    pub async fn server_version(&self) -> Result<ServerVersionInfo, ClientError> {
-        let server_info = self.inner.server_version().await?;
-        Ok(server_info.into())
-    }
-
     /// Checks if the server supports the LiveKit RTC focus for placing calls.
     pub async fn is_livekit_rtc_supported(&self) -> Result<bool, ClientError> {
         Ok(self
@@ -1806,23 +1797,6 @@ impl From<search_users::v3::Response> for SearchUsersResults {
     }
 }
 
-#[derive(uniffi::Record)]
-pub struct ServerVersionInfo {
-    /// The server name.
-    pub server_name: String,
-    /// The server version.
-    pub version: String,
-}
-
-impl From<matrix_sdk::ServerVersionInfo> for ServerVersionInfo {
-    fn from(value: matrix_sdk::ServerVersionInfo) -> Self {
-        Self {
-            server_name: value.server_name,
-            version: value.version,
-        }
-    }
-}
-
 #[derive(uniffi::Record)]
 pub struct UserProfile {
     pub user_id: String,

bindings/matrix-sdk-ffi/src/client_builder.rs

@@ -574,6 +574,17 @@ impl ClientBuilder {
 
         let sdk_client = inner_builder.build().await?;
 
+        // Log server version information at info level
+        if let Ok(server_info) = sdk_client.server_version().await {
+            tracing::info!(
+                server_name = %server_info.server_name,
+                version = %server_info.version,
+                "Connected to Matrix server"
+            );
+        } else {
+            tracing::debug!("Could not retrieve server version information");
+        }
+
         Ok(Arc::new(
             Client::new(
                 sdk_client,

crates/matrix-sdk/src/test_utils/mocks/mod.rs

@@ -1377,6 +1377,12 @@ impl MatrixMockServer {
         )));
         self.mock_endpoint(mock, EnablePushRuleEndpoint).expect_default_access_token()
     }
+
+    /// Create a prebuilt mock for the federation version endpoint.
+    pub fn mock_federation_version(&self) -> MockEndpoint<'_, FederationVersionEndpoint> {
+        let mock = Mock::given(method("GET")).and(path("/_matrix/federation/v1/version"));
+        self.mock_endpoint(mock, FederationVersionEndpoint)
+    }
 }
 
 /// Parameter to [`MatrixMockServer::sync_room`].
@@ -3955,3 +3961,25 @@ impl<'a> MockEndpoint<'a, EnablePushRuleEndpoint> {
         self.ok_empty_json()
     }
 }
+
+/// A prebuilt mock for the federation version endpoint.
+pub struct FederationVersionEndpoint;
+
+impl<'a> MockEndpoint<'a, FederationVersionEndpoint> {
+    /// Returns a successful response with the given server name and version.
+    pub fn ok(self, server_name: &str, version: &str) -> MatrixMock<'a> {
+        let response_body = json!({
+            "server": {
+                "name": server_name,
+                "version": version
+            }
+        });
+        self.respond_with(ResponseTemplate::new(200).set_body_json(response_body))
+    }
+
+    /// Returns a successful response with empty/missing server information.
+    pub fn ok_empty(self) -> MatrixMock<'a> {
+        let response_body = json!({});
+        self.respond_with(ResponseTemplate::new(200).set_body_json(response_body))
+    }
+}

crates/matrix-sdk/tests/integration/client.rs

@@ -1494,22 +1494,15 @@ async fn test_room_sync_state_after() {
 #[async_test]
 async fn test_server_version() {
     use matrix_sdk::test_utils::mocks::MatrixMockServer;
-    use serde_json::json;
-    use wiremock::{matchers::{method, path}, Mock, ResponseTemplate};
 
     let server = MatrixMockServer::new().await;
     let client = server.client_builder().build().await;
 
     // Mock the federation version endpoint
-    Mock::given(method("GET"))
-        .and(path("/_matrix/federation/v1/version"))
-        .respond_with(ResponseTemplate::new(200).set_body_json(json!({
-            "server": {
-                "name": "Synapse",
-                "version": "1.70.0"
-            }
-        })))
-        .mount(server.server())
+    server
+        .mock_federation_version()
+        .ok("Synapse", "1.70.0")
+        .mount()
         .await;
 
     let server_info = client.server_version().await.unwrap();
@@ -1521,17 +1514,15 @@ async fn test_server_version() {
 #[async_test]
 async fn test_server_version_with_missing_fields() {
     use matrix_sdk::test_utils::mocks::MatrixMockServer;
-    use serde_json::json;
-    use wiremock::{matchers::{method, path}, Mock, ResponseTemplate};
 
     let server = MatrixMockServer::new().await;
     let client = server.client_builder().build().await;
 
     // Mock the federation version endpoint with missing fields
-    Mock::given(method("GET"))
-        .and(path("/_matrix/federation/v1/version"))
-        .respond_with(ResponseTemplate::new(200).set_body_json(json!({})))
-        .mount(server.server())
+    server
+        .mock_federation_version()
+        .ok_empty()
+        .mount()
         .await;
 
     let server_info = client.server_version().await.unwrap();