From 967de64e19c05a2f14e7c97da000c1f3b8ccc4ed Mon Sep 17 00:00:00 2001 From: DanConwayDev Date: Tue, 24 Sep 2024 19:22:23 +0100 Subject: test: restructure ngit integration tests so that each file is ran as a seperate crate this makes it easier to see which tests are causing other tests to fail as they are in a smaller group. --- tests/ngit/login.rs | 1186 --------------------------------------------------- 1 file changed, 1186 deletions(-) delete mode 100644 tests/ngit/login.rs (limited to 'tests/ngit/login.rs') diff --git a/tests/ngit/login.rs b/tests/ngit/login.rs deleted file mode 100644 index 477b25b..0000000 --- a/tests/ngit/login.rs +++ /dev/null @@ -1,1186 +0,0 @@ -use anyhow::Result; -use git::GitTestRepo; -use serial_test::serial; -use test_utils::*; - -static EXPECTED_NSEC_PROMPT: &str = "login with nsec / bunker url / nostr address"; -static EXPECTED_LOCAL_REPOSITORY_PROMPT: &str = "just for this repository?"; -static EXPECTED_REQUIRE_PASSWORD_PROMPT: &str = "require password?"; -static EXPECTED_SET_PASSWORD_PROMPT: &str = "encrypt with password"; -static EXPECTED_SET_PASSWORD_CONFIRM_PROMPT: &str = "confirm password"; - -fn standard_first_time_login_encrypting_nsec() -> Result { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login", "--offline"]); - - p.expect_input_eventually(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_password(EXPECTED_SET_PASSWORD_PROMPT)? - .with_confirmation(EXPECTED_SET_PASSWORD_CONFIRM_PROMPT)? - .succeeds_with(TEST_PASSWORD)?; - - p.expect_end_eventually()?; - Ok(p) -} - -fn expect_qr_prompt_opt_for_other_methods(p: &mut CliTester) -> Result<()> { - p.expect_eventually("scan QR or paste into remote signer")?; - p.expect_eventually("\r\n")?; - p.expect_eventually("login with nsec / bunker url / nostr address instead")?; - p.expect_eventually("\r\n")?; - p.send_line("")?; - // p.expect_eventually("\r\r\r\r\r\r\r\r\r\r\r\r\r\r\r\r")?; - // p.expect_eventually("\r\r\r\r\r\r\r\r\r\r\r\r\r")?; - - Ok(()) -} -mod with_relays { - use anyhow::Ok; - use futures::join; - use test_utils::relay::{shutdown_relay, ListenerReqFunc, Relay}; - - use super::*; - - mod when_user_relay_list_aligns_with_fallback_relays { - // this simplifies testing - use super::*; - - mod when_first_time_login { - use super::*; - - // falls_back_to_fallback_relays - this is implict in the tests - - mod dislays_logged_in_with_correct_name { - - use super::*; - - async fn run_test_displays_correct_name( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, relay_listener2), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login"]); - expect_qr_prompt_opt_for_other_methods(&mut p)?; - p.expect_input_eventually(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(false))?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect("searching for profile...\r\n")?; - - p.expect_end_with("logged in as fred\r\n")?; - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - - async fn run_test_displays_fallback_to_npub( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, relay_listener2), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login"]); - - expect_qr_prompt_opt_for_other_methods(&mut p)?; - p.expect_input_eventually(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(false))?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect("searching for profile...\r\n")?; - - p.expect("cannot extract account name from account metadata...\r\n")?; - - p.expect_end_with( - format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str(), - )?; - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - - #[tokio::test] - #[serial] - async fn when_latest_metadata_and_relay_list_on_all_relays() -> Result<()> { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - ) - .await - } - - mod poorly_quality_metadata_event { - use super::*; - - #[tokio::test] - #[serial] - async fn when_metadata_contains_only_display_name() -> Result<()> { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - nostr::event::EventBuilder::metadata( - &nostr::Metadata::new().display_name("fred"), - ) - .to_event(&TEST_KEY_1_KEYS) - .unwrap(), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - None, - ) - .await - } - - #[tokio::test] - #[serial] - async fn when_metadata_contains_only_displayname() -> Result<()> { - println!( - "displayName: {}", - nostr::Metadata::new() - .custom_field("displayName", "fred") - .custom - .get("displayName") - .unwrap() - ); - println!( - "name: {}", - nostr::Metadata::new().name("fred").name.unwrap() - ); - - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - nostr::event::EventBuilder::metadata( - &nostr::Metadata::new() - .custom_field("displayName", "fred"), - ) - .to_event(&TEST_KEY_1_KEYS) - .unwrap(), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - None, - ) - .await - } - - #[tokio::test] - #[serial] - async fn displays_npub_when_metadata_contains_no_name_displayname_or_display_name() - -> Result<()> { - run_test_displays_fallback_to_npub( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - nostr::event::EventBuilder::metadata( - &nostr::Metadata::new().about("other info in metadata"), - ) - .to_event(&TEST_KEY_1_KEYS) - .unwrap(), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - None, - ) - .await - } - } - - #[tokio::test] - #[serial] - async fn when_latest_metadata_and_relay_list_on_some_relays_but_others_have_none() - -> Result<()> { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - None, - ) - .await - } - - #[tokio::test] - #[serial] - async fn when_latest_metadata_only_on_relay_and_relay_list_on_another() -> Result<()> - { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![generate_test_key_1_metadata_event("fred")], - )?; - Ok(()) - }), - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![generate_test_key_1_relay_list_event_same_as_fallback()], - )?; - Ok(()) - }), - ) - .await - } - - #[tokio::test] - #[serial] - async fn when_some_relays_return_old_metadata_event() -> Result<()> { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![generate_test_key_1_metadata_event_old("fred old")], - )?; - Ok(()) - }), - ) - .await - } - - #[tokio::test] - #[serial] - async fn when_some_relays_return_other_users_metadata() -> Result<()> { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![generate_test_key_2_metadata_event("carole")], - )?; - Ok(()) - }), - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event_old("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - ) - .await - } - - #[tokio::test] - #[serial] - async fn when_some_relays_return_other_event_kinds() -> Result<()> { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - let event = generate_test_key_1_kind_event(nostr::Kind::TextNote); - relay.respond_events( - client_id, - &subscription_id, - &vec![make_event_old_or_change_user(event, &TEST_KEY_1_KEYS, 0)], - )?; - Ok(()) - }), - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event_old("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - ) - .await - } - - mod when_specifying_command_line_nsec_only { - use super::*; - - #[tokio::test] - #[serial] - async fn displays_correct_name() -> Result<()> { - run_test_when_specifying_command_line_nsec_only_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - None, - ) - .await - } - async fn run_test_when_specifying_command_line_nsec_only_displays_correct_name( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, relay_listener2), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - ["login", "--nsec", TEST_KEY_1_NSEC], - ); - - p.expect("saved login details to local git config\r\n")?; - - p.expect("searching for profile...\r\n")?; - - p.expect_end_with("logged in as fred\r\n")?; - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - } - mod when_specifying_command_line_password_only { - use super::*; - - #[tokio::test] - #[serial] - async fn displays_correct_name() -> Result<()> { - run_test_when_specifying_command_line_password_only_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - None, - ) - .await - } - async fn run_test_when_specifying_command_line_password_only_displays_correct_name( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, relay_listener2), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - CliTester::new_from_dir( - &test_repo.dir, - [ - "login", - "--offline", - "--nsec", - TEST_KEY_1_NSEC, - "--password", - TEST_PASSWORD, - ], - ) - .expect_end_eventually()?; - - let mut p = CliTester::new_from_dir( - &test_repo.dir, - ["login", "--password", TEST_PASSWORD], - ); - - p.expect("searching for profile...\r\n")?; - - p.expect_end_with("logged in as fred\r\n")?; - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - } - - mod when_specifying_command_line_nsec_and_password { - use super::*; - - #[tokio::test] - #[serial] - async fn displays_correct_name() -> Result<()> { - run_test_when_specifying_command_line_nsec_and_password_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }), - None, - ).await - } - async fn run_test_when_specifying_command_line_nsec_and_password_displays_correct_name( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, relay_listener2), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - [ - "login", - "--nsec", - TEST_KEY_1_NSEC, - "--password", - TEST_PASSWORD, - ], - ); - - p.expect("saved login details to local git config\r\n")?; - - p.expect("searching for profile...\r\n")?; - - p.expect_end_with("logged in as fred\r\n")?; - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - } - } - - mod when_no_metadata_found { - use super::*; - - #[tokio::test] - #[serial] - async fn warm_user_and_displays_npub() -> Result<()> { - run_test_when_no_metadata_found_warns_user_and_uses_npub(None, None).await - } - - async fn run_test_when_no_metadata_found_warns_user_and_uses_npub( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, relay_listener2), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login"]); - - expect_qr_prompt_opt_for_other_methods(&mut p)?; - p.expect_input_eventually(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(false))?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect("searching for profile...\r\n")?; - - p.expect("cannot find profile...\r\n")?; - - p.expect_end_with(format!("logged in as {TEST_KEY_1_NPUB}\r\n").as_str())?; - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - } - - mod when_metadata_but_no_relay_list_found { - use super::*; - - #[tokio::test] - #[serial] - async fn warm_user_and_displays_name() -> Result<()> { - run_test_when_no_relay_list_found_warns_user_and_uses_npub( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![generate_test_key_1_metadata_event("fred")], - )?; - Ok(()) - }), - None, - ) - .await - } - - async fn run_test_when_no_relay_list_found_warns_user_and_uses_npub( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, relay_listener2), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login"]); - - expect_qr_prompt_opt_for_other_methods(&mut p)?; - p.expect_input_eventually(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(false))?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect("searching for profile...\r\n")?; - - p.expect("cannot find your relay list. consider using another nostr client to create one to enhance your nostr experience.\r\n")?; - - p.expect_end_with("logged in as fred\r\n")?; - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - } - } - - mod when_second_time_login_and_details_already_fetched { - use super::*; - - mod uses_cache_and_stores_and_retrieves_ncryptsec_from_local_git_config { - use super::*; - - #[tokio::test] - #[serial] - async fn dislays_logged_in_with_correct_name() -> Result<()> { - run_test_dislays_logged_in_with_correct_name(Some( - &|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event_same_as_fallback(), - ], - )?; - Ok(()) - }, - )) - .await - } - async fn run_test_dislays_logged_in_with_correct_name( - relay_listener: Option>, - ) -> Result<()> { - let (mut r51, mut r52) = ( - Relay::new(8051, None, relay_listener), - Relay::new(8052, None, None), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - [ - "login", - "--nsec", - TEST_KEY_1_NSEC, - "--password", - TEST_PASSWORD, - ], - ); - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_eventually_with("logged in as fred\r\n")?; - - for p in [51, 52] { - shutdown_relay(8000 + p)?; - } - - let mut p = CliTester::new_from_dir( - &test_repo.dir, - ["login", "--password", TEST_PASSWORD], - ); - - p.expect_end_eventually_with("logged in as fred\r\n")?; - - Ok(()) - }); - - // launch relay - let _ = join!(r51.listen_until_close(), r52.listen_until_close(),); - - cli_tester_handle.join().unwrap()?; - - Ok(()) - } - } - } - } - mod when_user_relay_list_contains_write_relays_not_in_fallback_list { - use super::*; - mod when_latest_metadata_not_on_fallback_relays_only_on_relays_in_user_list { - use super::*; - async fn run_test_displays_correct_name( - relay_listener1: Option>, - relay_listener2: Option>, - ) -> Result<()> { - let (mut r51, mut r52, mut r53, mut r55) = ( - Relay::new(8051, None, relay_listener1), - Relay::new(8052, None, None), - Relay::new(8053, None, relay_listener2), - Relay::new(8055, None, None), - ); - - let cli_tester_handle = std::thread::spawn(move || -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login"]); - - expect_qr_prompt_opt_for_other_methods(&mut p)?; - p.expect_input_eventually(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(false))?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect("searching for profile...\r\n")?; - - p.expect_end_with("logged in as fred\r\n")?; - for p in [51, 52, 53, 55] { - shutdown_relay(8000 + p)?; - } - Ok(()) - }); - - // launch relay - let _ = join!( - r51.listen_until_close(), - r52.listen_until_close(), - r53.listen_until_close(), - r55.listen_until_close(), - ); - - cli_tester_handle.join().unwrap()?; - Ok(()) - } - - /// this also tests that additional relays are queried - #[tokio::test] - #[serial] - async fn displays_correct_name() -> Result<()> { - run_test_displays_correct_name( - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event_old("Fred"), - generate_test_key_1_relay_list_event(), - ], - )?; - Ok(()) - }), - Some(&|relay, client_id, subscription_id, _| -> Result<()> { - relay.respond_events( - client_id, - &subscription_id, - &vec![ - generate_test_key_1_metadata_event("fred"), - generate_test_key_1_relay_list_event(), - ], - )?; - Ok(()) - }), - ) - .await - } - } - } -} - -/// using the offline flag simplifies the test. relay interaction is tested -/// seperately -mod with_offline_flag { - use super::*; - mod when_first_time_login { - use super::*; - - #[test] - fn prompts_for_nsec_and_password() -> Result<()> { - standard_first_time_login_encrypting_nsec()?; - Ok(()) - } - - #[test] - fn succeeds_with_text_logged_in_as_npub() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login", "--offline"]); - - p.expect_input(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_password(EXPECTED_SET_PASSWORD_PROMPT)? - .with_confirmation(EXPECTED_SET_PASSWORD_CONFIRM_PROMPT)? - .succeeds_with(TEST_PASSWORD)?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - } - - #[test] - fn succeeds_with_hex_secret_key_in_place_of_nsec() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login", "--offline"]); - - p.expect_input(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_SK_HEX)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_password(EXPECTED_SET_PASSWORD_PROMPT)? - .with_confirmation(EXPECTED_SET_PASSWORD_CONFIRM_PROMPT)? - .succeeds_with(TEST_PASSWORD)?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - } - - mod when_invalid_nsec { - use super::*; - - #[test] - fn prompts_for_nsec_until_valid() -> Result<()> { - let invalid_nsec_response = - "invalid. try again with nostr address / bunker uri / nsec"; - - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir(&test_repo.dir, ["login", "--offline"]); - - p.expect_input(EXPECTED_NSEC_PROMPT)? - // this behaviour is intentional. rejecting the response with dialoguer - // hides the original input from the user so they - // cannot see the mistake they made. - .succeeds_with(TEST_INVALID_NSEC)?; - - p.expect_input(invalid_nsec_response)? - .succeeds_with(TEST_INVALID_NSEC)?; - - p.expect_input(invalid_nsec_response)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_password(EXPECTED_SET_PASSWORD_PROMPT)? - .with_confirmation(EXPECTED_SET_PASSWORD_CONFIRM_PROMPT)? - .succeeds_with(TEST_PASSWORD)?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - } - } - } - - mod when_called_with_nsec_parameter_only { - use super::*; - - #[test] - fn valid_nsec_param_succeeds_without_prompts() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - ["login", "--offline", "--nsec", TEST_KEY_1_NSEC], - ); - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - } - - #[test] - fn forgets_identity() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - ["login", "--offline", "--nsec", TEST_KEY_1_NSEC], - ); - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str())?; - - p = CliTester::new_from_dir(&test_repo.dir, ["login", "--offline"]); - - p.expect_input(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.exit() - } - - mod when_logging_in_as_different_nsec { - use super::*; - - #[test] - fn valid_nsec_param_succeeds_without_prompts_and_logs_in() -> Result<()> { - standard_first_time_login_encrypting_nsec()?.exit()?; - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - ["login", "--offline", "--nsec", TEST_KEY_2_NSEC], - ); - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_2_NPUB).as_str()) - } - } - #[test] - fn invalid_nsec_param_fails_without_prompts() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - ["login", "--offline", "--nsec", TEST_INVALID_NSEC], - ); - - p.expect_end_with( - "Error: invalid nsec parameter\r\n\r\nCaused by:\r\n Invalid secret key\r\n", - ) - } - } - - mod when_called_with_nsec_and_password_parameter { - use super::*; - - #[test] - fn valid_nsec_param_succeeds_without_prompts() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - [ - "login", - "--offline", - "--nsec", - TEST_KEY_1_NSEC, - "--password", - TEST_PASSWORD, - ], - ); - p.expect("saved login details to local git config\r\n")?; - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - } - - #[test] - fn parameters_can_be_called_globally() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - [ - "--nsec", - TEST_KEY_1_NSEC, - "--password", - TEST_PASSWORD, - "login", - "--offline", - ], - ); - p.expect("saved login details to local git config\r\n")?; - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - } - - mod when_logging_in_as_different_nsec { - use super::*; - - #[test] - fn valid_nsec_param_succeeds_without_prompts_and_logs_in() -> Result<()> { - standard_first_time_login_encrypting_nsec()?.exit()?; - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - [ - "login", - "--offline", - "--nsec", - TEST_KEY_2_NSEC, - "--password", - TEST_PASSWORD, - ], - ); - p.expect("saved login details to local git config\r\n")?; - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_2_NPUB).as_str()) - } - } - - mod when_provided_with_new_password { - use super::*; - - #[test] - fn password_changes() -> Result<()> { - standard_first_time_login_encrypting_nsec()?.exit()?; - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - [ - "login", - "--offline", - "--nsec", - TEST_KEY_1_NSEC, - "--password", - TEST_INVALID_PASSWORD, - ], - ); - p.expect("saved login details to local git config\r\n")?; - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str())?; - - CliTester::new_from_dir( - &test_repo.dir, - ["--password", TEST_INVALID_PASSWORD, "login", "--offline"], - ) - .expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - } - } - - #[test] - fn invalid_nsec_param_fails_without_prompts() -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = CliTester::new_from_dir( - &test_repo.dir, - [ - "login", - "--offline", - "--nsec", - TEST_INVALID_NSEC, - "--password", - TEST_PASSWORD, - ], - ); - p.expect_end_with( - "Error: invalid nsec parameter\r\n\r\nCaused by:\r\n Invalid secret key\r\n", - ) - } - } - - mod when_weak_password { - use super::*; - - #[test] - #[serial] - // combined into a single test as it is computationally expensive to run - fn warns_it_might_take_a_few_seconds_then_succeeds_then_second_login_prompts_for_password_then_warns_again_then_succeeds() - -> Result<()> { - let test_repo = GitTestRepo::default(); - let mut p = - CliTester::new_with_timeout_from_dir(15000, &test_repo.dir, ["login", "--offline"]); - - p.expect_input(EXPECTED_NSEC_PROMPT)? - .succeeds_with(TEST_KEY_1_NSEC)?; - - p.expect_confirm(EXPECTED_LOCAL_REPOSITORY_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_confirm(EXPECTED_REQUIRE_PASSWORD_PROMPT, Some(false))? - .succeeds_with(Some(true))?; - - p.expect_password(EXPECTED_SET_PASSWORD_PROMPT)? - .with_confirmation(EXPECTED_SET_PASSWORD_CONFIRM_PROMPT)? - .succeeds_with(TEST_WEAK_PASSWORD)?; - - p.expect("this may take a few seconds...\r\n")?; - - p.expect("saved login details to local git config\r\n")?; - - p.expect_end_with(format!("logged in as {}\r\n", TEST_KEY_1_NPUB).as_str()) - - // commented out as 'login' command now assumes you want to - // login as a new user - // p = CliTester::new_with_timeout(10000, ["login", - // "--offline"]); - - // p.expect(format!("login as {}\r\n", - // TEST_KEY_1_NPUB).as_str())? - // .expect_password(EXPECTED_PASSWORD_PROMPT)? - // .succeeds_with(TEST_WEAK_PASSWORD)?; - - // p.expect("this may take a few seconds...\r\n")?; - - // p.expect_end_with(format!("logged in as {}\r\n", - // TEST_KEY_1_NPUB).as_str()) - } - } -} -- cgit v1.2.3