Method run_with_verifier

atomic-cli/src/commands/org/set.rs:91–127

Inner runner that takes the slug-verification step as a parameter. Splitting this out lets tests inject a deterministic verifier (succeed / fail) without making a real network call, so we can assert the load-bearing invariant: **when verification fails, the config file is not modified**.

(&self, verify: F)

    /// assert the load-bearing invariant: **when verification fails, the
    /// config file is not modified**.
    fn run_with_verifier<F>(&self, verify: F) -> CliResult<()>
    where
        F: FnOnce(&str) -> CliResult<()>,
    {
        if self.slug.is_empty() {
            return Err(CliError::InvalidArgument {
                message: "Organization slug cannot be empty.".to_string(),
            });
        }

        // Validate against the server first so a bad slug fails fast
        // rather than self-locking subsequent commands. The `?` here is
        // load-bearing: if it returns Err, config is not touched.
        if !self.no_verify {
            verify(&self.slug)?;
        }

        let mut config = GlobalConfig::load().map_err(|e| {
            CliError::Internal(anyhow::anyhow!("Failed to load global config: {e}"))
        })?;

        let previous = config.server.default_org.clone();
        config.server.default_org = Some(self.slug.clone());
        config.save().map_err(|e| {
            CliError::Internal(anyhow::anyhow!("Failed to save global config: {e}"))
        })?;

        print_success(&format!("Default organization set to: {}", self.slug));

        if let Some(prev) = &previous {
            if prev != &self.slug {
                print_hint(&format!("Previous default was: {prev}"));
            }
        }

        Ok(())
    }
}

/// Confirm the slug resolves to an org on the server.