Closed Philip-21 closed 8 months ago
Awesome work @Philip-21. Please rebase your branch on master so that you don't re-add the files already present in master.
On a side note, adding a slightly more descriptive commit message and description is always helpful for fellow developers :) https://cbea.ms/git-commit/
On a side note, adding a slightly more descriptive commit message and description is always helpful for fellow developers :) https://cbea.ms/git-commit/
Sure, the next commit will have it.
@VaibhavMalik4187 i created new functions in the playwrightSetup file to be called in the playwright test file There are several commands implemented in cypres which cant be done in playwright. looking at this file I can implement login, Setting Viewport, making api request and intercepting requests to be used for testcases.
@VaibhavMalik4187 i created new functions in the playwrightSetup file to be called in the playwright test file There are several commands implemented in cypres which cant be done in playwright. looking at this file I can implement login, Setting Viewport, making api request and intercepting requests to be used for testcases.
@Philip-21 you're making great progress. If you could just implement the helper functions used in beforeEachCallbackForCustomUrl
, then I'll quickly raise a PR to replace cypress with playwright in meshmap-snapshot action to run the loadDesign e2e test.
Ultimately, this request includes the implementation playwright version of the following cypress functions:
I'll need to spend some more time with playwright before I become able to convert the following functions.
Feel free to ping me in case you need help with anything.
@VaibhavMalik4187 i created new functions in the playwrightSetup file to be called in the playwright test file There are several commands implemented in cypres which cant be done in playwright. looking at this file I can implement login, Setting Viewport, making api request and intercepting requests to be used for testcases.
@Philip-21 you're making great progress. If you could just implement the helper functions used in
beforeEachCallbackForCustomUrl
, then I'll quickly raise a PR to replace cypress with playwright in meshmap-snapshot action to run the loadDesign e2e test.Ultimately, this request includes the implementation playwright version of the following cypress functions:
- intercept
- setViewPort
- login
I'll need to spend some more time with playwright before I become able to convert the following functions.
- setReleaseTag
- interceptCapabilities
- setMode
Feel free to ping me in case you need help with anything.
Alright @VaibhavMalik4187 thanks , i'll be intouch
@VaibhavMalik4187 i just implemented some new functions to be used for beforeEachCallbackForCustomUrl
I have been able to write and run some individual tests in the playWrightUtils.js these are just tests in accessing the meshery page and they ran successfully .
The main thing is to configure meshmap snapshots to run successfully, I am figuring out how i will configure and put things together to make it a success
I looked through the cypress-action files to see how that was achieved, I observed cypress has alot of inbuilt functionalities which playwright doesn't.
So, I imitated some cypress commands in the commands.js and helpers.js and I defined some of these comands in the playwrightsetup.JS
The constant.js which is similar to the cypress action folder , is for getting the meshmap designs and its api endpoints.
these are just the progress i have made so far
The approach used to run the cypress action, I feel it was a different approach as spec.js
file wasn't used to run it.
The function in the loadDesign.js runs the e2e tests as we can see in the action workflow. These are just my observations so far
I defined some setup functions that would used to run the e2e test as the way it was implemented for the cypress-action.
But I feel using the spec.js
will be better for running e2e tests for playwright, as i was able to run individual tests successfully in the spec
file . I am also figuring out how to call and perform the meshMap snapshot actions with configurations in the playwrightsetup.js and the design endpoint in the
constant.js which will be called and ran in the spec file
@MarioArriaga92 thanks i've fixed them
This is some inspiring work, @Philip-21 😄
This is some inspiring work, @Philip-21 😄
lol thanks
@Philip-21 any idea why all the PR Check failures?
@Philip-21 also, about the "tests_e2e" job, 2 questions/observations:
See screenshot for reference:
This is quite interesting, @Philip-21. Cool.
Hello @Philip-21 The workflow is pull_request_target
, it only runs on master branch, so its obvious to fail here. Your local changes are running good?
You can merge your branch into your local fork and see the running tests/debug from there.
Does it work?
All Playwright tests work perfectly, but the workflow is still detecting failed cypress actions. I am liaising with Abhi on this issue
On Wed, Oct 18, 2023 at 6:26 PM Lee Calcote @.***> wrote:
Does it work?
— Reply to this email directly, view it on GitHub https://github.com/layer5labs/meshmap-snapshot/pull/51#issuecomment-1769009191, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARP2QJWIUJRXCGB7GCFDABLYAAGLTAVCNFSM6AAAAAA4SNZOSGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRZGAYDSMJZGE . You are receiving this because you modified the open/close state.Message ID: @.***>
Notes for Reviewers
This PR fixes #