Skip to content
This repository was archived by the owner on Sep 8, 2022. It is now read-only.

Commit d7bc089

Browse files
authored
Merge pull request #81 from som-snytt/issue/80
Diff transcript shows correct order
2 parents 4344c94 + e3b4713 commit d7bc089

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/main/scala/scala/tools/partest/nest/Runner.scala

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -297,10 +297,11 @@ class Runner(val testFile: File, val suiteRunner: SuiteRunner, val nestUI: NestU
297297
b.toList
298298
}
299299

300+
// diff logfile checkfile
300301
def currentDiff = {
301302
val logged = file2String(logFile).linesIfNonEmpty.toList
302-
val (other, othername) = if (checkFile.canRead) (filteredCheck, checkFile.getName) else (Nil, "empty")
303-
compareContents(original = other, revised = logged, originalName = othername, revisedName = logFile.getName)
303+
val (checked, checkname) = if (checkFile.canRead) (filteredCheck, checkFile.getName) else (Nil, "empty")
304+
compareContents(original = logged, revised = checked, originalName = logFile.getName, revisedName = checkname)
304305
}
305306

306307
val gitRunner = List("/usr/local/bin/git", "/usr/bin/git") map (f => new java.io.File(f)) find (_.canRead)
@@ -392,8 +393,11 @@ class Runner(val testFile: File, val suiteRunner: SuiteRunner, val nestUI: NestU
392393
val bestDiff =
393394
if (updating.isEmpty) ""
394395
else if (checkFile.canRead)
395-
withTempFile(outFile, fileBase, filteredCheck) { f =>
396-
gitDiff(logFile, f) getOrElse f"diff $logFile $checkFile%n$diff"
396+
gitRunner match {
397+
case None => diff
398+
case _ => withTempFile(outFile, fileBase, filteredCheck) { f =>
399+
gitDiff(logFile, f) getOrElse diff
400+
}
397401
}
398402
else diff
399403
_transcript append bestDiff

0 commit comments

Comments
 (0)