Removed unnecessary member variable.

This commit is contained in:
NissimHadar 2019-02-14 10:38:19 -08:00
parent 701c18c9eb
commit 6752092899
2 changed files with 6 additions and 7 deletions

View file

@ -33,7 +33,6 @@ void AWSInterface::createWebPageFromResults(const QString& testResults,
// For example `D:/tt/TestResults--2019-02-10_17-30-57(local)[DESKTOP-6BO62Q9].zip` // For example `D:/tt/TestResults--2019-02-10_17-30-57(local)[DESKTOP-6BO62Q9].zip`
QStringList parts = testResults.split('/'); QStringList parts = testResults.split('/');
QString zipFilename = parts[parts.length() - 1]; QString zipFilename = parts[parts.length() - 1];
_zipFolderName = _workingDirectory + "/" + zipFilename.split('.')[0];
QStringList zipFolderNameParts = zipFilename.split(QRegExp("[\\(\\)\\[\\]]"), QString::SkipEmptyParts); QStringList zipFolderNameParts = zipFilename.split(QRegExp("[\\(\\)\\[\\]]"), QString::SkipEmptyParts);
@ -50,7 +49,8 @@ void AWSInterface::createWebPageFromResults(const QString& testResults,
_urlLineEdit = urlLineEdit; _urlLineEdit = urlLineEdit;
_urlLineEdit->setEnabled(false); _urlLineEdit->setEnabled(false);
extractTestFailuresFromZippedFolder(); QString zipFilenameWithoutExtension = zipFilename.split('.')[0];
extractTestFailuresFromZippedFolder(_workingDirectory + "/" + zipFilenameWithoutExtension);
createHTMLFile(); createHTMLFile();
if (updateAWSCheckBox->isChecked()) { if (updateAWSCheckBox->isChecked()) {
@ -61,12 +61,12 @@ void AWSInterface::createWebPageFromResults(const QString& testResults,
} }
} }
void AWSInterface::extractTestFailuresFromZippedFolder() { void AWSInterface::extractTestFailuresFromZippedFolder(const QString& folderName) {
// For a test results zip file called `D:/tt/TestResults--2018-10-02_16-54-11(9426)[DESKTOP-PMKNLSQ].zip` // For a test results zip file called `D:/tt/TestResults--2018-10-02_16-54-11(9426)[DESKTOP-PMKNLSQ].zip`
// the folder will be called `TestResults--2018-10-02_16-54-11(9426)[DESKTOP-PMKNLSQ]` // the folder will be called `TestResults--2018-10-02_16-54-11(9426)[DESKTOP-PMKNLSQ]`
// and, this folder will be in the working directory // and, this folder will be in the working directory
if (QDir(_zipFolderName).exists()) { if (QDir(folderName).exists()) {
QDir dir = _zipFolderName; QDir dir = folderName;
dir.removeRecursively(); dir.removeRecursively();
} }

View file

@ -30,7 +30,7 @@ public:
QCheckBox* updateAWSCheckBox, QCheckBox* updateAWSCheckBox,
QLineEdit* urlLineEdit); QLineEdit* urlLineEdit);
void extractTestFailuresFromZippedFolder(); void extractTestFailuresFromZippedFolder(const QString& folderName);
void createHTMLFile(); void createHTMLFile();
void startHTMLpage(QTextStream& stream); void startHTMLpage(QTextStream& stream);
@ -49,7 +49,6 @@ public:
private: private:
QString _testResults; QString _testResults;
QString _zipFolderName;
QString _workingDirectory; QString _workingDirectory;
QString _resultsFolder; QString _resultsFolder;
QString _htmlFailuresFolder; QString _htmlFailuresFolder;