search

WordPress / theme-test-data

Theme Test Data
622 stars 16 forks source link

Update themeunittestdata.wordpress.xml #24

Closed carolinan closed 5 years ago

carolinan commented 5 years ago

Removes Chip, Emil and Lance as users, and assigns their items to the themedemos user.

joyously commented 5 years ago

It would be better to have multiple users, rather than everything belonging to that one.

christianwach commented 5 years ago

Agree with @joyously - this test will be next-to-useless without multiple users. Can't they just be anonymised?

carolinan commented 5 years ago

I agree. Personally I'm not sure how to best arrange it.

Just trying to sort out the existing comments and checking what still works and what email adresses we can keep is daunting.

The unit test still lives on the test blog on wordpress.com and the authors need to have access to that blog. I do know of one more account that we can use as a second author, that the Theme Review Team team leads has access to.

Currently there is also no template that shows more than one user.

joyously commented 5 years ago

Why do they have to be real email addresses? I was just referring to the fact that it's nice to have multiple authors (with different choices of display name so that that is tested), for showing in the author archives, and in the author bio. Comments should have a range of people also, to test the avatars and site links and whether the commenter is the author of the post. I think a bunch of this might need to be manually tweaked after import, though. If so, the various page tests should mention it.

carolinan commented 5 years ago

Have you read the open issue?