It's pronounced Diane. I do data architecture, operations, and backend development. In my spare time I maintain Massive.js, a data mapper for Node.js and PostgreSQL.
If it does the wrong thing it doesn't matter how elegant it is. Of course reviewers should consider correctness a criterion for approval. That doesn't mean whiteboarding flowcharts or inspecting every conditional with a microscope. Tests help with this but don't achieve it on their own: a test documents the intent of the code, not that the intent matches the requirement.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
If it does the wrong thing it doesn't matter how elegant it is. Of course reviewers should consider correctness a criterion for approval. That doesn't mean whiteboarding flowcharts or inspecting every conditional with a microscope. Tests help with this but don't achieve it on their own: a test documents the intent of the code, not that the intent matches the requirement.