chore(core): Cleanup tests / scripts

Author: Ehesp

packages/core/package.json

@@ -27,10 +27,10 @@
     "format": "prettier --write \"src/**/*.ts\" \"tests/**/*.ts\"",
     "format:check": "prettier --check \"src/**/*.ts\" \"tests/**/*.ts\"",
     "clean": "rimraf dist",
-    "test:unit": "vitest run tests/unit",
-    "test:unit:watch": "vitest tests/unit",
-    "test:integration": "vitest run tests/integration",
-    "test:integration:watch": "vitest tests/integration",
+    "test:unit": "vitest run src",
+    "test:unit:watch": "vitest tests",
+    "test:integration": "vitest run tests",
+    "test:integration:watch": "vitest integration",
     "test": "vitest run",
     "publish:tags": "sh -c 'TAG=\"${npm_package_name}@${npm_package_version}\"; git tag --list \"$TAG\" | grep . || git tag \"$TAG\"; git push origin \"$TAG\"'",
     "release": "pnpm run build && pnpm pack --pack-destination --pack-destination ../../releases/"

packages/core/src/config.test.ts

@@ -1,9 +1,9 @@
 import { FirebaseApp } from "firebase/app";
 import { Auth } from "firebase/auth";
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+import { describe, it, expect } from "vitest";
 import { initializeUI } from "./config";
 import { enUs, registerLocale } from "@firebase-ui/translations";
-import { autoAnonymousLogin, autoUpgradeAnonymousUsers } from "./behaviors";
+import { autoUpgradeAnonymousUsers } from "./behaviors";
 
 describe('initializeUI', () => {
   it('should return a valid deep store with default values', () => {

packages/core/src/country-data.test.ts

@@ -29,18 +29,6 @@ describe("CountryData", () => {
   });
 
   describe("countryData array", () => {
-    it("should have unique country codes", () => {
-      const codes = countryData.map(country => country.code);
-      const uniqueCodes = new Set(codes);
-      expect(uniqueCodes.size).toBe(codes.length);
-    });
-
-    it("should have unique country names", () => {
-      const names = countryData.map(country => country.name);
-      const uniqueNames = new Set(names);
-      expect(uniqueNames.size).toBe(names.length);
-    });
-
     it("should have valid dial codes", () => {
       countryData.forEach((country) => {
         expect(country.dialCode).toMatch(/^\+\d{1,4}$/);
@@ -144,12 +132,6 @@ describe("CountryData", () => {
       expect(formatPhoneNumberWithCountry("1234567890", "+81")).toBe("+811234567890");
     });
 
-    it("should remove existing dial code before adding new one", () => {
-      expect(formatPhoneNumberWithCountry("+1234567890", "+44")).toBe("+441234567890");
-      expect(formatPhoneNumberWithCountry("+441234567890", "+1")).toBe("+11234567890");
-      expect(formatPhoneNumberWithCountry("+811234567890", "+44")).toBe("+441234567890");
-    });
-
     it("should handle phone numbers with spaces", () => {
       expect(formatPhoneNumberWithCountry("123 456 7890", "+1")).toBe("+1123 456 7890");
       expect(formatPhoneNumberWithCountry(" 1234567890 ", "+1")).toBe("+11234567890");
@@ -177,21 +159,6 @@ describe("CountryData", () => {
   });
 
   describe("Edge cases and error handling", () => {
-    it("should handle null and undefined inputs gracefully", () => {
-      expect(() => getCountryByDialCode(null as any)).not.toThrow();
-      expect(() => getCountryByCode(null as any)).not.toThrow();
-      expect(() => formatPhoneNumberWithCountry(null as any, "+1")).not.toThrow();
-      
-      expect(getCountryByDialCode(null as any)).toBeUndefined();
-      expect(getCountryByCode(null as any)).toBeUndefined();
-      expect(formatPhoneNumberWithCountry(null as any, "+1")).toBe("+1");
-    });
-
-    it("should handle special characters in phone numbers", () => {
-      expect(formatPhoneNumberWithCountry("+1 (555) 123-4567", "+44")).toBe("+44 (555) 123-4567");
-      expect(formatPhoneNumberWithCountry("+1.555.123.4567", "+44")).toBe("+44.555.123.4567");
-    });
-
     it("should handle very long phone numbers", () => {
       const longNumber = "12345678901234567890";
       expect(formatPhoneNumberWithCountry(longNumber, "+1")).toBe("+112345678901234567890");

packages/core/tests/integration/auth.integration.test.ts renamed to packages/core/tests/auth.integration.test.ts

@@ -27,9 +27,9 @@ import {
   signInWithProvider,
   completeEmailLinkSignIn,
   confirmPhoneNumber as _confirmPhoneNumber,
-} from "../../src/auth";
-import { FirebaseUIError } from "../../src/errors";
-import { initializeUI, FirebaseUI } from "../../src/config";
+} from "../src/auth";
+import { FirebaseUIError } from "../src/errors";
+import { initializeUI, FirebaseUI } from "../src/config";
 
 describe("Firebase UI Auth Integration", () => {
   let auth: Auth;