Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#59555 closed defect (bug) (fixed)

Elements: Add additional unit tests for elements block support

Reported by: aaronrobertshaw Owned by: flixos90
Priority: normal Milestone: 6.4
Component: Editor Version:
Severity: normal Keywords: gutenberg-merge has-patch has-unit-tests
Cc: Focuses:

Description

This ticket tracks the backporting of PHP files for the following Gutenberg changes:

​https://github.com/WordPress/gutenberg/pull/55113

These changes add further unit tests for the element block support to better cover classname application and style generation.

Change History (5)

This ticket was mentioned in ​PR #5418 on ​WordPress/wordpress-develop by ​@aaronrobertshaw.


3 years ago
#1

  • Keywords has-patch has-unit-tests added

This is a backport PR that includes the following PHP Gutenberg changes:

​https://github.com/WordPress/gutenberg/pull/55113

It provides additional unit tests for the elements block support and its associated classname and style generation.

Trac ticket: https://core.trac.wordpress.org/ticket/59555

#2 @flixos90
3 years ago

  • Milestone Awaiting Review → 6.4
  • Owner set to flixos90
  • Status new → reviewing
  • Type enhancement → defect (bug)

Per the guidance that exclusively performance related fixes shouldn't be labelled as enhancements, I'm changing this to a bug as it supports #59544.

Also, this ticket specifically is a tests only change, so it won't have any impact on the actual 6.4 release anyway.

​@aaronrobertshaw commented on ​PR #5418:


3 years ago
#3

Appreciate all the feedback @costdev & @felixarntz 👍

I believe I've addressed everything now. Let me know if I missed anything

​@flixos90 commented on ​PR #5418:


3 years ago
#4

Committed in https://core.trac.wordpress.org/changeset/56806 - which for some reason doesn't show up yet, but proof it was committed can be found here: ​https://github.com/WordPress/wordpress-develop/commit/4d5859bececc8b451ca95108d942f2a870bdf806 😆

#5 @flixos90
3 years ago

  • Resolution → fixed
  • Status reviewing → closed

Fixed by [56806], which for some reason hasn't shown up here yet 🤔

Note: See TracTickets for help on using tickets.