Skip to content
This repository has been archived by the owner on Jun 7, 2022. It is now read-only.

Commit

Permalink
Fix typescript type error in admin-e2e-tests package (#7765)
Browse files Browse the repository at this point in the history
* Fix type error

* Update changelog

* Better type handlings
  • Loading branch information
ilyasfoo committed Oct 23, 2021
1 parent ae3b49f commit 5c8cd29
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
2 changes: 2 additions & 0 deletions packages/admin-e2e-tests/CHANGELOG.md
Expand Up @@ -3,6 +3,8 @@

- Make unchecking free features more robust. #7761

- Fix typescript type error in admin-e2e-tests package #7765

# 0.1.2

- Add Customers to analytics pages tested #7573
Expand Down
8 changes: 5 additions & 3 deletions packages/admin-e2e-tests/src/pages/BasePage.ts
Expand Up @@ -142,9 +142,11 @@ export abstract class BasePage {
timeout: 10000,
} );
} catch ( e ) {
throw new Error(
`Could not navigate to url: ${ fullUrl } with error: ${ e.message }`
);
if ( e instanceof Error ) {
throw new Error(
`Could not navigate to url: ${ fullUrl } with error: ${ e.message }`
);
}
}
}
}

0 comments on commit 5c8cd29

Please sign in to comment.