From 05808613a1a184513387108d8bfd1250d74de4b0 Mon Sep 17 00:00:00 2001
From: s1gr1d <sigrid.huemer@posteo.at>
Date: Fri, 28 Jun 2024 15:44:17 +0200
Subject: [PATCH 1/2] test(nuxt): Add code snippet unit tests

---
 packages/nuxt/src/common/snippets.ts       | 10 +--
 packages/nuxt/test/common/snippets.test.ts | 75 ++++++++++++++++++++++
 2 files changed, 81 insertions(+), 4 deletions(-)
 create mode 100644 packages/nuxt/test/common/snippets.test.ts

diff --git a/packages/nuxt/src/common/snippets.ts b/packages/nuxt/src/common/snippets.ts
index 9086172387a7..158ff44935e3 100644
--- a/packages/nuxt/src/common/snippets.ts
+++ b/packages/nuxt/src/common/snippets.ts
@@ -1,16 +1,18 @@
 import * as fs from 'fs';
 import * as path from 'path';
 
-// todo: tests
 /** Returns an import snippet */
 export function buildSdkInitFileImportSnippet(filePath: string): string {
   const posixPath = filePath.split(path.sep).join(path.posix.sep);
 
-  return `import "${posixPath}";`;
+  // normalize to forward slashed for Windows-based systems
+  const normalizedPath = posixPath.replace(/\\/g, '/');
+
+  return `import '${normalizedPath}';`;
 }
 
-// todo: tests
-/** Adds an import statement right after <script setup> */
+/** Adds a top-level import statement right after <script setup>.
+ * This should happen as early as possible (e.g. in root component) */
 export function addImportStatement(filePath: string, importStatement: string): void {
   try {
     const data = fs.readFileSync(filePath, 'utf8');
diff --git a/packages/nuxt/test/common/snippets.test.ts b/packages/nuxt/test/common/snippets.test.ts
new file mode 100644
index 000000000000..19f97f91b91b
--- /dev/null
+++ b/packages/nuxt/test/common/snippets.test.ts
@@ -0,0 +1,75 @@
+import * as fs from 'fs';
+
+import * as path from 'path';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+import { addImportStatement, buildSdkInitFileImportSnippet } from '../../src/common/snippets';
+
+describe('Nuxt Code Snippets', () => {
+  describe('buildSdkInitFileImportSnippet', () => {
+    it('returns correct import statement for POSIX path', () => {
+      const filePath = 'src/myFile.ts';
+      const expectedOutput = "import 'src/myFile.ts';";
+      const result = buildSdkInitFileImportSnippet(filePath);
+      expect(result).toBe(expectedOutput);
+    });
+
+    it('returns correct import statement for Windows path', () => {
+      const filePath = 'src\\myFile.ts';
+      const expectedOutput = "import 'src/myFile.ts';";
+      const result = buildSdkInitFileImportSnippet(filePath);
+      expect(result).toBe(expectedOutput);
+    });
+
+    it('returns correct import statement for path with multiple segments', () => {
+      const filePath = path.join('src', 'myDir', 'myFile.ts');
+      const expectedOutput = "import 'src/myDir/myFile.ts';";
+      const result = buildSdkInitFileImportSnippet(filePath);
+      expect(result).toBe(expectedOutput);
+    });
+  });
+
+  describe('addImportStatement', () => {
+    vi.mock('fs');
+
+    beforeEach(() => {
+      vi.clearAllMocks();
+    });
+
+    it('should add import statement to file', () => {
+      const readFileSyncSpy = vi.spyOn(fs, 'readFileSync');
+      const writeFileSyncSpy = vi.spyOn(fs, 'writeFileSync');
+
+      const filePath = 'testFile.ts';
+      const importStatement = 'import { test } from "./test";';
+      const fileContent =
+        '<template>\n' +
+        '  <Suspense @resolve="onResolve">\n' +
+        '  </Suspense>\n' +
+        '</template>\n' +
+        '\n' +
+        '<script setup>\n' +
+        "import { provide } from 'vue'\n" +
+        "import { useNuxtApp } from '../nuxt'";
+
+      const fileContentExpected =
+        '<template>\n' +
+        '  <Suspense @resolve="onResolve">\n' +
+        '  </Suspense>\n' +
+        '</template>\n' +
+        '\n' +
+        '<script setup>\n' +
+        `${importStatement}\n` +
+        "import { provide } from 'vue'\n" +
+        "import { useNuxtApp } from '../nuxt'";
+
+      readFileSyncSpy.mockReturnValue(fileContent);
+      writeFileSyncSpy.mockImplementation(() => {});
+
+      addImportStatement(filePath, importStatement);
+
+      expect(fs.readFileSync).toHaveBeenCalledWith(filePath, 'utf8');
+      expect(fs.writeFileSync).toHaveBeenCalledWith(filePath, fileContentExpected, 'utf8');
+    });
+  });
+});

From f70035e90e672abf928cd0d685310e2f0d526113 Mon Sep 17 00:00:00 2001
From: Sigrid Huemer <32902192+s1gr1d@users.noreply.github.com>
Date: Mon, 1 Jul 2024 08:57:41 +0200
Subject: [PATCH 2/2] Update packages/nuxt/src/common/snippets.ts

Co-authored-by: Abhijeet Prasad <aprasad@sentry.io>
---
 packages/nuxt/src/common/snippets.ts | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/packages/nuxt/src/common/snippets.ts b/packages/nuxt/src/common/snippets.ts
index 158ff44935e3..fb84f21417e6 100644
--- a/packages/nuxt/src/common/snippets.ts
+++ b/packages/nuxt/src/common/snippets.ts
@@ -11,8 +11,10 @@ export function buildSdkInitFileImportSnippet(filePath: string): string {
   return `import '${normalizedPath}';`;
 }
 
-/** Adds a top-level import statement right after <script setup>.
- * This should happen as early as possible (e.g. in root component) */
+/** 
+ * Adds a top-level import statement right after <script setup>.
+ * This should happen as early as possible (e.g. in root component)
+ */
 export function addImportStatement(filePath: string, importStatement: string): void {
   try {
     const data = fs.readFileSync(filePath, 'utf8');