Skip to content

Hide compliance promo output in dry-run and non-interactive runs #53

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bencehezso merged 3 commits into from
Jul 24, 2026
Merged

Hide compliance promo output in dry-run and non-interactive runs #53

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
14bf46a
Guard compliance promo output for dry-run and non-interactive
bencehezso Jul 24, 2026
1c6aa60
Guard compliance promo output for dry-run and non-interactive - fix t…
bencehezso Jul 24, 2026
764767e
Merge branch 'main' into task/guard-compliance-promo-output
bencehezso Jul 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 43 additions & 16 deletions main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,13 @@ def handle_aws(args):
f"Exit Assessment {datetime.now().strftime('%Y%m%d_%H%M%S')}"
)

run_assessment(config, "aws", dry_run=args.dry_run, egress=args.egress)
run_assessment(
config,
"aws",
dry_run=args.dry_run,
non_interactive=args.non_interactive,
egress=args.egress,
)
return

if args.non_interactive:
Expand All @@ -196,7 +202,13 @@ def handle_aws(args):
config = build_config(
cloud_provider, exit_strategy, assessment_type, provider_details, args
)
run_assessment(config, "aws", dry_run=args.dry_run, egress=args.egress)
run_assessment(
config,
"aws",
dry_run=args.dry_run,
non_interactive=args.non_interactive,
egress=args.egress,
)


def _azure_cli_credential() -> DefaultAzureCredential:
Expand Down Expand Up @@ -365,7 +377,13 @@ def handle_azure(args):
f"Exit Assessment {datetime.now().strftime('%Y%m%d_%H%M%S')}"
)

run_assessment(config, "azure", dry_run=args.dry_run, egress=args.egress)
run_assessment(
config,
"azure",
dry_run=args.dry_run,
non_interactive=args.non_interactive,
egress=args.egress,
)
return

if args.non_interactive:
Expand All @@ -386,10 +404,18 @@ def handle_azure(args):
config = build_config(
cloud_provider, exit_strategy, assessment_type, provider_details, args
)
run_assessment(config, "azure", dry_run=args.dry_run, egress=args.egress)
run_assessment(
config,
"azure",
dry_run=args.dry_run,
non_interactive=args.non_interactive,
egress=args.egress,
)


def run_assessment(config, provider_name, *, dry_run=False, egress=False):
def run_assessment(
config, provider_name, *, dry_run=False, non_interactive=False, egress=False
):
# Record the assessment start time to propagate across stages
started_at = int(time.time())

Expand Down Expand Up @@ -694,17 +720,18 @@ def run_assessment(config, provider_name, *, dry_run=False, egress=False):
egress_failed = True

# Output the report path after the separator
console.print("-------------------------------------------")
console.print(
"Need to see how a regulatory-aligned report looks (DORA / FINMA / UK PRA evidence support)?"
)
console.print(
"We can generate one from your --dry-run output and share a sample with you -- or forward this to your risk or compliance team."
)
console.print(
"Point-in-time evidence only; not a compliance certification or a complete exit strategy."
)
console.print("Contact: request_report@escapecloud.io")
if not (dry_run or non_interactive):
console.print("-------------------------------------------")
console.print(
"Need to see how a regulatory-aligned report looks (DORA / FINMA / UK PRA evidence support)?"
)
console.print(
"We can generate one from your --dry-run output and share a sample with you -- or forward this to your risk or compliance team."
)
console.print(
"Point-in-time evidence only; not a compliance certification or a complete exit strategy."
)
console.print("Contact: request_report@escapecloud.io")
console.print("-------------------------------------------")
console.print("Outputs:", style="bold")
if payload_path:
Expand Down
12 changes: 9 additions & 3 deletions tests/test_utils_and_main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -367,7 +367,9 @@ def test_dry_run_flag_passed_from_handle_aws(self):
):
main.handle_aws(_ni_aws_args(dry_run=True))

mock_run.assert_called_once_with(ANY, "aws", dry_run=True, egress=False)
mock_run.assert_called_once_with(
ANY, "aws", dry_run=True, non_interactive=True, egress=False
)


def _ni_aws_args(**kwargs):
Expand Down Expand Up @@ -765,7 +767,9 @@ def test_handle_azure_passes_egress_flag(self):
):
main.handle_azure(_ni_azure_args(egress=True))

mock_run.assert_called_once_with(ANY, "azure", dry_run=False, egress=True)
mock_run.assert_called_once_with(
ANY, "azure", dry_run=False, non_interactive=True, egress=True
)

def test_handle_aws_passes_egress_flag(self):
with (
Expand All @@ -776,7 +780,9 @@ def test_handle_aws_passes_egress_flag(self):
):
main.handle_aws(_ni_aws_args(egress=True))

mock_run.assert_called_once_with(ANY, "aws", dry_run=False, egress=True)
mock_run.assert_called_once_with(
ANY, "aws", dry_run=False, non_interactive=True, egress=True
)


class MainExitCodeTests(unittest.TestCase):
Expand Down