Skip to content
Snippets Groups Projects

Reduce verbose component creation in tests

Merged Miguel Rincon requested to merge mrincon--runner-reduce-verbose-component-creation into master
All threads resolved!

What does this MR do and why?

Closes #350658 (closed)

This change reduces repetition in tests by using a simpler way to create extended wrappers for tests.

It also reduces the unnecessary use of localVue.

Screenshots or screen recordings

NA

How to set up and validate locally

Validated by the pipeline.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Miguel Rincon

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Justin Ho Tuan Duong approved this merge request

    approved this merge request

  • Justin Ho Tuan Duong requested review from @f_caplette and removed review request for @justin_ho

    requested review from @f_caplette and removed review request for @justin_ho

  • :wave: @justin_ho, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Frédéric Caplette resolved all threads

    resolved all threads

  • Frédéric Caplette approved this merge request

    approved this merge request

  • Frédéric Caplette enabled an automatic merge when the pipeline for e34ee3d1 succeeds

    enabled an automatic merge when the pipeline for e34ee3d1 succeeds

  • mentioned in commit 158bebbe

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Miguel Rincon changed the description

    changed the description

  • mentioned in issue #350658 (closed)

  • Please register or sign in to reply
    Loading