feat: set app version as dev when building locally

gitify-app · Jun 14, 2024 · f6c1f60 · f6c1f60
1 parent ba0a493
commit f6c1f60
Show file tree

Hide file tree

Showing 5 changed files with 101 additions and 7 deletions.
diff --git a/src/routes/Settings.test.tsx b/src/routes/Settings.test.tsx
@@ -31,6 +31,11 @@ describe('routes/Settings.tsx', () => {
     Object.defineProperty(process, 'platform', {
       value: originalPlatform,
     });
+
+    // Restore the original node env value
+    Object.defineProperty(process, 'platform', {
+      value: originalPlatform,
+    });
   });
 
   describe('General', () => {
@@ -481,6 +486,68 @@ describe('routes/Settings.tsx', () => {
   });
 
   describe('Footer section', () => {
+    describe('app version', () => {
+      let originalEnv: NodeJS.ProcessEnv;
+
+      beforeEach(() => {
+        // Save the original node env state
+        originalEnv = process.env;
+      });
+
+      afterEach(() => {
+        // Restore the original node env state
+        process.env = originalEnv;
+      });
+
+      it('should show production app version', async () => {
+        process.env = {
+          ...originalEnv,
+          NODE_ENV: 'production',
+        };
+
+        await act(async () => {
+          render(
+            <AppContext.Provider
+              value={{
+                auth: mockAuth,
+                settings: mockSettings,
+              }}
+            >
+              <MemoryRouter>
+                <SettingsRoute />
+              </MemoryRouter>
+            </AppContext.Provider>,
+          );
+        });
+
+        expect(screen.getByTitle('app-version')).toMatchSnapshot();
+      });
+
+      it('should show development app version', async () => {
+        process.env = {
+          ...originalEnv,
+          NODE_ENV: 'development',
+        };
+
+        await act(async () => {
+          render(
+            <AppContext.Provider
+              value={{
+                auth: mockAuth,
+                settings: mockSettings,
+              }}
+            >
+              <MemoryRouter>
+                <SettingsRoute />
+              </MemoryRouter>
+            </AppContext.Provider>,
+          );
+        });
+
+        expect(screen.getByTitle('app-version')).toMatchSnapshot();
+      });
+    });
+
     it('should open release notes', async () => {
       const openExternalLinkMock = jest.spyOn(comms, 'openExternalLink');
 

diff --git a/src/routes/Settings.tsx b/src/routes/Settings.tsx
@@ -38,8 +38,13 @@ export const SettingsRoute: FC = () => {
 
   useEffect(() => {
     (async () => {
-      const result = await getAppVersion();
-      setAppVersion(result);
+      console.log('process.env.NODE_ENV', process.env.NODE_ENV);
+      if (process.env.NODE_ENV === 'development') {
+        setAppVersion('dev');
+      } else {
+        const result = await getAppVersion();
+        setAppVersion(`v${result}`);
+      }
     })();
 
     ipcRenderer.on('gitify:update-theme', (_, updatedTheme: Theme) => {
@@ -283,7 +288,7 @@ export const SettingsRoute: FC = () => {
           title="View release notes"
           onClick={() => openGitifyReleaseNotes(appVersion)}
         >
-          Gitify v{appVersion}
+          <span title="app-version">Gitify {appVersion}</span>
         </button>
         <div>
           <button

diff --git a/src/routes/__snapshots__/Settings.test.tsx.snap b/src/routes/__snapshots__/Settings.test.tsx.snap
diff --git a/src/utils/links.test.ts b/src/utils/links.test.ts
@@ -37,7 +37,7 @@ describe('utils/links.ts', () => {
   });
 
   it('openGitifyReleaseNotes', () => {
-    openGitifyReleaseNotes('1.0.0');
+    openGitifyReleaseNotes('v1.0.0');
     expect(comms.openExternalLink).toHaveBeenCalledWith(
       'https://github.com/gitify-app/gitify/releases/tag/v1.0.0',
     );

diff --git a/src/utils/links.ts b/src/utils/links.ts
@@ -11,7 +11,7 @@ export function openGitifyRepository() {
 
 export function openGitifyReleaseNotes(version: string) {
   openExternalLink(
-    `https://github.com/${Constants.REPO_SLUG}/releases/tag/v${version}` as Link,
+    `https://github.com/${Constants.REPO_SLUG}/releases/tag/${version}` as Link,
   );
 }