chore(deps): update dependency @types/jest to v30 - abandoned#1096
chore(deps): update dependency @types/jest to v30 - abandoned#1096renovate[bot] wants to merge 3 commits intomainfrom
Conversation
7004c8f to
f24b975
Compare
… to 30.0.0 - Fix deprecated Jest matcher aliases and error comparisons - All tests, build, and lint checks pass
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
| return expect( | ||
| deleteByRecordNumber(apiClient, "1", [{ value: "appcode-1" }]), | ||
| ).rejects.toThrow(error); | ||
| ).rejects.toThrow("Failed to delete records"); |
There was a problem hiding this comment.
@shabaraba
ここだけに限らずなのですが、toThrowにわたす引数の変更って必要ありますでしょうか?
※エラーのオブジェクト渡すのはv30でも変更なさそうにみえたので。
https://jestjs.io/docs/expect#tothrowerror
There was a problem hiding this comment.
具体的なソースが見当たらないのでアレなんですが、どうやら30に上がるまでに、toThrow(error)としてもオブジェクト比較ではなく、error.messageで比較をする挙動に変わったようでした。
なので既存実装だと以下のような形でテストが落ちます。
Expected message: "Failed to delete records"
Received message and cause: "{\"cause\":\" Not exists record number. ID: appcode-1\\n\",\"detail\":\"\",\"message\":\"Failed to delete records\",\"name\":\"ValidatorError\"}"
Autoclosing SkippedThis PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error. |
This PR contains the following updates:
^29.5.14->^30.0.0Configuration
📅 Schedule: Branch creation - "after 9pm,before 9am" in timezone Asia/Tokyo, Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.