Mirrors/overte
3
0
Fork 0
mirror of https://github.com/overte-org/overte.git synced 2025-04-19 15:43:50 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Corrected pulling PR number from the executable name.

Browse source
This commit is contained in:
NissimHadar 2018-09-17 16:06:11 -07:00
parent f046d3b87d
commit 0c3e3e977e
3 changed files with 15 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
tools/auto-tester/src/Test.cpp
View file

@ -190,7 +190,7 @@ void Test::startTestsEvaluation(const bool isRunningFromCommandLine,
_isRunningInAutomaticTestRun = isRunningInAutomaticTestRun;
if (snapshotDirectory.isNull()) {
// Get list of JPEG images in folder, sorted by name
// Get list of PNG images in folder, sorted by name
QString previousSelection = _snapshotDirectory;
QString parent = previousSelection.left(previousSelection.lastIndexOf('/'));
if (!parent.isNull() && parent.right(1) != "/") {

26
tools/auto-tester/src/TestRunner.cpp
View file

@ -270,23 +270,21 @@ void TestRunner::startLocalServerProcesses() {
}
void TestRunner::runInterfaceWithTestScript() {
QString commandLine;
QString exeFile = QString("\"") + QDir::toNativeSeparators(_installationFolder) +
"\\interface.exe\"";
QString url = QString("hifi://localhost");
if (_runServerless->isChecked()) {
// Move to an empty area
commandLine =
QString("\"") + QDir::toNativeSeparators(_installationFolder) +
"\\interface.exe\" --url hifi://localhost/9999,9999,9999/0.0,0.0,0.0,1.0 --testScript https://raw.githubusercontent.com/" + _user +
"/hifi_tests/" + _branch + "/tests/testRecursive.js quitWhenFinished --testResultsLocation " +
_snapshotFolder;
} else {
// There is no content, so no need to move
commandLine = QString("\"") + QDir::toNativeSeparators(_installationFolder) +
"\\interface.exe\" --url hifi://localhost --testScript https://raw.githubusercontent.com/" + _user +
"/hifi_tests/" + _branch +
"/tests/content/entity/zone/testRecursive.js quitWhenFinished --testResultsLocation " + _snapshotFolder;
url = url + "/9999,9999,9999/0.0,0.0,0.0,1.0";
}
QString testScript =
QString("https://raw.githubusercontent.com/") + _user + "/hifi_tests/" + _branch + "/tests/testRecursive.js";
QString commandLine = exeFile + " --url " + url + " --testScript " + testScript +
" quitWhenFinished --testResultsLocation " + _snapshotFolder;
interfaceWorker->setCommandLine(commandLine);
emit startInterface();
}
@ -508,10 +506,10 @@ QString TestRunner::getPRNumberFromURL(const QString& url) {
try {
QStringList urlParts = url.split("/");
QStringList filenameParts = urlParts[urlParts.size() - 1].split("-");
if (filenameParts.size() != 5) {
if (filenameParts.size() <= 3) {
throw "URL not in expected format, should look like `https://deployment.highfidelity.com/jobs/pr-build/label%3Dwindows/13023/HighFidelity-Beta-Interface-PR14006-be76c43.exe`";
}
return filenameParts[3];
return filenameParts[filenameParts.size() - 2];
} catch (QString errorMessage) {
QMessageBox::critical(0, "Internal error: " + QString(__FILE__) + ":" + QString::number(__LINE__), errorMessage);
exit(-1);

4
tools/auto-tester/src/ui/AutoTester.ui
View file

@ -498,7 +498,7 @@
<string>Serveless</string>
</property>
<property name="checked">
<bool>true</bool>
<bool>false</bool>
</property>
</widget>
<widget class="QCheckBox" name="checkBoxRunLatest">
@ -529,7 +529,7 @@
<x>150</x>
<y>98</y>
<width>461</width>
<height>24</height>
<height>28</height>
</rect>
</property>
</widget>