ourresearch / total-impact-core

An api and backend code to gather the impacts of diverse scholarly products online.
http://impactstory.org
Other
55 stars 7 forks source link

test_alias_queue hangs #90

Closed hpiwowar closed 12 years ago

hpiwowar commented 12 years ago

test_alias_queue is stalling, looping on the database forever. The problem is triggered by line 70, alias_thread.run()

Bug may have been initiated by the cleanup in commit cd606f90645284ac03215a380fcd390a1d5e6d2e... this commit removed run_once, amongst other things.

Putting a @nottest on the test because now it hangs all tests.

kevcampb commented 12 years ago

I'll have a look at this. I assumed integration tests weren't in use when I committed to master. The aliases thread wasn't functional before my last commit, it didn't do the right things.

kevcampb commented 12 years ago

Now resolved. I've adding run_once back in to the alias thread, as that had been removed on the refactor.