fix(plugin): retryTo issue

.github/workflows/plugin.yml

@@ -0,0 +1,44 @@
+name: Playwright Tests with Plugins
+
+on:
+  push:
+    branches:
+      - 3.x
+  pull_request:
+    branches:
+      - '**'
+
+env:
+  CI: true
+  # Force terminal colors. @see https://www.npmjs.com/package/colors
+  FORCE_COLOR: 1
+
+jobs:
+  build:
+
+    runs-on: ubuntu-22.04
+
+    strategy:
+      matrix:
+        node-version: [20.x]
+
+    steps:
+    - uses: actions/checkout@v4
+    - name: Use Node.js ${{ matrix.node-version }}
+      uses: actions/setup-node@v4
+      with:
+        node-version: ${{ matrix.node-version }}
+    - uses: shivammathur/setup-php@v2
+      with:
+        php-version: 7.4
+    - name: npm install
+      run: |
+        npm install --legacy-peer-deps
+      env:
+        PUPPETEER_SKIP_CHROMIUM_DOWNLOAD: true
+    - name: Install browsers and deps
+      run: npx playwright install && npx playwright install-deps
+    - name: start a server
+      run: "php -S 127.0.0.1:8000 -t test/data/app &"
+    - name: run plugin tests
+      run: "./bin/codecept.js run -c test/acceptance/codecept.Playwright.retryTo.js --grep @plugin --verbose"

lib/plugin/retryTo.js

@@ -89,9 +89,9 @@ module.exports = function (config) {
     let err = null;
 
     return new Promise((done) => {
-      const tryBlock = () => {
+      const tryBlock = async () => {
         recorder.session.start(`retryTo ${tries}`);
-        callback(tries);
+        await callback(tries);
         recorder.add(() => {
           recorder.session.restore(`retryTo ${tries}`);
           done(null);

test/acceptance/codecept.Playwright.js

@@ -32,6 +32,9 @@ module.exports.config = {
     screenshotOnFail: {
       enabled: true,
     },
+    retryTo: {
+      enabled: true,
+    },
   },
   name: 'acceptance',
   gherkin: {

test/acceptance/codecept.Playwright.retryTo.js

@@ -0,0 +1,44 @@
+const TestHelper = require('../support/TestHelper');
+
+module.exports.config = {
+  tests: './*_test.js',
+  timeout: 10000,
+  output: './output',
+  grep: '@Playwright',
+  helpers: {
+    Playwright: {
+      url: TestHelper.siteUrl(),
+      show: false,
+      restart: process.env.BROWSER_RESTART || false,
+      browser: process.env.BROWSER || 'chromium',
+      ignoreHTTPSErrors: true,
+      webkit: {
+        ignoreHTTPSErrors: true,
+      },
+    },
+    JSONResponse: {
+      requestHelper: 'Playwright',
+    },
+    ScreenshotSessionHelper: {
+      require: '../support/ScreenshotSessionHelper.js',
+      outputPath: 'test/acceptance/output',
+    },
+    Expect: {},
+  },
+  include: {},
+  bootstrap: false,
+  mocha: {},
+  plugins: {
+    screenshotOnFail: {
+      enabled: true,
+    },
+    retryTo: {
+      enabled: true,
+    },
+  },
+  name: 'acceptance',
+  gherkin: {
+    features: './gherkin/*.feature',
+    steps: ['./gherkin/steps.js'],
+  },
+};

test/acceptance/retryTo_test.js

@@ -0,0 +1,10 @@
+const { I } = inject();
+
+Feature('Plugins');
+
+Scenario('retryTo works with await steps @plugin', async ({ I }) => {
+  await retryTo(async (tryNum) => {
+    const foo = await I.grabCurrentUrl();
+    if (tryNum < 3) I.waitForVisible('.nothing', 1);
+  }, 4);
+});