Skip to content

Commit

Permalink
#669: Attempt to fix background script
Browse files Browse the repository at this point in the history
moved checkInstallation before run test validation
  • Loading branch information
jan-vcapgemini committed Oct 1, 2024
1 parent d571df9 commit d7dc137
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -72,10 +72,9 @@ public void testAndroidStudioRun(String os, WireMockRuntimeInfo wmRuntimeInfo) t
commandlet.run();

// assert
checkInstallation(this.context);
assertThat(commandlet.getToolBinPath().resolve("android-studio-test")).hasContent(
ANDROID_STUDIO + " " + this.context.getSystemInfo().getOs() + " " + this.context.getWorkspacePath());

checkInstallation(this.context);
}

private void checkInstallation(IdeTestContext context) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,11 +131,9 @@ public void testIntellijRun(String os, WireMockRuntimeInfo wmRuntimeInfo) throws
commandlet.run();

// assert
SystemInfo currentSystemInfo = this.context.getSystemInfo();
Path workspacePath = this.context.getWorkspacePath();
assertThat(commandlet.getToolBinPath().resolve("intellijtest")).hasContent(
"intellij " + currentSystemInfo.getOs() + " " + workspacePath);
checkInstallation(this.context);
assertThat(commandlet.getToolBinPath().resolve("intellijtest")).hasContent(
"intellij " + this.context.getSystemInfo().getOs() + " " + this.context.getWorkspacePath());
}

private void checkInstallation(IdeTestContext context) {
Expand Down

0 comments on commit d7dc137

Please sign in to comment.