Apply suggestions from code review

Co-authored-by: Caner Akdas <canerakdas@gmail.com>
Signed-off-by: Aviv Keller <me@aviv.sh>

Author: avivkeller

apps/site/components/Blog/BlogPostCard/__tests__/index.test.jsx

@@ -1,6 +1,8 @@
-import { render, screen } from '@testing-library/react';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
+
+import { render, screen } from '@testing-library/react';
+
 import { isVisible } from '../../../../../../tests/utilities.mjs';
 
 import BlogPostCard from '@/components/Blog/BlogPostCard';

apps/site/hooks/react-client/__tests__/useClientContext.test.jsx

@@ -1,11 +1,11 @@
 import { renderHook } from '@testing-library/react';
 
-import useClientContext from '@/hooks/react-client/useClientContext';
-import { MatterContext } from '@/providers/matterProvider';
-
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 
+import useClientContext from '@/hooks/react-client/useClientContext';
+import { MatterContext } from '@/providers/matterProvider';
+
 describe('useClientContext', () => {
   it('should return client context values', () => {
     const mockContextValue = {

apps/site/hooks/react-client/__tests__/useCopyToClipboard.test.jsx

@@ -1,8 +1,9 @@
-import { render, fireEvent, screen } from '@testing-library/react';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 import { setTimeout } from 'node:timers/promises';
 
+import { render, fireEvent, screen } from '@testing-library/react';
+
 import useCopyToClipboard from '@/hooks/react-client/useCopyToClipboard';
 
 navigator.clipboard = { writeText: () => {} };

apps/site/hooks/react-client/__tests__/useNotification.test.jsx

@@ -1,10 +1,11 @@
 import { render } from '@testing-library/react';
 
-import useNotification from '@/hooks/react-client/useNotification';
-import { NotificationProvider } from '@/providers/notificationProvider';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 
+import useNotification from '@/hooks/react-client/useNotification';
+import { NotificationProvider } from '@/providers/notificationProvider';
+
 describe('useNotification', () => {
   it('should return the notification dispatch function', () => {
     // Arrange

apps/site/providers/__tests__/navigationStateProvider.test.jsx

@@ -1,7 +1,8 @@
-import { render } from '@testing-library/react';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 
+import { render } from '@testing-library/react';
+
 import { NavigationStateProvider } from '@/providers/navigationStateProvider';
 
 describe('NavigationStateProvider', () => {

apps/site/providers/__tests__/notificationProvider.test.jsx

@@ -1,8 +1,9 @@
-import { render, act, waitFor } from '@testing-library/react';
-import userEvent from '@testing-library/user-event';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 
+import { render, act, waitFor } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+
 import {
   NotificationProvider,
   NotificationDispatch,

apps/site/providers/__tests__/themeProvider.test.jsx

@@ -1,7 +1,8 @@
-import { render } from '@testing-library/react';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 
+import { render } from '@testing-library/react';
+
 describe('ThemeProvider', () => {
   it('renders NextThemeProvider with the correct props', async t => {
     const mockFn = t.mock.fn(

apps/site/tests/setup.mjs

@@ -1,4 +1,5 @@
-import '../../../tests/setup.mjs';
 import { register } from 'node:module';
 
+import '../../../tests/setup.mjs';
+
 register('./loader.mjs', import.meta.url);

packages/ui-components/Common/BaseActiveLink/__tests__/index.test.jsx

@@ -1,7 +1,8 @@
-import { render, screen } from '@testing-library/react';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 
+import { render, screen } from '@testing-library/react';
+
 import ActiveLink from '..';
 
 describe('ActiveLink', () => {

packages/ui-components/Common/BasePagination/PaginationListItem/__tests__/index.test.jsx

@@ -1,8 +1,10 @@
-import { render, screen } from '@testing-library/react';
-import { isVisible } from '../../../../../../tests/utilities.mjs';
 import { describe, it } from 'node:test';
 import assert from 'node:assert/strict';
 
+import { render, screen } from '@testing-library/react';
+
+import { isVisible } from '../../../../../../tests/utilities.mjs';
+
 import PaginationListItem from '@node-core/ui-components/Common/BasePagination/PaginationListItem';
 
 function renderPaginationListItem({