<@UB2J9BQA0>: I’d like to add some tests for the `...
# developmentf
fast-nail-55400
02/25/2022, 7:11 PM@hundreds-father-404: I’d like to add some tests for the
gogo_modgo_packagefast-nail-55400
02/25/2022, 7:11 PMThis would probably require Pants developers to have Go installed if they ever run
./pants test ::fast-nail-55400
02/25/2022, 7:12 PMBut it would let me add proper Go tests
h
hundreds-father-404
02/25/2022, 7:15 PMI'm tempted to say this means we should lean into going back to Pants installing Go. But given the bug with immutable caches, I'm hesitant to proceed on that
hundreds-father-404
02/25/2022, 7:16 PMI'm generally +1 for adding Go tests. The general project's philosophy is to lean heavily into dogfooding—if our dogfood is sh*t, we should feel that pain
The Go logic has gotten pretty complex, so it sounds like it would be useful for us to have tests for it in Go. Whereas now we're constrained to Python integration-tests
f
fast-nail-55400
02/25/2022, 7:23 PMcontext: I’m updating the analysis code to properly analyze cgo sources, thereby increasing complexity of that code.
👍 1