Currently, no reputation points are being given, currently, to a user when her letter is signed. This should give +10. The last time we looked at reputation scoring, we were using "voting" rather than "signing" on letters, and in the transition to voting, we never connected that event to reputation changes.
In this issue, we should revisit the whole reputation attribution strategy, and decide which events should affect reputation score, then implement that strategy comprehensively. Finally, we'll need to test the reputation scoring mechanisms.
We currently have behavior tests for voting on various scored posts, but that test does not check the according of reputation. We also have a test in UserProfileTest.test_user_rep() -- but it only tests the UserProfile.apply_rep() function itself, rather than testing that rep is applied in connection to certain user-triggered events. What we need is to perform the reputation testing as part of full (selenium-driven) behavior tests that trigger events which are meant to cause rep changes.
Currently, no reputation points are being given, currently, to a user when her letter is signed. This should give +10. The last time we looked at reputation scoring, we were using "voting" rather than "signing" on letters, and in the transition to voting, we never connected that event to reputation changes.
In this issue, we should revisit the whole reputation attribution strategy, and decide which events should affect reputation score, then implement that strategy comprehensively. Finally, we'll need to test the reputation scoring mechanisms.
We currently have behavior tests for voting on various scored posts, but that test does not check the according of reputation. We also have a test in
UserProfileTest.test_user_rep()-- but it only tests theUserProfile.apply_rep()function itself, rather than testing that rep is applied in connection to certain user-triggered events. What we need is to perform the reputation testing as part of full (selenium-driven) behavior tests that trigger events which are meant to cause rep changes.