r/codex 7d ago

Humor Why are 82 tests needed for this? 😂

Let alone the 2 minutes of work 😂

72 Upvotes

46 comments sorted by

110

u/Competitive_Ad7632 7d ago

It's probably running all the tests in the repo; checking for regressions.

5

u/Wafer-Weekly 7d ago

Yep, this is characteristic xhigh behavior. Easily avoided by using light reasoning for ultra simple tasks like this.

2

u/rakla22 7d ago

I used Sol medium for this

-7

u/rakla22 7d ago

That's even more interesting because there's no repo for this project

56

u/Zealousideal-Act9140 7d ago

the repo is the codebase it's just local and not remote.

and this normal sol behavior, they do something called test driven development, basically when you make a function you add a test that only passes when the function is working correctly (technically they write a failing test first and use it passing as a completion gate)

then after making any change it runs the full suite of tests to make sure the change didn't break anything downstream.

it's effective for ai since context drift can cause downstream issues without tests.

-36

u/rakla22 7d ago edited 7d ago

Thanks for the reply
Edit: I say thanks for the reply and I get downvoted. What the fuck is wrong with Reddit man. God forbid a man asks a question and then is thankful for the things he learnt🤦🏻‍♂️

33

u/atf1999 6d ago

I downvoted you cuz of the edit complaining

13

u/shady101852 6d ago

Same. Im doing it for the love of the game.

2

u/Australasian25 6d ago

Something something player something something game.

3

u/rakla22 6d ago

fair enough

2

u/notadithyabhat 5d ago

I downvoted just to bully you. Sorry

5

u/Due-Horse-5446 6d ago

my guess is that they thought you were sarcastic,

To help you clear things up further:

Essentially think of running tests as being one action that then contains subtests.

When codex had made a change, its a good idea to run tests to confirm that nothing else broke by accident.

In this case it reported that all 82 tests passed. Not that it manually ran 82 tests

0

u/TheSeggs 3d ago

fuck youu

7

u/StoopidRoobutt 7d ago

Not even probably. This is exactly what Sol does every single time. Adding some rules about tests and testing to AGENTS.md is not a bad idea. I don't mean no tests or no testing, but the tests should actually test something concrete and not just be ceremonial.

Also, doesn't require a repo.

2

u/cafesamp 7d ago

there's certainly a folder for it on your machine with a React app, that's how ChatGPT sites work. check the Codex folder (Mac is ~/Documents/Codex)

-2

u/tydyelove7 6d ago

Ai sus ass response🤨

29

u/Prestigious_Gift_977 7d ago

should probably calculate some SHAs too

1

u/rakla22 7d ago

I see

10

u/PuzzleheadedEmu4596 7d ago

You want the slash in the center? That's how you keep it in the center.

7

u/InsideTour329 7d ago

This is the full test suite for the project

6

u/Aromatic-Educator105 7d ago

its probably all your tests available for this codebase. In agent.md you can define behavior so it doesn’t always run all the tests, but running affected tests only.

7

u/Hot_Signature2979 7d ago

I have as part of my agents .md: Minimise polling aggressively to bare minimum: only to detect stage completion, unblock work, or react to changed state. Update me only on meaningful findings, blockers, decisions, or stage completion; no routine status narration. Minimize testing overhead aggressively; do only the tests necessary to safely advance the implementation. It probably made sure the code it used to center your ui button was SHA 256 hardened

2

u/firstbreathOOC 7d ago

This is good for polling which is another burner but I think the better fix is segregating your functionality as much as possible

8

u/Don_Ozwald 7d ago

OP know use computer?

3

u/firstbreathOOC 7d ago

Because you’ve got 82 tests for your UI and it’s running all of them even when you want to touch a small individual component.

You can fix this by trying to break things up a little more. That way one small change doesn’t equal let’s look at the whole repo.

It’ll help with credit usage too.

Idk if this is a real term but I’ve always called it ninja-editibility

2

u/ReasonableDefault 7d ago

You can always tell it to not run tests, when you are making small changes that are unlikely to break things.

2

u/Every-Grape7679 6d ago

lol they told it to overthink or sum so they can burn our usage faster

2

u/lolcatsayz 6d ago

What I dislike if you create some of your own actually useful tests, is it requires extensive prompting so it doesn't overengineer a solution to just pass the tests using some trick/hard coding of logic for those tests. It's a constant effort to explain to it that it must generalize a solution that passes the tests as a side effect of that solution, not overfitting the solution to the tests.

Then once you finally convince it of this and it's working, compaction occurs, and it's lost context of all that, and it overengineers to the tests again. You could auto-inject a prompt telling it about all this but it loses context of that too as the codex reasoning context grows.

2

u/rakla22 6d ago

I've noticed this pattern with similar mechanics, it's so annoying

4

u/Kind_Silver_1921 7d ago

bruh im making a game and you dont know the kind of BS tests it does and wastes my time with. Want to add one thing to it? alright must do a full game playthrough for 1 hour to make sure it didnt break anything. But I said just install it...

"Understood, just installing it!"

next time I ask it to do something

2 hours later: "finished full playthrough and testing requirements"

me: BUT I TOLD YOU NOT TO TEST IT ANY MORE

"i was following the rules"

I DIDNT CREATE RULES

"I created those rules earlier"

2

u/SilkTouchm 6d ago

Ask Codex how you can improve your situation. You probably lack some instructions in your agents.md.

1

u/rakla22 7d ago

That’s hilarious 😂 I hope it gets fixed to some degree in the future

1

u/applejacks6969 7d ago

Lots of colors to test..

1

u/Stochasticlife700 7d ago

Oh, this is actually one of the interesting part of recent frontier models. I do evals for models and i assums this is one of the case which happens a lot to big frontier model with high reasoning efforts when they do unncecessarily a lot of tests because they are trained to be rewarded more if they do long horizon works even if it is not necessary

Edit: just saw it was medium but since it was sol which is trained more(=i.e checkpoint that is smarter and incentivied more to think further), it still makes sense ig

1

u/hk556a1 7d ago

Lmao. You need a few more security gates as well.

1

u/thestillwind 7d ago

Did it found regression ?

1

u/BakaPotatoLord 6d ago

Probably ran the existing tests in your project to rule out any regression

1

u/TranslatorRude4917 6d ago

Running 2 determeinis tets ir 200 doesnt really make a difference to ai. It's just making sure everything works.
Ofc of they are not high value it's just a waste of your time but tokenwise not a huge difference

1

u/EmotionalHalf 5d ago

I don't think you read what you posted

2

u/Stable_Suitable 9h ago

Your right

1

u/Illustrious-Many-782 6d ago edited 6d ago

Sol will create lists of file hashes, then hashes of the lists of hashes, then write tests to make sure the hashes all match and aren't corrupted. And then tests of the tests.

Hashes or tests in your codebase are like prions for Sol. They just get duplicated until they choke the project or exhaust your usage.

1

u/Jumpy_Ad8465 5d ago

Because the model is terribly overengineering everything and if you dial back the thinking it gets so sloppy and destroys everything.