refactor: 628 - remove unused env variables (#629)

rolling-scopes · Nov 4, 2024 · 1f70018 · 1f70018
1 parent dfa0b38
commit 1f70018
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 32 deletions.
diff --git a/.env b/.env
diff --git a/src/shared/constants.ts b/src/shared/constants.ts
@@ -1,6 +1,3 @@
-export const TABLET_W = Number(process.env.NEXT_PUBLIC_TABLET);
-export const MOBILE_W = Number(process.env.NEXT_PUBLIC_MOBILE);
-export const DESKTOP_W = 1280;
 export const RS_INTRO_URL = 'https://www.youtube.com/embed/n4unZLVpnaU';
 export const RS_FOUNDATION_YEAR = '2013';
 

diff --git a/src/shared/helpers/generateSizes.ts b/src/shared/helpers/generateSizes.ts
diff --git a/src/shared/helpers/generateSrcSet.ts b/src/shared/helpers/generateSrcSet.ts
diff --git a/src/widgets/about-video/ui/about-video.test.tsx b/src/widgets/about-video/ui/about-video.test.tsx
@@ -22,15 +22,15 @@ describe('AboutVideo component', () => {
   });
 
   it('renders the placeholder in development mode', () => {
-    vi.stubEnv('NEXT_PUBLIC_DEV', 'true');
+    vi.stubEnv('NODE_ENV', 'development');
     render(<AboutVideo />);
     const placeholder = screen.getByText('Video Placeholder');
 
     expect(placeholder).toBeVisible();
   });
 
   it('renders the YouTube embed in production mode', () => {
-    vi.stubEnv('NEXT_PUBLIC_DEV', 'false');
+    vi.stubEnv('NODE_ENV', 'production');
     render(<AboutVideo />);
     const video = screen.getByTitle('Introduction to The Rolling Scopes School Online Courses');
 

diff --git a/src/widgets/about-video/ui/about-video.tsx b/src/widgets/about-video/ui/about-video.tsx
@@ -11,7 +11,7 @@ type AboutVideoProps = { lang?: 'en' | 'ru' };
 
 export const AboutVideo = ({ lang = 'en' }: AboutVideoProps) => {
   // Needed to prevent flakiness in screenshot tests
-  const isRunningInDev = process.env.NEXT_PUBLIC_DEV === 'true';
+  const isRunningInDev = process.env.NODE_ENV === 'development';
 
   return (
     <section className={cx('container')} data-testid="about-video">