334

My morning:

Me: Why did you just delete the failing unit tests?

Intern: I debugged it for a while and found one of the other developers broke it with his recent changes. I couldn't fix it.

Me: Did you let him know he broke it?

Intern: No.

Me: So you just deleted it and decided to pretend the feature isn't broken?

Intern: ... No ... I mean ... well you told us yesterday we needed to have all the tests passing.

(I NEED a stress ball people)

Comments
  • 63
    Well, I mean..... Technically... He's right... 😏
  • 23
    @geronimo ... you'd fit in well on this team. Want to replace me?
  • 18
    As a guy currently fixing a battery of ~25 unit tests to work with our completely remodeled website layout allow me to say:
    * eyeball twitch *
  • 9
    @kwilliams your twitchy eyeball warms my heart
  • 7
    Your little guys are scared I can tell, hahahaha.
  • 4
    @Heveabrasilien ... they should be now
  • 2
    @practiseSafeHex yeeeaahh... All the tests will pass.
  • 4
    It's not about fixing them its all about passing them ! Every thing passes 😜
  • 4
    Testing is stupid anyway.

    Tester c'est douter !
  • 5
    @geronimo Technically correct. The best kind of correct!
  • 5
    At least they didn't leave the tests but change them to always pass in an obscure way. Perhaps this is beyond internship level.
  • 11
    @cjhowald Assert.IsTrue (true);

    Test fixed.
  • 6
    @kwilliams or this:

    assert(result = expectedValue)
  • 12
    You don't need a stress ball... you need whiskey
  • 4
    Give this man a ++ so he can get a DevsRant stress ball
  • 4
    Is this what they call thinking "out of the box"? It's clearly an ugly world out there.
  • 8
    Luckily devRant don't send stressShotguns...
  • 6
    incoming new free swag: devrant pin up glass to throw at your frustrating teammate or intern for 2000 +1s
  • 4
    @notBob devRant shot glasses would be fun, though I wonder how many would survive international shipping intact
  • 1
    How did the other developer didn't knew he broke the test?
  • 1
    If the passing matters, he couldn't just use damn 'return' and notify the person, noooo...
  • 7
    To be fair, the kid was just an intern and I know back then I had these grandiose thoughts that all senior devs were completely professional and perfect. There is NO WAY a senior Dev would commit code that would break a test case. The issue must be in the test case then, right?

    (S)He sure learned the hard way that senior devs can be lazy and careless as fuck
  • 3
    @Yankeesrule I used the word intern as thats more his level as oppose to his role.

    He has several years experience in an unrelated language and we are training him up in a new language. He's also older than me.

    The "senior" was expecting that this project had the same level of automation as all of our other projects and would remotely run the tests and fire off an email if it failed.

    This was "too difficult" to implement when handed a step by step instruction sheet.
  • 1
    And the community agrees and you get a stress ball.
  • 4
    @practiseSafeHex yep it's amazing how lazy and uncommitted to learn some seniors can get. My coworker on my team is a tech lead while I am. Senior software engineer. Three months into the project and I still need to help her with git almost daily. Git isn't this flash in the pan technology. If we were talking javascript libraries than MAYBE. But Git?
  • 0
    He would fit perfectly in most of the Dev teams out there sadly.
    My "colleagues" would happily take him for me.
  • 1
    There. 256++ that was most satisfying
  • 1
    ++ in the hope that you throw it at the intern
  • 1
    @rightbrace ... too soft for them, need something bigger, heavier and more likely to take one of them out of action
  • 0
    Give them the task to write all working test, without assert(True == True).
Add Comment