Element-R: Add current version of the rust-sdk and vodozemac (#11785)

* Add current version of the rust-sdk and vodozemac * Use `CryptoAPI#getVersion` for old crypto * Update i18n * Fix test * Remove wrong comment
matrix-org · Oct 25, 2023 · 48a89a2 · 48a89a2
1 parent 5e8d274
commit 48a89a2
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/src/components/views/settings/tabs/user/HelpUserSettingsTab.tsx b/src/components/views/settings/tabs/user/HelpUserSettingsTab.tsx
@@ -69,17 +69,14 @@ export default class HelpUserSettingsTab extends React.Component<IProps, IState>
             });
     }
 
-    private getVersionInfo(): { appVersion: string; olmVersion: string } {
+    private getVersionInfo(): { appVersion: string; cryptoVersion: string } {
         const brand = SdkConfig.get().brand;
         const appVersion = this.state.appVersion || "unknown";
-        const olmVersionTuple = this.context.olmVersion;
-        const olmVersion = olmVersionTuple
-            ? `${olmVersionTuple[0]}.${olmVersionTuple[1]}.${olmVersionTuple[2]}`
-            : "<not-enabled>";
+        const cryptoVersion = this.context.getCrypto()?.getVersion() ?? "<not-enabled>";
 
         return {
             appVersion: `${_t("setting|help_about|brand_version", { brand })} ${appVersion}`,
-            olmVersion: `${_t("setting|help_about|olm_version")} ${olmVersion}`,
+            cryptoVersion: `${_t("setting|help_about|crypto_version")} ${cryptoVersion}`,
         };
     }
 
@@ -220,8 +217,8 @@ export default class HelpUserSettingsTab extends React.Component<IProps, IState>
     }
 
     private getVersionTextToCopy = (): string => {
-        const { appVersion, olmVersion } = this.getVersionInfo();
-        return `${appVersion}\n${olmVersion}`;
+        const { appVersion, cryptoVersion } = this.getVersionInfo();
+        return `${appVersion}\n${cryptoVersion}`;
     };
 
     public render(): React.ReactNode {
@@ -302,7 +299,7 @@ export default class HelpUserSettingsTab extends React.Component<IProps, IState>
             );
         }
 
-        const { appVersion, olmVersion } = this.getVersionInfo();
+        const { appVersion, cryptoVersion } = this.getVersionInfo();
 
         return (
             <SettingsTab>
@@ -314,7 +311,7 @@ export default class HelpUserSettingsTab extends React.Component<IProps, IState>
                             <CopyableText getTextToCopy={this.getVersionTextToCopy}>
                                 {appVersion}
                                 <br />
-                                {olmVersion}
+                                {cryptoVersion}
                                 <br />
                             </CopyableText>
                             {updateButton}

diff --git a/src/i18n/strings/en_EN.json b/src/i18n/strings/en_EN.json
@@ -2385,11 +2385,11 @@
             "brand_version": "%(brand)s version:",
             "chat_bot": "Chat with %(brand)s Bot",
             "clear_cache_reload": "Clear cache and reload",
+            "crypto_version": "Crypto version:",
             "help_link": "For help with using %(brand)s, click <a>here</a>.",
             "help_link_chat_bot": "For help with using %(brand)s, click <a>here</a> or start a chat with our bot using the button below.",
             "homeserver": "Homeserver is <code>%(homeserverUrl)s</code>",
             "identity_server": "Identity server is <code>%(identityServerUrl)s</code>",
-            "olm_version": "Olm version:",
             "title": "Help & About",
             "versions": "Versions"
         }

diff --git a/test/components/views/dialogs/UserSettingsDialog-test.tsx b/test/components/views/dialogs/UserSettingsDialog-test.tsx
@@ -28,6 +28,7 @@ import {
     mockClientMethodsUser,
     mockClientMethodsServer,
     mockPlatformPeg,
+    mockClientMethodsCrypto,
 } from "../../../test-utils";
 import { UIFeature } from "../../../../src/settings/UIFeature";
 import { SettingLevel } from "../../../../src/settings/SettingLevel";
@@ -70,6 +71,7 @@ describe("<UserSettingsDialog />", () => {
         mockClient = getMockClientWithEventEmitter({
             ...mockClientMethodsUser(userId),
             ...mockClientMethodsServer(),
+            ...mockClientMethodsCrypto(),
         });
         sdkContext = new SdkContextClass();
         sdkContext.client = mockClient;

diff --git a/test/test-utils/client.ts b/test/test-utils/client.ts
@@ -168,5 +168,6 @@ export const mockClientMethodsCrypto = (): Partial<
         isCrossSigningReady: jest.fn().mockResolvedValue(true),
         isSecretStorageReady: jest.fn(),
         getSessionBackupPrivateKey: jest.fn(),
+        getVersion: jest.fn().mockReturnValue("Version 0"),
     }),
 });