Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
deMark4258yGetting angry just thinking about this.
Are code quality metrics/tools part of your build process? I feel like the only way to train some people is to enforce that each version/release should have same or better code quality and test coverage... -
Coworker deleting my code, thinking I broke the application. Later finds out it was broken by someone else's commit. Yeah, it happened. I was beyond pissed. :/
-
plumbus11938yPrint out the diff of the logic change and diff of commented unit test.
Hang them on the fridge side by side.
Deny knowing anything about them if he's even dumb enough to point out its his code and you made fun. -
Henrik1558y@deMark doesn't really work sadly if people don't learn to enjoy improving quality. I have seen lots of people go to extreme lengths to cheat the quality metric tools to stop flagging their changes.
-
deMark4258y@Henrik I was thinking about this the other day that people could write fake unit tests to cheat the metrics but that's almost as much effort as writing real ones...
-
Henrik1558y@deMark sadly very easy to fake. If you are making a web api for example just write a test that calls one of the api methods then mock nothing in the test except the database, and no asserts in the test at all. And voila, free coverage without actually covering anything.
I have seen so many of those over my years, and very little to no interest from the people that made them to fix them. -
LMagnus20578yIt's no effort at all to fake.
Assert.IsTrue(true)
This is why code reviews are important.
Related Rants
-
cdrice105"You gave us bad code! We ran it and now production is DOWN! Join this bridgeline now and help us fix this!" ...
-
MoboTheHobo36My Friend: Dude our Linux Server is not working anymore! Me: What? What did you do? My friend: Nothing I swe...
-
tommy15Right now someone at Google is coding something useless for us to laugh at on April Fools.
My co-worker not only doesn't create unit tests, he comment out my own unit tests after he changes the code and the test breaks.
undefined
unit tests
worst co-worker
fml