• JoshCodes
    link
    fedilink
    English
    572 months ago

    Run it in your head, find the edge cases yourself, fix the bug… weakling.

    Or do what I do in real life which is patch in new bugs and even a security flaw or two.

    • Psaldorn
      link
      fedilink
      332 months ago

      Merging failing tests so everybody else has failing tests and wastes time figuring out why.

      Nothing neutral here

      • @RamblingPanda
        link
        12 months ago

        That’s what the pipeline is for. It’s not that hard to pinpoint the commit that lead to the errors.

        • Psaldorn
          link
          fedilink
          82 months ago

          If I rebase my branch with main I do not expect any failing tests. If you waste my time merging shit code, fuck you. Fix your shit.

          Unless prod is on fire and the CEO is prowling (even then, I’d argue standards should be maintained)

          • @RamblingPanda
            link
            52 months ago

            I don’t say this is good practice, you shouldn’t even be able to merge to main with failing tests. I’ve implemented an emergency flag to do this, but I don’t want to use it in normal, daily business.

  • @[email protected]
    link
    fedilink
    382 months ago

    Tests are just booby traps for the other engineers so they don’t break your code by mistake.

    • DacoTaco
      link
      fedilink
      13
      edit-2
      2 months ago

      Its funny cause its true. I often design tests to be “if a case/enum value is added this test will explode and tell them to add code here”

  • xep
    link
    fedilink
    332 months ago

    Real programmers test in production.

  • @[email protected]
    link
    fedilink
    202 months ago

    Oh I trust my code, but I don’t trust my coworkers not to break something on the very next commit.

  • @[email protected]
    link
    fedilink
    202 months ago

    While I know that these days, bugs in code can cause real-world harm (personal info leaks, superannuation records lost, lol google), I find it humorous to think of the equivalent, even worse outcomes in my discipline (chemical/process engineering).

    “Didn’t do any checks, fuck it, I know this calculation is fire 🔥”

    Later: 🔥🔥💥

    • @[email protected]
      link
      fedilink
      52 months ago

      It’s more: I have routed a few pipes in our test system and it’s now spitting out water known to be contaminated but now should have some extra sprinkles in so it’s fine.

      What I’m saying is it’s even worse than didn’t do any checks. It’s willfully ignoring existing checks intentionally.

  • @[email protected]
    link
    fedilink
    132 months ago

    I get a small amount of joy from clicking the “request changes” button and blocking some doofus from merging lazy untested code.

    • shastaxc
      link
      fedilink
      42 months ago

      I love going into a PR with 3 approvals already and shitting all over it

  • tiredofsametab
    link
    fedilink
    132 months ago

    I physically reacted to this post with a combination of disgust, anger, and fear. Do tests. All of the tests. Randomize the order in which your tests run. Cover all branches.

      • @[email protected]
        link
        fedilink
        22 months ago

        Tbh I’m not a web person (more of a backend person) and don’t know the recommended practices. display: grid; is a good friend of mine xD

        • @[email protected]
          link
          fedilink
          English
          22 months ago

          I think using display: grid; as your default is the better default, so you’re all set. :)

        • tiredofsametab
          link
          fedilink
          22 months ago

          People can pull <table> from my cold, dead hands.

          (though I’m usually only using it to display some status just for me and not for external consumption; the UI side can have a JSON if it ever comes to that).

          I used to be a full-stack dev, but I’ve been pure backend for so long now, everything I knew is outdated or deprecated.

          • @[email protected]
            link
            fedilink
            22 months ago

            everything I knew is outdated or deprecated

            Given the way the frontend world seems to work, this means you’ve been backend-only for at least a week lol

  • @[email protected]
    link
    fedilink
    English
    52 months ago

    You can’t trust others to not break your wonderful code. Write tests for the regression.

  • @PenisDuckCuck9001
    link
    12 months ago

    The best way is to try it over and over until it works and then assume it works but then go insane wondering where all the edge case bugs are coming from.

    I wrote a test one time.