Merge branch 'next' into chore/more-states-mk

Author: ST-DDT

package.json

@@ -88,16 +88,16 @@
   },
   "devDependencies": {
     "@actions/github": "~5.1.1",
-    "@algolia/client-search": "~4.18.0",
+    "@algolia/client-search": "~4.19.1",
     "@types/markdown-it": "~12.2.3",
     "@types/node": "~20.4.1",
     "@types/prettier": "~2.7.3",
     "@types/react": "~18.2.15",
     "@types/sanitize-html": "~2.9.0",
     "@types/semver": "~7.5.0",
     "@types/validator": "~13.7.17",
-    "@typescript-eslint/eslint-plugin": "~5.62.0",
-    "@typescript-eslint/parser": "~5.62.0",
+    "@typescript-eslint/eslint-plugin": "~6.1.0",
+    "@typescript-eslint/parser": "~6.1.0",
     "@vitest/coverage-v8": "~0.33.0",
     "@vitest/ui": "~0.33.0",
     "@vueuse/core": "~10.2.1",
@@ -128,7 +128,7 @@
     "typescript": "~4.9.5",
     "validator": "~13.9.0",
     "vite": "~4.4.2",
-    "vitepress": "1.0.0-beta.5",
+    "vitepress": "1.0.0-beta.6",
     "vitest": "~0.33.0",
     "vue": "~3.3.4"
   },

pnpm-lock.yaml

@@ -1,65 +1,32 @@
 export default [
-  'An Giang',
-  'Bà Rịa-Vũng Tàu',
-  'Bạc Liêu',
-  'Bắc Kạn',
-  'Bắc Giang',
+  'An Nhơn',
+  'Biên Hòa',
+  'Buôn Ma Thuột',
   'Bắc Ninh',
-  'Bến Tre',
-  'Bình Dương',
-  'Bình Định',
-  'Bình Phước',
-  'Bình Thuận',
   'Cà Mau',
-  'Cao Bằng',
   'Cần Thơ',
-  'Đà Nẵng',
-  'Đắk Lắk',
-  'Đắk Nông',
-  'Điện Biên',
-  'Đồng Nai',
-  'Đồng Tháp',
-  'Gia Lai',
-  'Hà Giang',
-  'Hà Nam',
-  'Hà Nội',
-  'Hà Tĩnh',
+  'Haiphong',
+  'Hanoi',
+  'Ho Chi Minh City',
+  'Huế',
   'Hải Dương',
-  'Hải Phòng',
-  'Hậu Giang',
-  'Hòa Bình',
-  'TP. Hồ Chí Minh',
-  'Hưng Yên',
-  'Khánh Hoà',
-  'Kiên Giang',
-  'Kon Tum',
-  'Lai Châu',
-  'Lạng Sơn',
-  'Lào Cai',
-  'Lâm Đồng',
-  'Long An',
+  'Long Xuyên',
+  'Mỹ Tho',
   'Nam Định',
-  'Nghệ An',
-  'Ninh Bình',
-  'Ninh Thuận',
-  'Phú Thọ',
-  'Phú Yên',
-  'Quảng Bình',
-  'Quảng Nam',
-  'Quảng Ngãi',
-  'Quảng Ninh',
-  'Quảng Trị',
+  'Nghi Sơn',
+  'Nha Trang',
+  'Phan Thiết',
+  'Quy Nhơn',
+  'Quảng Hà',
   'Sóc Trăng',
-  'Sơn La',
-  'Tây Ninh',
+  'Thanh Hóa',
   'Thái Bình',
   'Thái Nguyên',
-  'Thanh Hoá',
-  'Thừa Thiên-Huế',
-  'Tiền Giang',
-  'Trà Vinh',
-  'Tuyên Quang',
-  'Vĩnh Long',
-  'Vĩnh Phúc',
-  'Yên Bái',
+  'Thủ Dầu Một',
+  'Thủ Đức',
+  'Tân An',
+  'Tân Uyên',
+  'Vinh',
+  'Việt Trì',
+  'Vũng Tàu',
 ];

src/locales/vi/location/city_name.ts

@@ -8,6 +8,7 @@ import city_pattern from './city_pattern';
 import country from './country';
 import default_country from './default_country';
 import postcode from './postcode';
