Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

I've been tasked with doing a very superficial review of a codebase produced by an adult who purports to have decades of database/backend experience with the assistance of a well-known agent.

While skimming tests for the python backend, I spotted the following:

    @patch.dict(os.environ, {"ENVIRONMENT": "production"})
    def test_settings_environment_from_env(self) -> None:
        """Test environment setting from env var."""
        from importlib import reload

        import app.config

        reload(app.config)

        # Settings should use env var
        assert os.environ.get("ENVIRONMENT") == "production"
This isn't an outlier. There are smells everywhere.


If it is so obvious to you that there is a smell here then an agent would have caught it. Try it yourself.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: