Merge branch 'main' into renovate/devdependencies

faker-js · Apr 4, 2022 · 9158389 · 9158389
2 parents a2a74d5 + fb1b87e
commit 9158389
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/src/random.ts b/src/random.ts
@@ -326,7 +326,7 @@ export class Random {
       const randomWordMethod = this.faker.random.arrayElement(wordMethods);
 
       result = randomWordMethod();
-    } while (bannedChars.some((char) => result.includes(char)));
+    } while (!result || bannedChars.some((char) => result.includes(char)));
 
     return this.faker.random.arrayElement(result.split(' '));
   }

diff --git a/test/system.spec.ts b/test/system.spec.ts
@@ -268,4 +268,19 @@ describe('system', () => {
       });
     }
   });
+
+  describe('extra tests', () => {
+    describe('commonFileName()', () => {
+      afterEach(() => {
+        faker.locale = 'en';
+      });
+
+      it('#770', () => {
+        faker.seed(5423027051750305);
+        faker.setLocale('sk');
+        faker.system.commonFileName('xml');
+        faker.system.commonFileName('xml');
+      });
+    });
+  });
 });