Skip to content

all: remove underscore in test function names

Administrator requested to merge rvt/no-test-underscore into master

Created by: rvantonder

As in title. I added underscores to my tests because I've seen it in other places, and I realize this is a Go convention for some projects, but should not necessarily be for ours. I tend to agree with @keegancsmith:

I think its pretty rare in our codebase and I am not sure I am a fan of it.

For consistency I'm putting up this PR to remove underscores throughout.

If these seem overzealous, LMK.

Merge request reports