refactor: set seed directly (#563)

4 files changed, 35 insertions, 20 deletions

diff --git a/src/datatype.ts b/src/datatype.ts
index 5dc1793c..aa7139dd 100644
--- a/src/datatype.ts
+++ b/src/datatype.ts
@@ -4,14 +4,7 @@ import type { Faker } from '.';
  * Module to generate various primitive values and data types.
  */
 export class Datatype {
-  constructor(private readonly faker: Faker, seed?: number | number[]) {
-    // Use a user provided seed if it is an array or number
-    if (Array.isArray(seed) && seed.length) {
-      this.faker.mersenne.seed_array(seed);
-    } else if (!Array.isArray(seed) && !isNaN(seed)) {
-      this.faker.mersenne.seed(seed);
-    }
-
+  constructor(private readonly faker: Faker) {
     // Bind `this` so namespaced is working correctly
     for (const name of Object.getOwnPropertyNames(Datatype.prototype)) {
       if (name === 'constructor' || typeof this[name] !== 'function') {
diff --git a/src/faker.ts b/src/faker.ts
index 83f59a06..572bbaf2 100644
--- a/src/faker.ts
+++ b/src/faker.ts
@@ -129,10 +129,13 @@ export class Faker {
     });
   }
 
-  seed(value?: number | number[]): void {
-    this.seedValue = value;
-    this.random = new Random(this, this.seedValue);
-    this.datatype = new Datatype(this, this.seedValue);
+  seed(seed?: number | number[]): void {
+    this.seedValue = seed;
+    if (Array.isArray(seed) && seed.length) {
+      this.mersenne.seed_array(seed);
+    } else if (!Array.isArray(seed) && !isNaN(seed)) {
+      this.mersenne.seed(seed);
+    }
   }
 
   /**
diff --git a/src/random.ts b/src/random.ts
index dd58c3ce..b84a2ece 100644
--- a/src/random.ts
+++ b/src/random.ts
@@ -18,14 +18,7 @@ function arrayRemove<T>(arr: T[], values: T[]): T[] {
  * Generates random values of different kinds. Some methods are deprecated and have been moved to dedicated modules.
  */
 export class Random {
-  constructor(private readonly faker: Faker, seed?: number | number[]) {
-    // Use a user provided seed if it is an array or number
-    if (Array.isArray(seed) && seed.length) {
-      this.faker.mersenne.seed_array(seed);
-    } else if (!Array.isArray(seed) && !isNaN(seed)) {
-      this.faker.mersenne.seed(seed);
-    }
-
+  constructor(private readonly faker: Faker) {
     // Bind `this` so namespaced is working correctly
     for (const name of Object.getOwnPropertyNames(Random.prototype)) {
       if (name === 'constructor' || typeof this[name] !== 'function') {
diff --git a/test/faker.spec.ts b/test/faker.spec.ts
new file mode 100644
index 00000000..54fa04ac
--- /dev/null
+++ b/test/faker.spec.ts
@@ -0,0 +1,26 @@
+import { beforeEach, describe, expect, it } from 'vitest';
+import { faker } from '../src';
+
+describe('faker', () => {
+  beforeEach(() => {
+    faker.locale = 'en';
+  });
+
+  // This is only here for coverage
+  // The actual test is in mersenne.spec.ts
+  describe('seed()', () => {
+    it('seed(number)', () => {
+      faker.seed(1);
+
+      const actual = faker.animal.cat();
+      expect(actual).toBe('Korat');
+    });
+
+    it('seed(number[])', () => {
+      faker.seed([1, 2, 3]);
+
+      const actual = faker.animal.cat();
+      expect(actual).toBe('Oriental');
+    });
+  });
+});