Closed hbenl closed 3 years ago
Thanks, looks good! I've updated the converter accordingly. A few things:
setTimeout
before calling the command inside of activate
testExplorer.useNativeTesting
is on -- it could be useful for debugging, but for end users just results in duplicate explorer views I've just published this.
useNativeTesting
the default.Thanks! Realized I forgot to push the updates to the converter extension, I've now done so in 0.0.8
@hbenl I think you can re-add the extension dependency now. Right @connor4312?
Yep
@TylerLeonhardt Thanks for the reminder.
@connor4312 Here's the PR for the changes we talked about. I would suggest that you register/unregister your converter whenever the
testExplorer.useNativeTesting
flag changes. By the way, you probably don't need to worry about missing events if you register the converter late: you will miss events for past test runs, but not those for loading the tests: when an adapter sends its test tree, theTestHub
remembers it and when a controller registers later on, theTestHub
will send "synthetic" load events containing the test tree to it.