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

Issue 123 - Unit tests #136

Merged
merged 24 commits into from
Jul 12, 2017
Merged

Issue 123 - Unit tests #136

merged 24 commits into from
Jul 12, 2017

Conversation

MattGeri
Copy link

This PR covers most of the cases in issue #123 except PULL operations that were not covered already.

  • Merges the test-coverage branch
  • Adds tests for RSS Pull Client
  • Refactors broken items (specifically around the logger)
  • Cleaning up syntax issues

@philipjohn Could you take a look when you get a chance.

sboisvert and others added 23 commits March 25, 2016 15:18
@sboisvert
Set correct cache class
76c9580 
By default simplepie will try to save to disk. In core this works by changing the cache class to a custom one that stores using transients. Fixes PHP Warnings when cache can't be found and also enables feeds to be cached.
@alexiskulash
Create readme.md
a00ecd4
@emrikol
Merge pull request Automattic#115 from alexiskulash/patch-1
30a003a 
Created a readme.md to summarize contribution needs for the plugin
@emrikol
Update readme.md
76ad91d 
Swapped out header to be compatible with Markdown
Add Travis configuration. Adapted from _s.
2917e94
Specify VIP standards rather than use a separate ruleset file
7be4077
@philipjohn
Merge pull request Automattic#122 from Automattic/phpcs-ci
f3d1280 
Setting up PHPCS checks with Travis
Tested tag to 4.7
98fdd13
Tests scaffold and travis rules
72950be
Correct plugin filename
5145d94
Add first Sitegroup test
4a9ce09
Test deleting sitegroups
b9dcc32
Test editing sitegroups
474d7f6
File header
624d63f
Use create_and_get() as it's more efficient
2c763dc
Initial stab at RSS Pull tests. Failing.
4e8e193
Clean up test data, and ensure all meta are in
86b4a08
@MattGeri
Merge remote-tracking branch 'origin/test-coverage' into feature/AUTO-3
9217c57 
# Conflicts:
#	.travis.yml
#	bin/install-wp-tests.sh
#	includes/class-syndication-wp-rss-client.php
#	tests/bootstrap.php
@MattGeri
Issue 123
a913fec 
* Fixes after merge with `test-coverage` branch
* Move tests from sitegroup to bootstrap
* Update the RSS pull client tests
@MattGeri
Issue 123
281e297 
* Adding tests for RSS Pull Client
* Refactoring broken items (specifically around the logger)
* Cleaning up syntax issues
@MattGeri
Issue 123
90b18ef 
* Implement tests for XML Push Client using a custom inteceptor
* Fix date issue in the push client
* Set tests to run as multisite
@MattGeri
Issue 123
a69191b 
* Comment for delete test method
@MattGeri
Issue 123
ffb4351 
* Remove rogue comment
@MattGeri MattGeri mentioned this pull request Jul 11, 2017
Open
@philipjohn philipjohn self-assigned this Jul 11, 2017
@philipjohn philipjohn added this to the v2.1 milestone Jul 12, 2017
@philipjohn philipjohn merged commit 3deae05 into Automattic:2.1 Jul 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@philipjohn philipjohn

Labels
None yet
Projects
None yet
Milestone
v2.1
Development

Successfully merging this pull request may close these issues.
5 participants
@MattGeri @philipjohn @sboisvert @alexiskulash @emrikol