Skip to content

Commit faae912

Browse files
committed
Merge pull request #2281 from dlsniper/improve-coverage-support
Improve coverage support for packages (fixes #2276)
2 parents 9bd1c85 + 4e3df5c commit faae912

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/com/goide/runconfig/testing/GoTestRunningState.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@
4040
import com.intellij.openapi.vfs.VirtualFile;
4141
import com.intellij.psi.PsiFile;
4242
import com.intellij.psi.PsiManager;
43+
import com.intellij.util.PathUtil;
4344
import com.intellij.util.containers.ContainerUtil;
4445
import org.jetbrains.annotations.NotNull;
4546
import org.jetbrains.annotations.Nullable;
@@ -83,11 +84,14 @@ protected GoExecutor patchExecutor(@NotNull GoExecutor executor) throws Executio
8384
String relativePath = FileUtil.getRelativePath(myConfiguration.getWorkingDirectory(),
8485
myConfiguration.getDirectoryPath(),
8586
File.separatorChar);
87+
// TODO Once Go gets support for covering multiple packages the ternary condition should be reverted
88+
// See https://golang.org/issues/6909
89+
String pathSuffix = (myCoverageFilePath == null) ? "..." : ".";
8690
if (relativePath != null) {
87-
executor.withParameters("./" + relativePath + "/...");
91+
executor.withParameters("./" + relativePath + "/" + pathSuffix);
8892
}
8993
else {
90-
executor.withParameters("./...");
94+
executor.withParameters("./" + pathSuffix);
9195
executor.withWorkDirectory(myConfiguration.getDirectoryPath());
9296
}
9397
addFilterParameter(executor, myConfiguration.getPattern());

0 commit comments

Comments
 (0)