datawire / teleproxy

18 stars 9 forks source link

edgectl: smoke tests should not build and use the edgectl binary #180

Open ark3 opened 5 years ago

ark3 commented 5 years ago

With #173 landed, we now have make check calling go test calling make [...]/edgectl. The dtest/testprocess.Make/MakeSudo features exist to avoid this sort of thing, but edgectl end-to-end tests cannot use them right now because edgectl re-execs itself again to call its internal Teleproxy. We should refactor Teleproxy so it can be run as a function call in a subroutine. Then we should be able to remove this testing hackery.