synpress icon indicating copy to clipboard operation
synpress copied to clipboard

Docs: Incorrect Documentation

Open kr-keshav-kk opened this issue 1 year ago โ€ข 5 comments

๐Ÿ”Ž Have you searched existing issues to avoid duplicates?

  • [X] I have made sure that my issue is not a duplicate.

๐Ÿงช Have you tested your code using latest version of Synpress?

๐Ÿ’ก Are you able to provide enough information to be able to reproduce your issue locally?

  • [X] I can provide enough details to reproduce my issue on local environment.

Synpress version

4.0.3

Node.js version

22.11.0

Operating system

15.0

Run mode

Playwright + Synpress (as plugin)

CI platform (if applicable)

No response

Are you running your tests inside docker? (if applicable)

  • [ ] This issue could be related to docker.

What happened?

Docs are outdated https://docs.synpress.io/docs/setup-playwright

import { MetaMask } from "@synthetixio/synpress"; -> Metamask is not define in synthetixio/synpress

import { testWithSynpress, MetaMask, unlockForFixture, } from "@synthetixio/synpress"; -> unlockForFixture is also not defined in synthetixio/synpress

What is your expected behavior?

Please update the docs.

How to reproduce the bug.

Just look at the docs and the synpress example docs

Relevant log output

No response

kr-keshav-kk avatar Nov 18 '24 10:11 kr-keshav-kk

import { MetaMask, unlockForFixture } from "@synthetixio/synpress/playwright"

SssSxxS avatar Nov 21 '24 09:11 SssSxxS

thanks @SssSxxS . But i think the docs should be updated with these new changes. This old docs is misleading.

kr-keshav-kk avatar Nov 21 '24 10:11 kr-keshav-kk

Getting started in general seems to be bonked. I can't seem to be able to get through the playwright getting started.

Have others had success?

PatrickAlphaC avatar Feb 06 '25 03:02 PatrickAlphaC

@PatrickAlphaC hey, were you able to get through??

kr-keshav-kk avatar Apr 22 '25 14:04 kr-keshav-kk

I did @kr-keshav-kk

https://github.com/Cyfrin/ts-tsender-ui-cu/tree/main/test

It's very different from the docs, but I didn't make a docs PR because there were hints that the whole thing was going to be revamped.

PatrickAlphaC avatar Apr 22 '25 20:04 PatrickAlphaC