refactor

Author: shellmayr

static/app/utils/gettingStartedDocs/python.spec.tsx

@@ -16,52 +16,48 @@ describe('getPythonInstallSnippet', () => {
       packageName: 'sentry-sdk',
     });
 
-    expect(result).toBe(`pip install "sentry-sdk"`);
+    expect(result.pip).toBe(`pip install "sentry-sdk"`);
   });
 
   it('generates pip install command with minimum version and extras', () => {
     const result = getPythonInstallSnippet({
       packageName: 'sentry-sdk[with-extras]',
       minimumVersion: '1.2.3',
     });
-    expect(result).toBe(`pip install --upgrade "sentry-sdk[with-extras]>=1.2.3"`);
+    expect(result.pip).toBe(`pip install --upgrade "sentry-sdk[with-extras]>=1.2.3"`);
   });
 
   it('generates uv install command with default parameters', () => {
     const result = getPythonInstallSnippet({
       packageName: 'sentry-sdk',
-      packageManager: 'uv',
     });
 
-    expect(result).toBe(`uv add "sentry-sdk"`);
+    expect(result.uv).toBe(`uv add "sentry-sdk"`);
   });
 
   it('generates uv install command with minimum version and extras', () => {
     const result = getPythonInstallSnippet({
       packageName: 'sentry-sdk[with-extras]',
-      packageManager: 'uv',
       minimumVersion: '2.3.4',
     });
 
-    expect(result).toBe(`uv add --upgrade "sentry-sdk[with-extras]>=2.3.4"`);
+    expect(result.uv).toBe(`uv add --upgrade "sentry-sdk[with-extras]>=2.3.4"`);
   });
 
   it('generates poetry install command with default parameters', () => {
     const result = getPythonInstallSnippet({
       packageName: 'sentry-sdk',
-      packageManager: 'poetry',
     });
 
-    expect(result).toBe(`poetry add "sentry-sdk"`);
+    expect(result.poetry).toBe(`poetry add "sentry-sdk"`);
   });
 
   it('generates poetry install command with minimum version and extras', () => {
     const result = getPythonInstallSnippet({
       packageName: 'sentry-sdk[with-extras]',
-      packageManager: 'poetry',
       minimumVersion: '2.3.4',
     });
 
-    expect(result).toBe(`poetry add "sentry-sdk[with-extras]>=2.3.4"`);
+    expect(result.poetry).toBe(`poetry add "sentry-sdk[with-extras]>=2.3.4"`);
   });
 });

static/app/utils/gettingStartedDocs/python.tsx

@@ -12,12 +12,10 @@ import {t, tct} from 'sentry/locale';
 
 export function getPythonInstallSnippet({
   packageName,
-  packageManager = 'pip',
   minimumVersion,
 }: {
   packageName: string;
   minimumVersion?: string;
-  packageManager?: 'pip' | 'uv' | 'poetry';
 }) {
   // We are using consistent double quotes here for all package managers after aligning with the Python SDK team.
   // Not using quotes may lead to some shells interpreting the square brackets, and using double quotes over single quotes is a convention.
@@ -33,7 +31,7 @@ export function getPythonInstallSnippet({
     poetry: `poetry add ${versionedPackage}`,
   };
 
-  return packageManagerCommands[packageManager].trim();
+  return packageManagerCommands;
 }
 
 export function getPythonInstallConfig({
@@ -45,6 +43,7 @@ export function getPythonInstallConfig({
   minimumVersion?: string;
   packageName?: string;
 } = {}): Configuration[] {
+  const packageManagerCommands = getPythonInstallSnippet({packageName, minimumVersion});
   return [
     {
       description,
@@ -54,31 +53,19 @@ export function getPythonInstallConfig({
           label: 'pip',
           value: 'pip',
           language: 'bash',
-          code: getPythonInstallSnippet({
-            packageName,
-            packageManager: 'pip',
-            minimumVersion,
-          }),
+          code: packageManagerCommands.pip,
         },
         {
           label: 'uv',
           value: 'uv',
           language: 'bash',
-          code: getPythonInstallSnippet({
-            packageName,
-            packageManager: 'uv',
-            minimumVersion,
-          }),
+          code: packageManagerCommands.uv,
         },
         {
           label: 'poetry',
           value: 'poetry',
           language: 'bash',
-          code: getPythonInstallSnippet({
-            packageName,
-            packageManager: 'poetry',
-            minimumVersion,
-          }),
+          code: packageManagerCommands.poetry,
         },
       ],
     },