'declare stream': make --expiration mandatory

Author: michaelklishin

src/cli.rs

@@ -566,7 +566,7 @@ fn declare_subcommands() -> [Command; 12] {
                 Arg::new("expiration")
                     .long("expiration")
                     .help("stream expiration, e.g. 12h for 12 hours, 7D for 7 days, or 1M for 1 month")
-                    .required(false)
+                    .required(true)
                     .value_parser(clap::value_parser!(String)),
             )
             .arg(

tests/streams_tests.rs

@@ -61,7 +61,7 @@ fn list_streams() -> Result<(), Box<dyn std::error::Error>> {
 
     // list streams in vhost 1
     run_succeeds(["-V", vh1, "list", "queues"])
-        .stdout(predicate::str::contains(s1).and(predicate::str::contains("new_stream2").not()));
+        .stdout(predicate::str::contains(s1).and(predicate::str::contains("random_stream").not()));
 
     // delete the stream in vhost 1
     run_succeeds(["-V", vh1, "delete", "stream", "--name", s1]);