feat: rename packages/test-node to packages/test-js

Author: dmythro

bun.lock

@@ -33,8 +33,8 @@
         "tsconfig": "workspace:*",
       },
     },
-    "packages/test-node": {
-      "name": "test-node",
+    "packages/test-js": {
+      "name": "test-js",
       "devDependencies": {
         "@types/bun": "catalog:default",
         "tsconfig": "workspace:*",
@@ -365,7 +365,7 @@
 
     "sucrase": ["sucrase@3.35.0", "", { "dependencies": { "@jridgewell/gen-mapping": "^0.3.2", "commander": "^4.0.0", "glob": "^10.3.10", "lines-and-columns": "^1.1.6", "mz": "^2.7.0", "pirates": "^4.0.1", "ts-interface-checker": "^0.1.9" }, "bin": { "sucrase": "bin/sucrase", "sucrase-node": "bin/sucrase-node" } }, "sha512-8EbVDiu9iN/nESwxeSxDKe0dunta1GOlHufmSSXxMD2z2/tMZpDMpvXQGsc+ajGo8y2uYUmixaSRUc/QPoQ0GA=="],
 
-    "test-node": ["test-node@workspace:packages/test-node"],
+    "test-js": ["test-js@workspace:packages/test-js"],
 
     "thenify": ["thenify@3.3.1", "", { "dependencies": { "any-promise": "^1.0.0" } }, "sha512-RVZSIV5IG10Hk3enotrhvz0T9em6cyHBLkH/YAZuKqd8hRkKhSfCGIcP2KUY0EPxndzANBmNllzWPwak+bheSw=="],
 

packages/test-node/getCountryCode.test.ts renamed to packages/test-js/getCountryCode.test.ts

@@ -0,0 +1,71 @@
+// biome-ignore-all lint/performance/noDynamicNamespaceImportAccess: test file
+
+import { describe, expect, test } from 'bun:test'
+import fs from 'node:fs'
+import * as source from 'src/index.ts'
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
+// @ts-expect-error
+import * as mjs from '../../dist/mjs/index.js'
+
+const exportDataList: Partial<keyof typeof source>[] = ['continents', 'countries', 'languages']
+const exportFnList: Partial<keyof typeof source>[] = ['getEmojiFlag']
+
+function evalIIFE(name = 'Countries') {
+  const script = fs.readFileSync('../../dist/index.iife.js', { encoding: 'utf-8' })
+  // biome-ignore lint/security/noGlobalEval: -
+  eval(`this.${name} = (function () { ${script}\nreturn ${name}})()`)
+}
+
+describe('dist', () => {
+  test('has proper CJS ES6 export', () => {
+    expect(typeof mjs).toBe('object')
+
+    for (const prop of exportFnList) {
+      expect(Object.hasOwn(mjs, prop)).toBe(true)
+    }
+
+    for (const prop of exportDataList) {
+      expect(Object.hasOwn(mjs, prop)).toBe(true)
+
+      const cjsKeys = Object.keys(mjs[prop])
+      const srcKeys = Object.keys(source[prop])
+      expect(cjsKeys).toEqual(srcKeys)
+    }
+  })
+
+  test('all English country names should contain only A-Z characters', () => {
+    const nameReg = /^[a-z\s.()-]+$/i
+    const nonAZ: string[] = []
+    for (const c of Object.values(source.countries)) {
+      if (!nameReg.test(c.name)) {
+        nonAZ.push(c.name)
+      }
+    }
+
+    // It helps see incorrect names right away in logs
+    expect(nonAZ).toEqual([])
+  })
+
+  test('loads ES6 <script> properly', () => {
+    const context: { Countries?: typeof source } = {}
+    evalIIFE.call(context)
+
+    const contextCountries = context.Countries
+
+    expect(contextCountries).toBeTruthy()
+    expect(contextCountries).toBeObject()
+
+    for (const prop of exportFnList) {
+      expect(contextCountries).toHaveProperty(prop)
+    }
+
+    for (const prop of exportDataList) {
+      expect(contextCountries).toHaveProperty(prop)
+
+      // biome-ignore lint/style/noNonNullAssertion: -
+      const windowProps = Object.keys(contextCountries![prop]) as string[]
+      const dataProps = Object.keys(source[prop]) as string[]
+      expect(windowProps).toEqual(dataProps)
+    }
+  })
+})

packages/test-node/getCountryData.test.ts renamed to packages/test-js/getCountryData.test.ts

@@ -1,5 +1,5 @@
 {
-  "name": "test-node",
+  "name": "test-js",
   "type": "module",
   "devDependencies": {
     "@types/bun": "catalog:default",