[wp-trac] [WordPress Trac] #17851: Wrapping Sections with add_settings_section

WordPress Trac noreply at wordpress.org
Thu Sep 1 20:09:48 UTC 2022


#17851: Wrapping Sections with add_settings_section
-------------------------------------------------+-------------------------
 Reporter:  griffinjt                            |       Owner:  chriscct7
     Type:  enhancement                          |      Status:  reviewing
 Priority:  normal                               |   Milestone:  6.1
Component:  Administration                       |     Version:  3.1.3
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch good-first-bug has-unit-   |     Focuses:
  tests changes-requested needs-dev-note needs-  |
  testing                                        |
-------------------------------------------------+-------------------------
Changes (by audrasjb):

 * keywords:  has-patch good-first-bug has-unit-tests =>
     has-patch good-first-bug has-unit-tests changes-requested needs-dev-
     note needs-testing


Comment:

 As per today's bug scrub, @costdev mentioned:

 > Multiple-assertion test methods in the PR will need a $message parameter
 added to each assertion ([https://make.wordpress.org/core/handbook/testing
 /automated-testing/writing-phpunit-tests/#using-assertions Ref]) -> "This
 parameter should always be used if more than one assertion is used in a
 test method."

 Let's update the provided unit tests.
 Self assigning for proper testing.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/17851#comment:28>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list