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

Move tests modules #19

Merged
merged 1 commit into from
Apr 9, 2015
Merged

Move tests modules #19

merged 1 commit into from
Apr 9, 2015

Conversation

dawehner
Copy link
Collaborator

@dawehner dawehner commented Apr 9, 2015

Our test module is part of the lib/ tree, which seems to confuse some autoloaders like xautoload in
connection with simpletest.

Move it to tests/modules/service_container_test instead

@LionsAd LionsAd added the RTBM label Apr 9, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 38d554e on dawehner:move-modules into 5a216f1 on LionsAd:7.x-1.x.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 38d554e on dawehner:move-modules into 5a216f1 on LionsAd:7.x-1.x.

LionsAd added a commit that referenced this pull request Apr 9, 2015
@LionsAd
Merge pull request #19 from dawehner/move-modules
15eabce 
Move tests modules
@LionsAd LionsAd merged commit 15eabce into LionsAd:7.x-1.x Apr 9, 2015
@LionsAd
Copy link
Owner

LionsAd commented Apr 10, 2015

Pushed to d.org, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
RTBM
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dawehner @coveralls @LionsAd