(java): implement environment variable scanning in java sdk

generators/java/sdk/CHANGELOG.md

@@ -5,7 +5,7 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.7.0-rc0] - 2024-02-04
+## [0.7.0-rc1] - 2024-02-04
 - Chore: Bump intermediate representation to v31
 - Feature: The SDK generator now supports idempotency headers. Users 
   will be able to specify the idempotency headers in RequestOptions.
@@ -18,6 +18,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
     .idempotencyKey("...")
     .build());
   ```
+- Feature: The SDK generator now supports scanning API credentials
+  via environment varaibles. 
+  ```java
+  Imdb imdb = Imdb.builder()
+    .apiKey("...") // defaults to System.getenv("IMDB_API_KEY")
+    .build();
+  ```
 
 ## [0.6.1] - 2024-02-03
 

generators/java/sdk/VERSION

@@ -1 +1 @@
-0.7.0-rc0
+0.7.0-rc1

packages/seed/src/cli.ts

@@ -45,6 +45,11 @@ function addTestCommand(cli: Argv) {
                     demandOption: false,
                     description: "Runs on all fixtures if not provided"
                 })
+                .option("keepDocker", {
+                    type: "boolean",
+                    demandOption: false,
+                    description: "Keeps the docker container after the tests are finished"
+                })
                 .option("update", {
                     type: "boolean",
                     alias: "u",
@@ -94,7 +99,8 @@ function addTestCommand(cli: Argv) {
                     language: workspace.workspaceConfig.language,
                     docker: parsedDockerImage,
                     logLevel: argv["log-level"],
-                    numDockers: argv.parallel
+                    numDockers: argv.parallel,
+                    keepDocker: argv.keepDocker
                 });
             } else {
                 await testWorkspaceFixtures({
@@ -106,7 +112,8 @@ function addTestCommand(cli: Argv) {
                     scripts: workspace.workspaceConfig.scripts,
                     logLevel: argv["log-level"],
                     numDockers: argv.parallel,
-                    taskContextFactory
+                    taskContextFactory,
+                    keepDocker: argv.keepDocker
                 });
             }
         }

packages/seed/src/commands/test/runDockerForWorkspace.ts

@@ -23,7 +23,8 @@ export async function runDockerForWorkspace({
     irVersion,
     outputVersion,
     outputMode,
-    fixtureName
+    fixtureName,
+    keepDocker
 }: {
     absolutePathToOutput: AbsoluteFilePath;
     docker: ParsedDockerName;
@@ -35,6 +36,7 @@ export async function runDockerForWorkspace({
     outputVersion?: string;
     outputMode: OutputMode;
     fixtureName: string;
+    keepDocker: boolean | undefined;
 }): Promise<void> {
     const generatorGroup: GeneratorGroup = {
         groupName: "test",
@@ -56,7 +58,7 @@ export async function runDockerForWorkspace({
         organization: DUMMY_ORGANIZATION,
         workspace,
         generatorGroup,
-        keepDocker: true,
+        keepDocker: keepDocker ?? false,
         context: taskContext,
         irVersionOverride: irVersion,
         outputVersionOverride: outputVersion

packages/seed/src/commands/test/testCustomFixture.ts

@@ -15,7 +15,8 @@ export async function testCustomFixture({
     language,
     docker,
     logLevel,
-    numDockers
+    numDockers,
+    keepDocker
 }: {
     pathToFixture: AbsoluteFilePath;
     workspace: SeedWorkspace;
@@ -24,6 +25,7 @@ export async function testCustomFixture({
     docker: ParsedDockerName;
     logLevel: LogLevel;
     numDockers: number;
+    keepDocker: boolean | undefined;
 }): Promise<void> {
     const lock = new Semaphore(numDockers);
     const outputDir = await tmp.dir();
@@ -46,7 +48,8 @@ export async function testCustomFixture({
         outputDir: absolutePathToOutput,
         outputMode: "github",
         outputFolder: "custom",
-        id: "custom"
+        id: "custom",
+        keepDocker
     });
 
     if (result.type === "failure") {

packages/seed/src/commands/test/testWorkspaceFixtures.ts

@@ -44,7 +44,8 @@ export async function testWorkspaceFixtures({
     docker,
     scripts,
     taskContextFactory,
-    numDockers
+    numDockers,
+    keepDocker
 }: {
     workspace: SeedWorkspace;
     irVersion: string | undefined;
@@ -55,6 +56,7 @@ export async function testWorkspaceFixtures({
     logLevel: LogLevel;
     taskContextFactory: TaskContextFactory;
     numDockers: number;
+    keepDocker: boolean | undefined;
 }): Promise<void> {
     const lock = new Semaphore(numDockers);
 
@@ -94,7 +96,8 @@ export async function testWorkspaceFixtures({
                             RelativeFilePath.of(fixtureConfigInstance.outputFolder)
                         ),
                         outputMode: fixtureConfigInstance.outputMode ?? workspace.workspaceConfig.defaultOutputMode,
-                        outputFolder: fixtureConfigInstance.outputFolder
+                        outputFolder: fixtureConfigInstance.outputFolder,
+                        keepDocker
                     })
                 );
             }
@@ -114,7 +117,8 @@ export async function testWorkspaceFixtures({
                     taskContext: taskContextFactory.create(`${workspace.workspaceName}:${fixture}`),
                     outputDir: join(workspace.absolutePathToWorkspace, RelativeFilePath.of(fixture)),
                     outputMode: workspace.workspaceConfig.defaultOutputMode,
-                    outputFolder: fixture
+                    outputFolder: fixture,
+                    keepDocker
                 })
             );
         }
@@ -175,7 +179,8 @@ export async function acquireLocksAndRunTest({
     outputDir,
     absolutePathToWorkspace,
     outputMode,
-    outputFolder
+    outputFolder,
+    keepDocker
 }: {
     id: string;
     lock: Semaphore;
@@ -191,6 +196,7 @@ export async function acquireLocksAndRunTest({
     absolutePathToWorkspace: AbsoluteFilePath;
     outputMode: OutputMode;
     outputFolder: string;
+    keepDocker: boolean | undefined;
 }): Promise<TestResult> {
     taskContext.logger.debug("Acquiring lock...");
     await lock.acquire();
@@ -208,7 +214,8 @@ export async function acquireLocksAndRunTest({
         outputDir,
         absolutePathToWorkspace,
         outputMode,
-        outputFolder
+        outputFolder,
+        keepDocker
     });
     taskContext.logger.debug("Releasing lock...");
     lock.release();
@@ -228,7 +235,8 @@ async function testWithWriteToDisk({
     outputDir,
     absolutePathToWorkspace,
     outputMode,
-    outputFolder
+    outputFolder,
+    keepDocker
 }: {
     id: string;
     fixture: string;
@@ -243,6 +251,7 @@ async function testWithWriteToDisk({
     absolutePathToWorkspace: AbsoluteFilePath;
     outputMode: OutputMode;
     outputFolder: string;
+    keepDocker: boolean | undefined;
 }): Promise<TestResult> {
     try {
         const workspace = await loadAPIWorkspace({
@@ -275,7 +284,8 @@ async function testWithWriteToDisk({
             irVersion,
             outputVersion,
             outputMode,
-            fixtureName: fixture
+            fixtureName: fixture,
+            keepDocker
         });
         for (const script of scripts ?? []) {
             taskContext.logger.info(`Running script on ${fixture}`);