history icon indicating copy to clipboard operation
history copied to clipboard

Test improvement: push and replace of Partial<Path>

Open thejohnhoffer opened this issue 3 years ago • 0 comments

Expected behavior

Tests should ensure history.push and history.replace work with to: Partial<Path>.

Actual behavior

All test sequences only test with to: string.

Details

This seems to be an oversight with the tests. Both history.push and history.replace have a first argument of type To. The type definition for To allows two options: Partial<Path> and string.

However, it seems that all test sequences call push and replace with strings, never with objects that contain keys of Path. I've created a PR that solves this issue.

thejohnhoffer avatar Dec 21 '21 15:12 thejohnhoffer