WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#25118 closed defect (bug) (fixed)

Unit Tests: Restrict locales in test_locale_floats in db.php

Reported by: nofearinc Owned by: SergeyBiryukov
Priority: normal Milestone: 3.7
Component: Unit Tests Version: 3.6
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

While running phpunit tests/db.php from the wordpress-tests project I got the following error:

1) Tests_DB::test_locale_floats
    constant(): Couldn't find constant LC_PAPER

Apparently I don't have a constant defined on my Linux (Fedora) box called LC_PAPER. I didn't see that as a possible argument in the ​setlocale function either.

I would suggest that we verify if a category is an existing constant when we try to set it as a locale.

Attachments (1)

25118.patch​ (549 bytes ) - added by nofearinc 13 years ago.

Download all attachments as: .zip

Change History (3)

@nofearinc
13 years ago

#1 @SergeyBiryukov
13 years ago

  • Milestone Awaiting Review → 3.7
  • Version trunk → 3.6

Related: #19861, [746/tests], [1271/tests].

#2 @SergeyBiryukov
13 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 25084:

Check if a locale category is an existing constant when restoring locale settings in Tests_DB::test_locale_floats(). props nofearinc. fixes #25118.

Note: See TracTickets for help on using tickets.