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

Remove actual dependency of dspace-stats on dspace-api #28

Merged
merged 1 commit into from Jun 19, 2012

Conversation

mwoodiupui
Copy link
Member

TestLocationUtils doesn't need to extend AbstractUnitTest. Removing the 'extends' clause removes the actual (as opposed to declared) dependency on dspace-api and should allow building both with and without tests enabled.

@tdonohue
Copy link
Member

I've tried this out locally, and it seems to fix the unit testing issues for me as well.

I'll merge this pull request shortly.

tdonohue added a commit that referenced this pull request Jun 19, 2012
Remove actual dependency of dspace-stats on dspace-api
@tdonohue tdonohue merged commit 7d9a0fa into DSpace:master Jun 19, 2012
artlowel referenced this pull request in atmire/DSpace Jun 13, 2014
Remove actual dependency of dspace-stats on dspace-api
hardyoyo pushed a commit to hardyoyo/DSpace that referenced this pull request Aug 5, 2015
re DSpace#1444 removed underline from sidebar header style
hardyoyo added a commit to hardyoyo/DSpace that referenced this pull request Jul 17, 2017
hardyoyo added a commit to hardyoyo/DSpace that referenced this pull request Aug 18, 2017
hardyoyo added a commit to hardyoyo/DSpace that referenced this pull request Sep 12, 2017
hardyoyo added a commit to hardyoyo/DSpace that referenced this pull request Dec 15, 2017
andreapetrucci93 pushed a commit to andreapetrucci93/DSpace that referenced this pull request Jul 6, 2018
hardyoyo added a commit to hardyoyo/DSpace that referenced this pull request Nov 28, 2018
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.

None yet

2 participants