fix: remove unused code. updated the deescription of unit test

Author: yash-builder

packages/sdks-tests/src/e2e-tests/hit-content-api.spec.ts

@@ -79,33 +79,4 @@ test.describe('Get Content', () => {
     expect(headers?.['x-builder-sdk-gen']).toBe(getSdkGeneration(sdk));
     expect(headers?.['x-builder-sdk-version']).toMatch(/\d+\.\d+\.\d+/); // Check for semver format
   });
-
-  test('should include componentsUsed by default when omit is empty string', async ({ page, sdk }) => {
-    test.skip(!excludeGen1(sdk));
-
-    const urlMatch = /https:\/\/cdn\.builder\.io\/api\/v3\/content/;
-    let requestUrl: string | undefined;
-
-    await page.route(urlMatch, async route => {
-      requestUrl = route.request().url();
-      return route.fulfill({
-        status: 200,
-        json: {
-          meta: {
-            componentsUsed: {
-              MyFunComponent: 1,
-            },
-          }
-        }
-      });
-    });
-
-    await page.goto('/get-content-with-omit', { waitUntil: 'networkidle' });
-
-    // Add null check before assertions
-    expect(requestUrl).toBeDefined();
-    expect(requestUrl!).not.toContain('omit=meta.componentsUsed');
-    expect(requestUrl!.includes('omit=')).toBeTruthy();
-    expect(new URL(requestUrl!).searchParams.get('omit')).toBe('');
-  });
 });

packages/sdks-tests/src/specs/index.ts

@@ -397,13 +397,6 @@ export const getProps = async (args: {
         apiEndpoint: 'content',
       };
       break;
-    case '/get-content-with-omit':
-      extraProps = {
-        apiEndpoint: 'content',
-        omit: '',
-        apiKey: REAL_API_KEY,
-      };
-      break;
     case '/get-query':
       extraProps = {
         options: { apiEndpoint: 'query', format: 'html', model: 'abcd', key: 'abcd' },

packages/sdks/e2e/react/src/App.tsx

@@ -54,18 +54,6 @@ function App() {
     }
   }, []);
 
-  useEffect(() => {
-    if (window.location.pathname === '/get-content-with-omit' && props) {
-      fetchOneEntry({
-        model: 'page',
-        userAttributes: {
-          urlPath: window.location.pathname,
-        },
-        ...props,
-      });
-    }
-  }, [props]);
-
   if (window.location.pathname === '/data-preview') {
     if (!props?.content) {
       return <div>Loading...</div>;

packages/sdks/src/functions/get-content/__snapshots__/generate-content-url.test.ts.snap

@@ -30,10 +30,10 @@ exports[`Generate Content URL > generate content url with omit, fields, offset,
 
 exports[`Generate Content URL > generates the proper value for a simple query 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&omit=meta.componentsUsed&query.id=%22c1b81bab59704599b997574eb0736def%22"`;
 
-exports[`Generate Content URL > handles undefined, null, and empty string as omit parameter values > omit-empty-string 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&omit="`;
+exports[`Generate Content URL > handles empty string as omit parameter values > omit-empty-string 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&omit="`;
 
-exports[`Generate Content URL > handles undefined, null, and empty string as omit parameter values > omit-null 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&omit=name"`;
+exports[`Generate Content URL > handles defined string as omit parameter values > omit-null 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&omit=name"`;
 
-exports[`Generate Content URL > handles undefined, null, and empty string as omit parameter values > omit-undefined 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&omit=meta.componentsUsed"`;
+exports[`Generate Content URL > handles undefined as omit parameter values > omit-undefined 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&omit=meta.componentsUsed"`;
 
 exports[`Generate Content URL > preserves both userAttributes.locale and top-level locale when both provided 1`] = `"https://cdn.builder.io/api/v3/content/page?apiKey=YJIGb4i01jvw0SRdL5Bt&limit=30&noTraverse=true&includeRefs=true&locale=en-US&omit=meta.componentsUsed&userAttributes=%7B%22locale%22%3A%22es-ES%22%2C%22foo%22%3A%22bar%22%7D"`;

packages/sdks/src/functions/get-content/generate-content-url.test.ts

@@ -35,7 +35,7 @@ describe('Generate Content URL', () => {
     expect(output).toMatchSnapshot();
   });
 
-  test('handles undefined, null, and empty string as omit parameter values', () => {
+  test('handles undefined, defined, and empty string as omit parameter values', () => {
     // Test with undefined (should default to 'meta.componentsUsed')
     const outputUndefined = generateContentUrl({
       apiKey: testKey,