chore: suggest gherkin syntax for test description

[NicolasMassart]

Description

This PR is a suggestion to replace bullet points list by a Gherkin syntax code block for the PR template test section.

Pros

• This really looks like a Cucumber scenario
• Dev and QA would then speak the same language
• It allows to refer to Gherkin syntax as a guideline and framework
• Easy, or at least not more complicated to write

Cons

• none as you can still write the bullet point if you want, it's just a template suggestion.
• I'm curious to see what you think about this. Let me know in the PR.

Examples

Examples of PRs where I already use this instead of the bullet list (but it's not in the template):

• fix: metrics call during onboarding #13226
• chore: move metrics identify to state listener #13203
• fix: update marketing traits when tracked settings change [PR 1/2] #13254

Related issues

n/a

Manual testing steps

n/a

Screenshots/Recordings

Before

This is how the Manual testing steps template looks now:

---

1. Go to this page...

---

After

This is how the Manual testing steps template would look if we use Gherkin syntax:

---

Feature: my feature name
  Scenario: user [verb for user action]
    Given [describe expected initial app state] 
    When user [verb for user action]
    Then [describe expected outcome]

---

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Copilot]

PR Overview

This PR updates the PR template to suggest using a Gherkin syntax code block for the manual testing steps section instead of the traditional bullet points.

• Replaces the bullet list with a Gherkin feature scenario example.
• Provides a clear example to align communication between dev and QA teams.

Reviewed Changes

File	Description
.github/pull-request-template.md	Updated the Manual testing steps section with Gherkin syntax

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.22%. Comparing base (b539406) to head (31b3ff8).
Report is 21 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #13227      +/-   ##
==========================================
+ Coverage   65.72%   66.22%   +0.49%     
==========================================
  Files        2228     2243      +15     
  Lines       47433    47934     +501     
  Branches     6682     6756      +74     
==========================================
+ Hits        31176    31742     +566     
+ Misses      14263    14176      -87     
- Partials     1994     2016      +22     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud