fix(next-drupal): update TypeScript definition for translatePath and …

…translatePathFromContext Fixes #359
chapter-three · Feb 14, 2024 · 9f6ae58 · 9f6ae58
1 parent ac704e3
commit 9f6ae58
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/packages/next-drupal/src/client.ts b/packages/next-drupal/src/client.ts
@@ -894,7 +894,7 @@ export class DrupalClient {
  async translatePath(
  path: string,
  options?: JsonApiWithAuthOptions
- ): Promise<DrupalTranslatedPath> {
+ ): Promise<DrupalTranslatedPath | null> {
  options = {
  withAuth: this.withAuth,
  ...options,
@@ -927,7 +927,7 @@ export class DrupalClient {
  options?: {
  pathPrefix?: PathPrefix
  } & JsonApiWithAuthOptions
- ): Promise<DrupalTranslatedPath> {
+ ): Promise<DrupalTranslatedPath | null> {
  options = {
  pathPrefix: "/",
  ...options,

diff --git a/packages/next-drupal/tests/client.test.ts b/packages/next-drupal/tests/client.test.ts
@@ -1445,13 +1445,15 @@ describe("getResourceFromContext", () => {
  },
  }
 
- const recipe = await client.getResourceFromContext(path, context, {
- params: {
- "fields[node--recipe]": "title,path,status",
- },
- })
+ if (path) {
+ const recipe = await client.getResourceFromContext(path, context, {
+ params: {
+ "fields[node--recipe]": "title,path,status",
+ },
+ })
 
- await expect(recipe).toMatchSnapshot()
+ await expect(recipe).toMatchSnapshot()
+ }
  })
 })