Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLI never completes execution when there is an error in replay #132

Open
ergunsh opened this issue May 13, 2022 · 6 comments
Open

CLI never completes execution when there is an error in replay #132

ergunsh opened this issue May 13, 2022 · 6 comments
Assignees

Comments

@ergunsh
Copy link
Contributor

ergunsh commented May 13, 2022

Expected Behavior

CLI should report the error and complete the execution

Actual Behavior

Execution hangs and you need to press Ctrl + C to cancel execution

Steps to Reproduce the Problem

  1. Save the attached replay
  2. Run npx @puppeteer/replay <saved-recording-path>

You can make the json pass by removing --wrong from the selector in the click step.

Specifications

  • Version: @puppeteer/replay 0.1.2
  • Platform: MacOS

Attachment

{
  "title": "hn-app-refresh",
  "steps": [
    {
      "type": "setViewport",
      "width": 1280,
      "height": 1361,
      "deviceScaleFactor": 1,
      "isMobile": false,
      "hasTouch": false,
      "isLandscape": false
    },
    {
      "type": "navigate",
      "url": "https://hn-vanilla-pwa.ergunsh.vercel.app/",
      "assertedEvents": [
        {
          "type": "navigation",
          "url": "https://hn-vanilla-pwa.ergunsh.vercel.app/",
          "title": "HN App"
        }
      ]
    },
    {
      "type": "click",
      "selectors": [
        [
          "#postListContainer--wrong > div > div:nth-child(1) > h4"
        ]
      ],
      "target": "main",
      "offsetX": 148,
      "offsetY": 10.5625
    }
  ]
}
@OrKoN
Copy link
Collaborator

OrKoN commented May 13, 2022

It looks similar to #81 the browser close is in afterAllSteps which is not called on error. Would you like to take this issue?

@ergunsh
Copy link
Contributor Author

ergunsh commented May 13, 2022

Sure! I'm creating bugs for my rotation 😬 😂

@GENERALTHE
Copy link

🤗

@KrishCrypto

This comment was marked as off-topic.

@Aluk97

This comment was marked as off-topic.

@mylifetochoose
Copy link

It's not true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants
@OrKoN @ergunsh @GENERALTHE @Aluk97 @KrishCrypto @mylifetochoose and others