WordPress.org
Get WordPress

Make WordPress Core

Opened 5 months ago

Last modified 5 months ago

#65173 new enhancement

Add unit tests for saveDomDocument()

Reported by: pbearne Owned by:
Priority: normal Milestone: Awaiting Review
Component: Administration Version:
Severity: normal Keywords: has-patch has-unit-tests
Cc: Focuses: tests

Description

This ticket adds unit tests for the saveDomDocument() function in wp-admin/includes/misc.php. These tests ensure that the function correctly saves a DOMDocument to a file and properly converts line endings to CRLF.

Change History (2)

This ticket was mentioned in ​PR #11721 on ​WordPress/wordpress-develop by ​@pbearne.
5 months ago #1

  • Keywords has-patch has-unit-tests added

Description:
This PR adds unit tests for the saveDomDocument() function in wp-admin/includes/misc.php. These tests ensure that the function correctly saves a DOMDocument to a file and properly converts line endings to CRLF, which is required for IIS configuration files.

The tests cover:

  • Successful saving of a DOMDocument to a specified path.
  • Verification that line endings are converted to CRLF.

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

AI Disclosure:

  • AI assistance: Yes
  • Tool(s): Junie (JetBrains)
  • Model(s): gemini-3-flash-preview
  • Used for: Code analysis, test implementation, and workflow management.

#2 @desrosj
5 months ago

  • Focuses tests added

Adding the tests focus, which is used to indicate a ticket is solely focused on adding tests.

Note: See TracTickets for help on using tickets.