Re: [Semanticscuttle-devel] user session problems
Brought to you by:
cweiske
From: Christian W. <cw...@cw...> - 2011-05-17 20:28:37
|
Hello Mark, > Unit errors: 1a) One in Bookmark2TagTest::testGetPopularTagsDays > appears to exist in master, so I can't claim that. I spent some time > trying to resolve it, but was unable to. What error do you get? I don't get any errors in master. > 1b) Another unresolved issue is in > Tag2TagTest::testSearchThroughLinkedTags which is a result of a change > I did in getBookmarks(). I'll go ahead and apologize now if I'm wrong > in my understanding of how the tag2tag's work. The change I made > seems to make sense, and seemed to produce the results that I was > expecting. However, when I saw this test fail, red flags started > raising and now my change is in doubt. So either the update to > getBookmarks() is all wrong and all of it needs to be rejected, or it > is right and the test needs to be updated. Regardless if I'm right or > wrong, I'd love to use this opportunity to properly document this area > of work to help eliminate confusion in the future. I'll have a look at it. > 1c) The rest of the unit errors are in the API and it appears to be my > environment that is making them fail. I'll work on that later. If > someone could confirm they still pass. As I said, everyt single test works for me. > I wanted to go ahead and commit my changes because it'll be another > day or two to review what additional unit tests need to be > added/changed based on the above changes. Headed off to take a > certification test tomorrow which has taken up lots of my time. Fun > times. :) Good luck! > One thing outstanding that you mentioned is the desire to alter the > ajax script to use JSON. I fully agree with you and the change needs > to be made. However, because it is consistent with the other ajax > calls, I hoping you'll allow me to make those changes after the 0.98.0 > release. Fine with me. -- Regards/Mit freundlichen Grüßen Christian Weiske -=≡ Geeking around in the name of science since 1982 ≡=- |