this post was submitted on 12 Sep 2023
851 points (99.0% liked)

Programmer Humor

19606 readers
1111 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS
 
all 31 comments
sorted by: hot top controversial new old
[–] [email protected] 37 points 1 year ago (1 children)

I recently spent about two weeks trying to figure out why an intercontinental connection between two of our sites was broken. Not really my job, I just care about application level, but the network guys were beyond useless.

In the end I had the problem isolated to a specific network segment in India, which made them look at the right system and fix things. The reason? "We put up a firewall the day your problems started which blocks everything, if we allow your connection it works".

[–] [email protected] 4 points 1 year ago

"If we allow your connection, it works"

Oh, surprising

[–] [email protected] 30 points 1 year ago (1 children)

The one I hate? Your unit tests pass when run locally, and in your sandbox environment, and in dev, and in UAT, but prod? Fuck that, failing with reckless abandon.

[–] SomeKindaName 2 points 1 year ago

Are you kidding me? Tests in prod? Hopefully that means you didn't fully roll out. That means someone didn't get a 3am page.

This sounds like a god-damned resounding success.

[–] shotgun_crab 29 points 1 year ago (3 children)

You guys have unit tests…?

[–] BradleyUffner 11 points 1 year ago (2 children)

Ohh yeah, loads! Do they actually test anything useful? Not so much.

[–] gornius 1 points 1 year ago

I actually had to refactor configuration module some time ago. These really came in handy. But was it worth it? Well... it saved some time, the time that could be used to debug problems manually, but it made me a lot more confident that the functionality that worked before, worked after.

[–] jaybone 1 points 1 year ago

You just have to mock everything, so it’s basically testing nothing. But you get those code coverage numbers.

[–] [email protected] 3 points 1 year ago

The ones I have seen so far are probably written by the same people who don't understand the usefulness of comments, I reckon. And maintained with the same enthusiasm.

[–] alphacyberranger 21 points 1 year ago (4 children)

Everyone talks about testing, but I have never seen it in the wild

[–] Jaydeep 12 points 1 year ago (1 children)

It's common in critical projects.

[–] alphacyberranger 10 points 1 year ago* (last edited 1 year ago)

Chuckles.... you have not worked in my company.

[–] [email protected] 8 points 1 year ago

After many years (10+), I finally find a company that actually, really, implements CI/CD. Then I look at the tests and it's actually the most inane shit imaginable, tacked on top of ancient existing code, not maintained. I spent more time fixing the stupid tests than actually fixing the bugs I was tasked fixing. Amazing.

[–] buzziebee 5 points 1 year ago (1 children)

We test the shit out of our Apis. We do more API level/integration testing though.

I.e. a test will be something like "if the db is in this state, and we hit this endpoint with these params, does it return what we expect and update the db correctly".

Our app is primarily about users maintaining stuff on big datasets with complicated aggregation and approval logic. So setting up a scenario and checking the app does what the business logic says it will do is what we want to know.

It makes refactoring wayyyyy less painful to just know that the app will always behave itself. Rather than testing whether a function can add 1 + 2 correctly, we can test each endpoint does what it's supposed to do.

It gives us loads of confidence that the backend is doing what it's supposed to. If you do a huge refactor you don't need to worry about whether you broke the test or if the test is failing correctly. If the tests all pass everything is working as it should.

Downside is longer test execution times (because a temporary db needs set up) when running the full suite. Worth the trade off for us though.

[–] jaybone 1 points 1 year ago

Those are good tests. But that sounds more like an integration test than a unit test. And we should have both.

[–] [email protected] 2 points 1 year ago (1 children)

Meanwhile I'm very upset our unit test coverage is only 40%.

Like, it's the number 1 priority for the principle & staff engineers to get that up to 80% across the codebase.

[–] [email protected] 2 points 1 year ago

And it's number 1 priority for management to employ as few developers as possible and stretch their team as thinly as possible. Hence still no unit tests in any of the companies I've worked at recently, despite everyone knowing they're worth it, including lip service from management. They just won't invest in testing, no matter what. One company even fired all the testers then complained to the developers that the product was getting less reliable.

[–] [email protected] 12 points 1 year ago (4 children)

Working at a company with no automated tests. There's not even a collection of regression tests or anything to follow. I was wondering if anyone could share or point me towards a good template to start building out test cases as a first step?

[–] big_slap 7 points 1 year ago

I think this is something you're gonna have to just jump into and start since you don't have anything to work off of. it's going to take a lot of work, but at least you'll be able to work off your own ATFs once you finish. good luck..

[–] [email protected] 6 points 1 year ago

I mean, start with trivial cases of the core functionality of what your system does. Then build upon it based on your own findings and what your clients report.

E.g. if your system loads images then put in a tiny 5x5px solid square or checkerboard pattern and see if it loads. Then try putting multiple images, different formats (webp, gif, png, tga) etc. see if that breaks anything, keep building out.

[–] [email protected] 2 points 1 year ago

It probably really depends on the project, though I'd probably try and start with the tests that are easiest/nicest to write and those which will be most useful. Look for complex logic that is also quite self-contained.

That will probably help to convince others of the value of tests if they aren't onboard already.

[–] [email protected] 2 points 1 year ago (1 children)

if there are zero automated tests, things probably weren't written with (automated) testing in mind, so there may be a lot of coupling... starting off with integration tests just to validate existing behavior is a good start. hopefully the existing applications aren't also inextricably bound to environments that are run on pet servers and managed by other teams...

[–] [email protected] 1 points 1 year ago

Thankfully, the team has full control without external team dependencies

[–] [email protected] 12 points 1 year ago (1 children)

Ahhh my favorite way to fix a broken test: just remove it!

[–] [email protected] 4 points 1 year ago

Mark it as intended-to-fail, that way if someone fixes your bug it's their test failure.

[–] [email protected] 9 points 1 year ago (1 children)

I'm in this meme right now.

I just made a tiny change that somehow broke a major feature in testing. The disbelief is real.

[–] [email protected] 14 points 1 year ago

Rollback, tests still fail.

Flip a table.

[–] [email protected] 7 points 1 year ago

sometimes it is the unit tests.

[–] [email protected] 6 points 1 year ago

Wait! US-EAST-1 is dead (Shia surprise)

[–] [email protected] 4 points 1 year ago

Oof I am here right now, which is why I'm procrastinating on socials...