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.

Sign up for GitHub

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

Jump to bottom

Feat/test meta remarks #4732

Open
wants to merge 6 commits into
base: 3.x
Choose a base branch
from
Open

Feat/test meta remarks #4732

wants to merge 6 commits into from

Conversation

DavertMik
Copy link
Contributor

@DavertMik DavertMik commented Jan 10, 2025 •
edited
Loading

  • Added metadata to tests
Scenario('my test linked to Jira').meta('ISSUE', 'TST-123')

Scenario('my test linked to Jira', meta: {issue: 'TST-123'})
  • Setting browser, browserVersion, and windowSize in Playwright as metadata
  • Added notes to tests to store internal info
  • Using AI heal fixes as notes
  • Improved heal plugin to print info in workers mode

image

  • AI heal add heal to metadata
  • Improved output for Playwright stack traces
  • Enhanced ouptut for steps
  • Added metadata for final output

Applicable helpers:

  • Playwright
  • Puppeteer
  • WebDriver
  • REST
  • FileHelper
  • Appium
  • TestCafe

Applicable plugins:

  • heal

Type of change

  • 🔥 Breaking changes
  • 🚀 New functionality
  • 🐛 Bug fix
  • 🧹 Chore
  • 📋 Documentation changes/updates
  • ♨️ Hot fix
  • 🔨 Markdown files fix - not related to source code
  • 💅 Polish code

Checklist:

  • Tests have been added
  • Documentation has been added (Run npm run docs)
  • Lint checking (Run npm run lint)
  • Local tests are passed (Run npm test)

@DavertMik DavertMik requested a review from kobenguyent January 10, 2025 03:47
@DavertMik DavertMik mentioned this pull request Jan 10, 2025
Merged
32 tasks
kobenguyent
kobenguyent reviewed Jan 11, 2025
View reviewed changes
lib/command/workers/runTests.js

// run tests
(async function () {
;(async function () {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess this code format adds this unnecessarily

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kobenguyent kobenguyent kobenguyent approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DavertMik @kobenguyent