Skip to content

Commit

Permalink
Added SYNK_API
Browse files Browse the repository at this point in the history
  • Loading branch information
bartenbach committed Feb 23, 2022
1 parent d8bcb3a commit 58673fa
Show file tree
Hide file tree
Showing 4 changed files with 17 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ static ProcessBuilder asProcessBuilder(
String cliExecutablePath,
Command command,
Optional<String> apiToken,
Optional<String> apiUrl,
List<String> args,
boolean color
) {
Expand All @@ -39,6 +40,7 @@ static ProcessBuilder asProcessBuilder(
}

apiToken.ifPresent((t) -> pb.environment().put("SNYK_TOKEN", t));
apiUrl.ifPresent((t) -> pb.environment().put("SNYK_API", t));

return pb;
}
Expand Down
7 changes: 7 additions & 0 deletions src/main/java/io/snyk/snyk_maven_plugin/goal/SnykMojo.java
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ public abstract class SnykMojo extends ComposedMojo {
@Parameter
private String apiToken;

@Parameter
private String apiUrl;

@Parameter
private List<String> args;

Expand Down Expand Up @@ -63,6 +66,10 @@ public Optional<String> getApiToken() {
return Optional.ofNullable(apiToken);
}

public Optional<String> getApiUrl() {
return Optional.ofNullable(apiUrl);
}

public Optional<File> getExecutable() {
return Optional.ofNullable(cli)
.map(cli -> cli.executable);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ private int executeCommand() throws MojoExecutionException {
executablePath,
mojo.getCommand(),
mojo.getApiToken(),
mojo.getApiUrl(),
mojo.getArguments(),
mojo.supportsColor()
);
Expand All @@ -67,6 +68,7 @@ private String getVersion(String executablePath) {
executablePath,
Command.VERSION,
Optional.empty(),
Optional.empty(),
emptyList(),
false
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ public void shouldIncludePathCommandNameIntegrationName() {
"/path/to/cli",
Command.TEST,
Optional.empty(),
Optional.empty(),
emptyList(),
false
);
Expand All @@ -38,6 +39,7 @@ public void shouldIncludeTrimmedArguments() {
"/path/to/cli",
Command.TEST,
Optional.empty(),
Optional.empty(),
asList(
"--print-deps",
" --all-projects ",
Expand Down Expand Up @@ -65,6 +67,7 @@ public void shouldNotChangeIntegrationName() {
"/path/to/cli",
Command.TEST,
Optional.empty(),
Optional.empty(),
singletonList("--integration-name=this-is-not-ok"),
false
);
Expand All @@ -85,6 +88,7 @@ public void shouldNotModifyEnvironmentByDefault() {
"/path/to/cli",
Command.TEST,
Optional.empty(),
Optional.empty(),
emptyList(),
false
);
Expand All @@ -100,6 +104,7 @@ public void shouldIncludeAPIToken() {
"/path/to/cli",
Command.TEST,
Optional.of("fake-token"),
Optional.empty(),
emptyList(),
false
);
Expand All @@ -116,6 +121,7 @@ public void shouldForceColorWhenEnabled() {
"/path/to/cli",
Command.TEST,
Optional.empty(),
Optional.empty(),
emptyList(),
true
);
Expand Down

0 comments on commit 58673fa

Please sign in to comment.