chore: removal vestigal endpoint properties

Parkreiner · Parkreiner · commit a67fbcfd198a · 2024-06-03T13:49:44.000Z
diff --git a/plugins/backstage-plugin-coder/src/api/UrlSync.ts b/plugins/backstage-plugin-coder/src/api/UrlSync.ts
@@ -42,14 +42,10 @@ const PROXY_URL_KEY_FOR_DISCOVERY_API = 'proxy';
 
 type UrlPrefixes = Readonly<{
   proxyPrefix: string;
-  apiRoutePrefix: string;
-  assetsRoutePrefix: string;
 }>;
 
 export const defaultUrlPrefixes = {
   proxyPrefix: `/api/proxy`,
-  apiRoutePrefix: '', // Left as empty string because code assumes that CoderSdk will add /api/v2
-  assetsRoutePrefix: '', // Deliberately left as empty string
 } as const satisfies UrlPrefixes;
 
 export type UrlSyncSnapshot = Readonly<{
@@ -104,12 +100,10 @@ export class UrlSync implements UrlSyncApi {
   }
 
   private prepareNewSnapshot(newProxyUrl: string): UrlSyncSnapshot {
-    const { assetsRoutePrefix, apiRoutePrefix } = this.urlPrefixes;
-
     return {
       baseUrl: newProxyUrl.replace(proxyRouteReplacer, ''),
-      assetsRoute: `${newProxyUrl}${CODER_PROXY_PREFIX}${assetsRoutePrefix}`,
-      apiRoute: `${newProxyUrl}${CODER_PROXY_PREFIX}${apiRoutePrefix}`,
+      assetsRoute: `${newProxyUrl}${CODER_PROXY_PREFIX}`,
+      apiRoute: `${newProxyUrl}${CODER_PROXY_PREFIX}`,
     };
   }
 
diff --git a/plugins/backstage-plugin-coder/src/testHelpers/mockBackstageData.ts b/plugins/backstage-plugin-coder/src/testHelpers/mockBackstageData.ts
@@ -75,7 +75,7 @@ export const mockBackstageUrlRoot = 'http://localhost:7007';
  * the final result is.
  */
 export const mockBackstageApiEndpointWithoutSdkPath =
-  `${mockBackstageUrlRoot}${defaultUrlPrefixes.proxyPrefix}${CODER_PROXY_PREFIX}${defaultUrlPrefixes.apiRoutePrefix}` as const;
+  `${mockBackstageUrlRoot}${defaultUrlPrefixes.proxyPrefix}${CODER_PROXY_PREFIX}` as const;
 
 /**
  * The API endpoint to use with the mock server during testing. Adds additional
@@ -94,7 +94,7 @@ export const mockBackstageApiEndpoint =
  * the final result is.
  */
 export const mockBackstageAssetsEndpoint =
-  `${mockBackstageUrlRoot}${defaultUrlPrefixes.proxyPrefix}${CODER_PROXY_PREFIX}${defaultUrlPrefixes.assetsRoutePrefix}` as const;
+  `${mockBackstageUrlRoot}${defaultUrlPrefixes.proxyPrefix}${CODER_PROXY_PREFIX}` as const;
 
 export const mockBearerToken = 'This-is-an-opaque-value-by-design';
 export const mockCoderAuthToken = 'ZG0HRy2gGN-mXljc1s5FqtE8WUJ4sUc5X';
