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.

Sign up for GitHub

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

Jump to bottom

Posts: post-count tests in wpcom api test suite should be updated #208

Closed
hoverduck opened this issue Nov 19, 2015 · 0 comments
Closed

Posts: post-count tests in wpcom api test suite should be updated #208

hoverduck opened this issue Nov 19, 2015 · 0 comments
Assignees
@retrofox
Labels
API Posts [Pri] Low Address when resources are available. [Type] Task

Comments

@hoverduck
Copy link
Contributor

Although it is generally good practice to create your expected artifacts for a given test in a setup function, we don't really follow that practice in our API tests. With consistency and speed concerns, we should just manually create the posts we need for test purposes.
@hoverduck hoverduck added this to the Core: Iteration 17 milestone Nov 19, 2015
@retrofox retrofox self-assigned this Dec 1, 2015
@hoverduck hoverduck added the [Pri] Low Address when resources are available. label Dec 2, 2015
@rralian rralian modified the milestone: Core: Iteration 17 Feb 2, 2016
jsnajdr pushed a commit that referenced this issue Jan 27, 2020
@retrofox @jsnajdr
Merge pull request #208 from Automattic/update/update-site-media
895da98 
Add Site.Media.Edit() method
This was referenced Jun 1, 2020
Open
Open
@snyk-bot snyk-bot mentioned this issue Jul 18, 2020
Open
This was referenced Jan 7, 2021
Open
Open
Open
Open
Open
This was referenced May 5, 2021
Open
Open
This was referenced May 26, 2021
Open
Open
@mdno mdno mentioned this issue Nov 14, 2022
Open
@anarh anarh mentioned this issue Nov 15, 2022
Open
@mdno mdno mentioned this issue Nov 15, 2022
Open
@anarh anarh mentioned this issue Nov 16, 2022
Open
@mdno mdno mentioned this issue Nov 21, 2022
Open
@snyk-bot snyk-bot mentioned this issue Nov 22, 2022
Open
@mdno mdno mentioned this issue Nov 28, 2022
Open
@anarh anarh mentioned this issue Nov 29, 2022
Open
@mdno mdno mentioned this issue Jan 9, 2023
Open
@anarh anarh mentioned this issue Jan 10, 2023
Open
@anarh anarh mentioned this issue Nov 28, 2023
Open
@mdno mdno mentioned this issue Nov 28, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@retrofox retrofox

Labels
API Posts [Pri] Low Address when resources are available. [Type] Task
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@retrofox @rralian @apeatling @hoverduck