Skip to content

Commit

Permalink
ProcessTestApp: Make setupSubProcess const
Browse files Browse the repository at this point in the history
Change-Id: I1dd45baebb32135cba6a35040d8e87a0bcc5d3ac
Reviewed-by: Eike Ziller <[email protected]>
Reviewed-by: <[email protected]>
  • Loading branch information
JarekKobus committed Nov 9, 2022
1 parent c5db642 commit a8801ef
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
5 changes: 2 additions & 3 deletions tests/auto/utils/qtcprocess/processtestapp/processtestapp.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -77,22 +77,21 @@ SubProcessConfig::SubProcessConfig(const char *envVar, const QString &envVal)
{
}

void SubProcessConfig::setupSubProcess(QtcProcess *subProcess)
void SubProcessConfig::setupSubProcess(QtcProcess *subProcess) const
{
subProcess->setEnvironment(m_environment);
const FilePath filePath = FilePath::fromString(s_pathToProcessTestApp
+ QLatin1String("/processtestapp")).withExecutableSuffix();
subProcess->setCommand(CommandLine(filePath, {}));
}

void SubProcessConfig::setupSubProcess(QProcess *subProcess)
void SubProcessConfig::setupSubProcess(QProcess *subProcess) const
{
subProcess->setProcessEnvironment(m_environment.toProcessEnvironment());
subProcess->setProgram(FilePath::fromString(s_pathToProcessTestApp
+ QLatin1String("/processtestapp")).withExecutableSuffix().toString());
}


static void doCrash()
{
qFatal("The application has crashed purposefully!");
Expand Down
4 changes: 2 additions & 2 deletions tests/auto/utils/qtcprocess/processtestapp/processtestapp.h
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,8 @@ class SubProcessConfig
{
public:
SubProcessConfig(const char *envVar, const QString &envVal);
void setupSubProcess(Utils::QtcProcess *subProcess);
void setupSubProcess(QProcess *subProcess);
void setupSubProcess(Utils::QtcProcess *subProcess) const;
void setupSubProcess(QProcess *subProcess) const;

static void setPathToProcessTestApp(const QString &path);

Expand Down

0 comments on commit a8801ef

Please sign in to comment.