Skip to content

Commit

Permalink
[dagster-pipes-test] make job name flag consistent with other flags
Browse files Browse the repository at this point in the history
Signed-off-by: Thanabodee Charoenpiriyakij <[email protected]>
  • Loading branch information
wingyplus committed Dec 25, 2024
1 parent e04e213 commit 88498c8
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ public class MainTest implements Runnable {
private boolean env = false;

@CommandLine.Option(
names = {"--jobName"},
names = {"--job-name"},
description = "Provide value of 'jobName' for testing"
)
private String jobName;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ struct Cli {
require_equals = false,
)]
env: bool,
#[arg(long = "jobName")]
#[arg(long = "job-name")]
job_name: Option<String>,
#[arg(long)]
extras: Option<String>,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ def my_asset(
args = self.BASE_ARGS + [
"--env",
f"--extras={str(extras_path)}",
f"--jobName={job_name}",
f"--job-name={job_name}",
]

return pipes_subprocess_client.run(
Expand Down Expand Up @@ -227,7 +227,7 @@ def my_asset(
"--full",
"--env",
f"--extras={metadata_path}",
f"--jobName={job_name}",
f"--job-name={job_name}",
]

invocation_result = pipes_subprocess_client.run(
Expand Down Expand Up @@ -419,7 +419,7 @@ def my_asset(
args = self.BASE_ARGS + [
"--full",
"--env",
f"--jobName={job_name}",
f"--job-name={job_name}",
"--custom-payload-path",
str(custom_payload_path),
]
Expand Down Expand Up @@ -495,7 +495,7 @@ def my_asset(
args = self.BASE_ARGS + [
"--full",
"--env",
f"--jobName={job_name}",
f"--job-name={job_name}",
"--report-asset-materialization",
str(asset_materialization_path),
]
Expand Down Expand Up @@ -592,7 +592,7 @@ def my_asset(
args = self.BASE_ARGS + [
"--full",
"--env",
f"--jobName={job_name}",
f"--job-name={job_name}",
"--report-asset-check",
str(report_asset_check_path),
]
Expand Down

0 comments on commit 88498c8

Please sign in to comment.