feat(expect-webdriverio): add toHaveLocalStorageItem matcher

[NaamuKim]

What I do

Added tests with the toHaveLocalStorageItem function

Why this function is needed

before

const tokenExists = await browser.execute(() => {
    return localStorage.getItem('authToken') !== null
})
expect(tokenExists).toBe(true)

const tokenValue = await browser.execute(() => {
    return localStorage.getItem('authToken')
})
expect(tokenValue).toMatch(/^[A-Za-z0-9-_]+\./)

after

await expect(browser).toHaveLocalStorageItem('authToken')
await expect(browser).toHaveLocalStorageItem('authToken', /^[A-Za-z0-9-_]+\./)

If you need anything else, let me know and I'll get it implemented quickly.
If this API is okay, I'll also add a PR for Session storage.

[christian-bromann]

Thanks for the contribution and sorry for the delayed review. One request, can we ipdate the API docs: https://github.com/webdriverio/expect-webdriverio/blob/main/docs/API.md

[NaamuKim]

Thanks for your review!
I updated docs in this commit17b882d

[christian-bromann]

@NaamuKim mhm 🤔 seems like some tests are failing:

Error: src/matchers/browser/toHaveLocalStorageItem.ts(14,9): error TS2322: Type '"toHaveLocalStorageItem"' is not assignable to type 'keyof Matchers<void, unknown>'.
Error: src/matchers/browser/toHaveLocalStorageItem.ts(48,9): error TS2322: Type '"toHaveLocalStorageItem"' is not assignable to type 'keyof Matchers<void, unknown>'.

[NaamuKim]

@christian-bromann I added types to matchers, can you rerun the tests?

[christian-bromann]

Great stuff, thanks a lot 👍

[christian-bromann]

@NaamuKim it seems like there is still a valid test error:

 FAIL  test/index.test.ts > index
AssertionError: expected 42 to deeply equal 41

- Expected
+ Received

- 41
+ 42

 ❯ test/index.test.ts:8:27
      6|     expect(expectExport).toBeDefined()
      7|     expect(utils.compareText).toBeDefined()
      8|     expect(matchers.size).toEqual(41)
       |                           ^
      9| })
     10| 

[dprevost-LMI]

@NaamuKim, do you want a hand with this one to finish it up?

[christian-bromann]

@dprevost-LMI I suggest let's get this over the goal line

[NaamuKim]

Sorry, I've been busy for a while. I've solved the test and fixed the crash, so please check it out. There's a lot of updated content. It looks good.

[dprevost-LMI]

Nice matcher.
FYI: Following support of $$(), I'll adapt some code personally later on!
Thanks for the contribution!