Spent some time investigating which port is being taken in the tests, and why. But then got tired of that shit, and took a break, and took another ticket. Filed by our African QA - long profile ids not accepted by the service.
Started investigating. Ha, she even wrote that our db table declares the field as INT4. Is not she cool?
So I ALTERed TABLE on staging - and then updated a bunch of tests to work with a long sample id.
But then, strangely, the tests were failing - until I figured that I also had to update the DB schema in the project, because the tests create test db every launch. Omg.
Anyway, it was a pleasure to deal with a sane code.
That's it for today. Will send the PR for review Monday.
Also, reviewed Qian's code; his code is really beautiful; a little bit verbose, but it's okay. Pretty well detailed.