+import state from './state';
 import street_pattern from './street_pattern';
 
 const location: LocationDefinition = {
@@ -16,6 +17,7 @@ const location: LocationDefinition = {
   country,
   default_country,
   postcode,
+  state,
   street_pattern,
 };
 

src/locales/vi/location/index.ts

@@ -0,0 +1,66 @@
+export default [
+  'An Giang',
+  'Bà Rịa-Vũng Tàu',
+  'Bắc Giang',
+  'Bắc Kạn',
+  'Bạc Liêu',
+  'Bắc Ninh',
+  'Bến Tre',
+  'Bình Định',
+  'Bình Dương',
+  'Bình Phước',
+  'Bình Thuận',
+  'Cà Mau',
+  'Cần Thơ',
+  'Cao Bằng',
+  'Đà Nẵng',
+  'Đắk Lắk',
+  'Đắk Nông',
+  'Điện Biên',
+  'Đồng Nai',
+  'Đồng Tháp',
+  'Gia Lai',
+  'Hà Giang',
+  'Hà Nam',
+  'Hà Tây',
+  'Hà Tĩnh',
+  'Hải Dương',
+  'Hải Phòng',
+  'Hà Nội',
+  'Hậu Giang',
+  'Hồ Chí Minh',
+  'Hòa Bình',
+  'Hưng Yên',
+  'Khánh Hòa',
+  'Kiên Giang',
+  'Kon Tum',
+  'Lai Châu',
+  'Lâm Đồng',
+  'Lạng Sơn',
+  'Lào Cai',
+  'Long An',
+  'Nam Định',
+  'Nghệ An',
+  'Ninh Bình',
+  'Ninh Thuận',
+  'Phú Thọ',
+  'Phú Yên',
+  'Quảng Bình',
+  'Quảng Nam',
+  'Quảng Ngãi',
+  'Quảng Ninh',
+  'Quảng Trị',
+  'Sóc Trăng',
+  'Sơn La',
+  'Tây Ninh',
+  'Thái Bình',
+  'Thái Nguyên',
+  'Thanh Hóa',
+  'Thừa Thiên-Huế',
+  'Tiền Giang',
+  'Trà Vinh',
+  'Tuyên Quang',
+  'Vĩnh Long',
+  'Vĩnh Phúc',
+  'Yên Bái',
+];

src/locales/vi/location/state.ts

@@ -649,6 +649,11 @@ export class HelpersModule {
    * and outputs a unique array of strings based on that source.
    * This method does not store the unique state between invocations.
    *
+   * If there are not enough unique values to satisfy the length, if
+   * the source is an array, it will only return as many items as are
+   * in the array. If the source is a function, it will return after
+   * a maximum number of attempts has been reached.
+   *
    * @template T The type of the elements.
    *
    * @param source The strings to choose from or a function that generates a string.
@@ -671,8 +676,11 @@ export class HelpersModule {
     const set = new Set<T>();
     try {
       if (typeof source === 'function') {
-        while (set.size < length) {
+        const maxAttempts = 1000 * length;
+        let attempts = 0;
+        while (set.size < length && attempts < maxAttempts) {
           set.add(source());
+          attempts++;
         }
       }
     } catch {

src/modules/helpers/index.ts

@@ -806,6 +806,14 @@ describe('helpers', () => {
           expect(unique).toHaveLength(input.length);
         });
 
+        it('function with length longer than possible values returns', () => {
+          const fn = () => faker.helpers.arrayElement(['a', 'b']);
+          const length = 3;
+          const unique = faker.helpers.uniqueArray(fn, length);
+          expect(unique).not.toContainDuplicates();
+          expect(unique).toHaveLength(2);
+        });
+
         it('works as expected when seeded', () => {
           const input = ['a', 'a', 'a', 'a', 'a', 'f', 'g', 'h', 'i', 'j'];
           const length = 5;

test/helpers.spec.ts

@@ -23,8 +23,6 @@ interface CustomMatchers {
 }
 
 declare module 'vitest' {
-  // eslint-disable-next-line @typescript-eslint/no-empty-interface
   interface Assertion extends CustomMatchers {}
-  // eslint-disable-next-line @typescript-eslint/no-empty-interface
   interface AsymmetricMatchersContaining extends CustomMatchers {}
 }