Context Navigation

← Previous Ticket
Next Ticket →

#35952 closed enhancement (fixed)

Unit test: Tests covering is_serialized_string method

Reported by:	borgesbruno	Owned by:	boonebgorges
Milestone:	4.6	Priority:	normal
Severity:	normal	Version:
Component:	General	Keywords:	has-unit-tests has-patch
Focuses:		Cc:

Description

I've created tests for the method is_serialized_string on /wp-includes/functions.php file, to improve de code coverage.

Attachments (2)

functions.diff (869 bytes) - added by borgesbruno 9 years ago.: 35952.patch
functions.2.diff (928 bytes) - added by borgesbruno 9 years ago.

Download all attachments as: .zip

Change History (11)

@borgesbruno
9 years ago

Attachment functions.diff added

35952.patch

#1 @ericlewis
9 years ago

Thanks for the patch and welcome to WordPress' bug tracker @borgesbruno!

If you could, please submit the patch against the root of the WordPress directory instead of in a subdirectory. Here's some more tips for writing patches.

@borgesbruno
9 years ago

Attachment functions.2.diff added

#2 @borgesbruno
9 years ago

Tks for the tip @ericlewis, I did that on the last file, sorry for the name of files, I was not able to remove or rename the attachments.
The functions2.diff is the right patch, the next patch I will do better, this is the first one.

Last edited 9 years ago by borgesbruno (previous) (diff)