Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
16 changes: 8 additions & 8 deletions rust/src/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -132,9 +132,9 @@ mod tests {

/// Regression test for a real bug found while wiring feature-flagging up
/// properly: with no `min_stage` override anywhere, cli-engine's own
/// default (`Stage::Ga`) hides `hosting` and the Developer Platform
/// namespace entirely. Guards that the *global* default stays `Ga` per
/// product decision — i.e. these stay hidden absent an environment override.
/// default (`Stage::Ga`) hides `hosting`. Guards that the *global* default
/// stays `Ga` per product decision — i.e. beta and experimental modules
/// stay hidden absent an environment override.
#[tokio::test]
async fn beta_and_experimental_modules_stay_hidden_at_the_default_min_stage() {
let cli = Cli::new(
Expand All @@ -151,16 +151,16 @@ mod tests {
}

#[tokio::test]
async fn platform_namespace_is_hidden_at_the_default_min_stage() {
async fn platform_namespace_is_visible_at_the_default_min_stage() {
let cli = Cli::new(
CliConfig::new("gddy", "GoDaddy developer CLI", "gddy")
.with_min_stage(Stage::Ga)
.with_modules(super::all_modules()),
);
let output = cli.run(["gddy", "platform", "--help"]).await;
assert_ne!(
assert_eq!(
output.exit_code, 0,
"platform should stay hidden at the Ga default: {}",
"platform should be visible at the Ga default: {}",
output.rendered
);
}
Expand Down Expand Up @@ -193,7 +193,7 @@ mod tests {
}

#[tokio::test]
async fn platform_guides_are_discoverable_even_when_the_namespace_is_hidden() {
async fn platform_guides_are_discoverable() {
let cli = Cli::new(
CliConfig::new("gddy", "GoDaddy developer CLI", "gddy")
.with_min_stage(Stage::Ga)
Expand Down Expand Up @@ -223,7 +223,7 @@ mod tests {
async fn platform_namespace_exposes_the_gpa_command_tree() {
let cli = Cli::new(
CliConfig::new("gddy", "GoDaddy developer CLI", "gddy")
.with_min_stage(Stage::Experimental)
.with_min_stage(Stage::Ga)
.with_modules(super::all_modules()),
);

Expand Down
10 changes: 4 additions & 6 deletions rust/src/platform/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,18 +4,16 @@
//! catalog, and webhook implementations retain their existing handlers and
//! clients, while `platform` provides their unambiguous common CLI path.

use cli_engine::{GroupSpec, Module, RuntimeGroupSpec, Stage};
use cli_engine::{GroupSpec, Module, RuntimeGroupSpec};

pub fn module() -> Module {
Module::new("Platform", |_ctx| {
RuntimeGroupSpec::new(
GroupSpec::new("platform", "Build and manage GoDaddy Platform integrations")
.with_long(
"Build integrations for the GoDaddy Developer Platform. Manage your app, \
GroupSpec::new("platform", "Build and manage GoDaddy Platform integrations").with_long(
"Build integrations for the GoDaddy Developer Platform. Manage your app, \
browse action contracts, and inspect webhook event types from one \
namespace. Use `gddy platform app init` to create an app.",
)
.with_feature_flag("platform", Stage::Experimental),
),
)
.with_group(crate::application::group())
.with_group(crate::actions_catalog::group())
Expand Down
Loading