Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prepare v3.2.2 release #404

Merged
merged 1 commit into from
Apr 3, 2017
Merged

Prepare v3.2.2 release #404

merged 1 commit into from
Apr 3, 2017

Conversation

philipjohn
Copy link
Contributor

Full changeset for this release: 69f58e2...bd13386

Full changeset for this release: 69f58e2...bd13386
@mjangda
Copy link
Member

mjangda commented Mar 15, 2017

Main issue I spotted is #367 (comment) -- other than that, I think we're looking good.

@mjangda
Copy link
Member

mjangda commented Mar 15, 2017

Actually, need to look at the test failures too:

There were 2 failures:
1) Test_Author_Queried_Object::test__author_queried_object_fix
is_archive, is_author is expected to be true. is_404 is expected to be false.
Failed asserting that false is true.
/tmp/wordpress-tests-lib/includes/testcase.php:609
/home/travis/build/Automattic/Co-Authors-Plus/tests/test-author-queried-object.php:73

2) Test_Manage_CoAuthors::test_add_coauthor_to_post
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
     0 => 9
-    1 => 10
 )

@philipjohn
Copy link
Contributor Author

#406 addresses the various other instances of get_the_terms()

  1. Test_Author_Queried_Object::test__author_queried_object_fix

Looks like this broke with [6dbcd3c], see 425.9

  1. Test_Manage_CoAuthors::test_add_coauthor_to_post

This one broke with [52da0aa], see 433.9

The latter is fixed by [d5305b5].

@mjangda
Copy link
Member

mjangda commented Mar 20, 2017

Test_Author_Queried_Object::test__author_queried_object_fix
Looks like this broke with [6dbcd3c], see 425.9

@sboisvert @trepmal can you please take a look? ^^

@trepmal
Copy link
Contributor

trepmal commented Mar 23, 2017

The temporary tables are due to the way WordPress gets installed on the fly for testing.

https://core.trac.wordpress.org/browser/trunk/tests/phpunit/includes/testcase.php#L312 - summarized:

add_filter( 'query', function( $query ) {
	if ( 'CREATE TABLE' === substr( trim( $query ), 0, 12 ) ) {
		return substr_replace( trim( $query ), 'CREATE TEMPORARY TABLE', 0, 12 );
	}
	return $query;
} );

Interestingly (or not), we're only encountering this because we test in multisite. The initial tables that are installed are not temporary, but the subsite tables are (since those pass through the query filter).

If I simply comment out those filters and rerun the tests, they pass.

#398 (comment)

@trepmal
Copy link
Contributor

trepmal commented Mar 23, 2017

I kinda missed the intro in my reply... but #398 provides the context

@trepmal trepmal mentioned this pull request Mar 23, 2017
@philipjohn philipjohn merged commit 82444c3 into master Apr 3, 2017
@philipjohn philipjohn deleted the release/3.2.2 branch April 3, 2017 15:17
rebeccahum pushed a commit that referenced this pull request Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants