`"bring funk name"` doesn't play nicely with multiple cursors

[pokey]

On the following file:

function aaa() {
  console.log("Hello from |");
  console.log("Goodbye from |");
}

function bbb() {
  console.log("Hello from |");
}

(Note there are 3 cursors). When I say "bring funk name", I would expect to get:

function aaa() {
  console.log("Hello from aaa");
  console.log("Goodbye from aaa");
}

function bbb() {
  console.log("Hello from bbb");
}

But instead I get

function aaa() {
  console.log("Hello from aaa");
  console.log("Goodbye from ");
}

function bbb() {
  console.log("Hello from bbb");
}

Seems to work fine if you have multiple cursors in just one function, or exactly one cursor per function, but seems to break if you have differing numbers of cursors per function. Guessing maybe related to target deduplication in our pipeline or something?

[AndreasArvidsson]

Indeed if I remove uniqTargets() from TargetPipelineRunner.run() this bug goes away, but in its place 14 tests fail. This one is probably going to be a bit tricky to solve.

[AndreasArvidsson]

@pokey Here is quite a simplistic solution, but one that fixes your particular case.
#3083