id remove testutils, add more tests

[binjip978]

No description provided.

[codecov]

Codecov Report

Patch coverage: 92.30% and project coverage change: +0.15 🎉

Comparison is base (809e479) 74.93% compared to head (80bb85d) 75.08%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2641      +/-   ##
==========================================
+ Coverage   74.93%   75.08%   +0.15%     
==========================================
  Files         413      413              
  Lines       41661    41662       +1     
==========================================
+ Hits        31217    31283      +66     
+ Misses      10444    10379      -65     

Impacted Files	Coverage Δ
cmds/core/id/id.go	73.91% <92.30%> (+57.24%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[rminnich]

looks good. One question: is it worth using t.Run for any of these tests.

golangci-lint always yells at me when I don't :-)

[binjip978]

From this document https://github.com/golang/go/wiki/TestComments#go-test-comments I don't see why we need to add t.Run to table driven tests. But I'm overusing t.Fatal in table driven tests. Will probably address it in separate PR to cover multiple commands.