test: remove usage of vitest.todo (#2960)

faker-js · Jun 23, 2024 · 6b1159d · 6b1159d
1 parent 4514fce
commit 6b1159d
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 23 deletions.
diff --git a/test/modules/helpers.spec.ts b/test/modules/helpers.spec.ts
@@ -1029,16 +1029,6 @@ describe('helpers', () => {
           );
         });
 
-        // TODO @ST-DDT 2023-01-17: Restore this test when the definitions proxy is restored: #893
-        it.todo('should support deprecated definition aliases', () => {
-          expect(faker.definitions.location.city_name).toContain(
-            faker.helpers.fake('{{address.city_name}}')
-          );
-          expect(faker.definitions.person.first_name).toContain(
-            faker.helpers.fake('{{name.first_name}}')
-          );
-        });
-
         it('should not trim whitespace', () => {
           expect(faker.helpers.fake('   ---   ')).toBe('   ---   ');
         });

diff --git a/test/support/seeded-runs.ts b/test/support/seeded-runs.ts
@@ -160,19 +160,6 @@ class TestGenerator<
     return this;
   }
 
-  /**
-   * Temporarily ignores this method. Useful for testing purposes.
-   *
-   * @param method The name of the method.
-   *
-   * @deprecated Implement a proper test.
-   */
-  todo(method: MethodOf<TModule>): this {
-    this.expectNotTested(method);
-    vi_it.todo(method);
-    return this;
-  }
-
   /**
    * Generates a test for a method without arguments.
    *