qbs_benchmarker: Validate commit parameters

It's possible to copy&paste old commit instead of a new one and easy to
overlook that mistake.

Change-Id: I408f256ba6b2f419b7cf90b7ed9ee582f91b081b
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>

1 files changed, 4 insertions, 0 deletions

diff --git a/tests/benchmarker/commandlineparser.cpp b/tests/benchmarker/commandlineparser.cpp
index e725ad0a0..9b08bd60d 100644
--- a/tests/benchmarker/commandlineparser.cpp
+++ b/tests/benchmarker/commandlineparser.cpp
@@ -82,6 +82,10 @@ void CommandLineParser::parse()
     }
     m_oldCommit = parser.value(oldCommitOption);
     m_newCommit = parser.value(newCommitOption);
+    if (m_oldCommit == m_newCommit) {
+        throw Exception(QStringLiteral("Error parsing command line: "
+                "'new commit' and 'old commit' must be different commits.\n%1").arg(parser.helpText()));
+    }
     m_testProjectFilePath = parser.value(testProjectOption);
     m_qbsRepoDirPath = parser.value(qbsRepoOption);
     const QStringList activitiesList = parser.value(activitiesOption).split(',');