feat(trello): add command 'trello board list'

src/lib.rs

@@ -35,6 +35,8 @@ enum Command {
     Check,
     Import,
     #[clap(subcommand)]
+    Trello(TrelloCommand),
+    #[clap(subcommand)]
     Nextcloud(NextcloudCommand),
 }
 
@@ -62,6 +64,20 @@ enum NextcloudStackCommand {
     },
 }
 
+#[derive(Parser, Debug)]
+enum TrelloCommand {
+    #[clap(subcommand)]
+    Board(TrelloBoardCommand),
+}
+
+#[derive(Parser, Debug)]
+enum TrelloBoardCommand {
+    List {
+        #[clap(long, action = clap::ArgAction::SetTrue)]
+        dump: bool,
+    },
+}
+
 #[derive(Clone)]
 pub struct Ctx {
     pub fs: FileSystem,
@@ -116,6 +132,9 @@ pub async fn run(ctx: Ctx) -> color_eyre::Result<()> {
                 Command::Init => Err(eyre!("Config file already exists. Not overwriting it.")),
                 Command::Check => todo!("check"),
                 Command::Import => todo!("import"),
+                Command::Trello(TrelloCommand::Board(TrelloBoardCommand::List { dump })) => {
+                    trello::boards::list(ctx, dump).await
+                }
                 Command::Nextcloud(NextcloudCommand::Board(NextcloudBoardCommand::List {
                     dump,
                 })) => nextcloud::board::list(ctx, dump).await,

src/nextcloud/tests.rs

@@ -223,9 +223,9 @@ mod commands {
 }
 
 mod given {
-    use kxio::{fs::TempFileSystem, net::MockNet, print::Printer};
+    use kxio::{fs::TempFileSystem, print::Printer};
 
-    use crate::{config::TrelloConfig, s, AppConfig, FullCtx};
+    use crate::{config::TrelloConfig, s};
 
     use super::*;
 
@@ -258,15 +258,15 @@ mod given {
         }
     }
 
-    pub(crate) fn a_full_context(mock_net: MockNet, fs: TempFileSystem) -> FullCtx {
+    // pub(crate) fn a_full_context(mock_net: MockNet, fs: TempFileSystem) -> FullCtx {
-        FullCtx {
+    //     FullCtx {
-            fs: fs.as_real(),
+    //         fs: fs.as_real(),
-            net: mock_net.into(),
+    //         net: mock_net.into(),
-            prt: given::a_printer(),
+    //         prt: given::a_printer(),
-            cfg: AppConfig {
+    //         cfg: AppConfig {
-                trello: given::a_trello_config(),
+    //             trello: given::a_trello_config(),
-                nextcloud: given::a_nextcloud_config(),
+    //             nextcloud: given::a_nextcloud_config(),
-            },
+    //         },
-        }
+    //     }
-    }
+    // }
 }

src/tests/mod.rs

@@ -1,17 +1,21 @@
 //
+use std::collections::HashMap;
 
 // type TestResult = Result<(), Box<dyn std::error::Error>>;
 
 use assert2::let_assert;
 
+use kxio::{
+    fs::{FileSystem, TempFileSystem},
+    net::{MockNet, Net},
+    print::Printer,
+};
+
+use crate::{f, init::run, Ctx, NAME};
+
 mod init {
-
-    use test_log::test;
-
-    use crate::{f, init::run, NAME};
-
-    use super::given;
     use super::*;
+    use test_log::test;
 
     #[test]
     fn when_file_does_not_exist_should_create() {
@@ -92,10 +96,7 @@ mod config {
     }
 }
 mod template {
-
+    use super::*;
-    use std::collections::HashMap;
-
-    use crate::template;
 
     #[test]
     fn expand_should_substitute_values() {
@@ -104,7 +105,7 @@ mod template {
         let params = HashMap::from([("param1", "-v1-"), ("param2", "-v2-")]);
 
         //when
-        let result = template::expand(template, params);
+        let result = crate::template::expand(template, params);
 
         //then
         assert_eq!(result, "pre-v1-mid-v2-post");
@@ -112,13 +113,7 @@ mod template {
 }
 
 mod given {
-    use kxio::{
+    use super::*;
-        fs::{FileSystem, TempFileSystem},
-        net::{MockNet, Net},
-        print::Printer,
-    };
-
-    use crate::Ctx;
 
     pub fn a_context(fs: FileSystem, net: Net, prt: Printer) -> Ctx {
         Ctx { fs, net, prt }
@@ -135,4 +130,32 @@ mod given {
     pub fn a_printer() -> Printer {
         kxio::print::test()
     }
+
+    // pub fn a_config() -> AppConfig {
+    //     AppConfig {
+    //         trello: a_trello_config(),
+    //         nextcloud: a_nextcloud_config(),
+    //     }
+    // }
+
+    // pub fn a_trello_config() -> TrelloConfig {
+    //     TrelloConfig {
+    //         api_key: s!("trello-api-key").into(),
+    //         api_secret: s!("trello-api-secret").into(),
+    //         board_name: s!("Trello Platform Changes").into(),
+    //     }
+    // }
+
+    // pub fn a_nextcloud_config() -> NextcloudConfig {
+    //     let hostname = s!("nextcloud.example.org").into();
+    //     let username = s!("username").into();
+    //     let password = s!("password").into();
+    //     let board_id = NextcloudBoardId::new(2);
+    //     NextcloudConfig {
+    //         hostname,
+    //         username,
+    //         password,
+    //         board_id,
+    //     }
+    // }
 }

src/trello/api/members.rs

@@ -2,12 +2,10 @@
 use kxio::{net::Net, print::Printer};
 
 use crate::api_result::APIResult;
-use crate::{
+use crate::config::TrelloConfig;
-    f,
+use crate::trello::{
-    trello::{
+    types::{auth::TrelloAuth, board::TrelloBoard},
-        types::{auth::TrelloAuth, board::TrelloBoard},
+    url,
-        url,
-    },
 };
 
 /// Get lists from named board that Member belongs to
@@ -40,12 +38,13 @@ use crate::{
 ///  --url "https://api.trello.com/1/members/$TRELLO_USERNAME/boards?key=$TRELLO_KEY&token=$TRELLO_SECRET&lists=open" \
 ///  --header 'Accept: application/json'
 pub async fn get_boards_that_member_belongs_to(
-    auth: &TrelloAuth,
+    cfg: &TrelloConfig,
     net: &Net,
     prt: &Printer,
 ) -> APIResult<Vec<TrelloBoard>> {
+    let auth = TrelloAuth::new(&cfg.api_key, &cfg.api_secret);
     APIResult::new(
-        net.get(url(f!("/members/{}/boards?lists=open", **auth.user())))
+        net.get(url("/members/me/boards?lists=open"))
             .headers(auth.into())
             .header("Accept", "application/json")
             .send()

src/trello/api/tests/given.rs

@@ -1,8 +1,6 @@
 //
 use kxio::{net::MockNet, print::Printer};
 
-use crate::trello::types::auth::{TrelloApiKey, TrelloApiSecret, TrelloAuth, TrelloUser};
-
 pub(crate) fn a_network() -> MockNet {
     kxio::net::mock()
 }
@@ -11,10 +9,9 @@ pub(crate) fn a_printer() -> Printer {
     kxio::print::test()
 }
 
-pub(crate) fn an_auth() -> TrelloAuth {
+// pub(crate) fn an_auth<'cfg>(cfg: &'cfg TrelloConfig) -> TrelloAuth<'cfg> {
-    TrelloAuth::new(
+//     TrelloAuth {
-        TrelloApiKey::new("foo"),
+//         api_key: &cfg.api_key,
-        TrelloApiSecret::new("bar"),
+//         api_secret: &cfg.api_secret,
-        TrelloUser::new("baz"),
+//     }
-    )
+// }
-}

src/trello/api/tests/mod.rs

@@ -1,65 +1,65 @@
-// use super::*;
+//
+use std::collections::HashMap;
+
+use kxio::net::StatusCode;
+use serde_json::json;
+
+use crate::{
+    config::TrelloConfig,
+    s,
+    trello::{api::members::get_boards_that_member_belongs_to, types::board::TrelloBoard},
+};
 
-use super::*;
 mod given;
 
 type TestResult = color_eyre::Result<()>;
 
 mod members {
 
-     use std::collections::HashMap;
+    use super::*;
- 
+
-     use kxio::net::StatusCode;
+    #[tokio::test]
-     use serde_json::json;
+    async fn get_member_boards() -> TestResult {
- 
+        //given
-     use crate::{
+        let net = given::a_network();
-         s,
+        let prt = given::a_printer();
-         trello::{
+        let trello_config = TrelloConfig {
-             api::members::get_boards_that_member_belongs_to,
+            api_key: s!("foo").into(),
-             types::{board::TrelloBoard, TrelloBoardId, TrelloBoardName},
+            api_secret: s!("bar").into(),
-         },
+            board_name: s!("board-name").into(),
-     };
+        };
- 
+
-     use super::*;
+        net.on()
- 
+            .get("https://api.trello.com/1/members/me/boards?lists=open")
-     #[tokio::test]
+            .headers(HashMap::from([
-     async fn get_member_boards() -> TestResult {
+                (
-         //given
+                    s!("authorization"),
-         let net = given::a_network();
+                    s!("OAuth oauth_consumer_key=\"foo\", oauth_token=\"bar\""),
-         let prt = given::a_printer();
+                ),
-         let auth = given::an_auth();
+                (s!("accept"), s!("application/json")),
- 
+            ]))
-         net.on()
+            .respond(StatusCode::OK)
-             .get("https://api.trello.com/1/members/baz/boards?lists=open")
+            .body(s!(json!([
-             .headers(HashMap::from([
+                {"id": "1", "name": "board-name", "lists":[]}
-                 (
+            ])))?;
-                     s!("authorization"),
+
-                     s!("OAuth oauth_consumer_key=\"foo\", oauth_token=\"bar\""),
+        //when
-                 ),
+        let result = get_boards_that_member_belongs_to(&trello_config, &net.into(), &prt)
-                 (s!("accept"), s!("application/json")),
-             ]))
-             .respond(StatusCode::OK)
-             .body(s!(json!([
-                 {"id": "1", "name": "board-1", "lists":[]}
-             ])))?;
- 
-         //when
-        let result = get_boards_that_member_belongs_to(&auth, &net.into(), &prt)
             .await
             .result?;
- 
+
-         assert_eq!(
+        assert_eq!(
             result,
-             vec![TrelloBoard::new(
+            vec![TrelloBoard {
-                 TrelloBoardId::new("1"),
+                id: s!("1").into(),
-                 TrelloBoardName::new("board-1"),
+                name: s!("board-name").into(),
-                 vec![]
+                lists: vec![]
-             )]
+            }]
-         );
+        );
- 
+
-         //then
+        //then
-         Ok(())
+        Ok(())
-     }
+    }
 }
 // TODO: boards
 // TODO: lists

src/trello/boards.rs

@@ -0,0 +1,18 @@
+//
+use crate::{p, FullCtx};
+
+pub(crate) async fn list(ctx: FullCtx, dump: bool) -> color_eyre::Result<()> {
+    let api_result =
+        super::api::members::get_boards_that_member_belongs_to(&ctx.cfg.trello, &ctx.net, &ctx.prt)
+            .await;
+    if dump {
+        p!(ctx.prt, "{}", api_result.text);
+    } else {
+        let mut boards = api_result.result?;
+        boards.sort_by(|a, b| a.name.cmp(&b.name));
+        boards.into_iter().for_each(|board| {
+            p!(ctx.prt, "{}:{}", board.id, board.name);
+        });
+    }
+    Ok(())
+}

src/trello/mod.rs

@@ -1,14 +1,16 @@
 //
 // pub mod api;
+pub mod api;
+pub mod boards;
 pub mod types;
 
 // #[cfg(test)]
 // mod tests;
 
-// use crate::f;
+use crate::f;
-//
+
-// pub fn url(path: impl Into<String>) -> String {
+pub fn url(path: impl Into<String>) -> String {
-//     let path = path.into();
+    let path = path.into();
-//     assert!(path.starts_with("/"));
+    assert!(path.starts_with("/"));
-//     f!("https://api.trello.com/1{path}")
+    f!("https://api.trello.com/1{path}")
-// }
+}

src/trello/tests.rs

@@ -1,3 +1,4 @@
+
 mod board {
     // use crate::trello::{
     //     // api::boards::TrelloBoards as _,
@@ -34,4 +35,79 @@ mod board {
     //     //then
     //     assert_eq!(result, Some(&board));
     // }
+mod commands {
+    mod board {
+        use crate::trello::{
+            api::boards::TrelloBoards as _,
+            types::{
+                TrelloBoard, TrelloBoardId, TrelloBoardName, TrelloList, TrelloListId,
+                TrelloListName,
+            },
+        };
+
+        #[test]
+        fn name_returns_name() {
+            //given
+            let board = TrelloBoard::new(
+                TrelloBoardId::new("board-id"),
+                TrelloBoardName::new("board-name"),
+                vec![],
+            );
+
+            //when
+            let result = board.name();
+
+            //then
+            assert_eq!(result, &TrelloBoardName::new("board-name"));
+        }
+
+        #[test]
+        fn lists_should_return_lists() {
+            //given
+            let lists = vec![TrelloList::new(
+                TrelloListId::new("list-id"),
+                TrelloListName::new("list-name"),
+            )];
+            let board = TrelloBoard::new(
+                TrelloBoardId::new("board-id"),
+                TrelloBoardName::new("board-name"),
+                lists.clone(),
+            );
+
+            //when
+            let result = board.lists();
+
+            //then
+            assert_eq!(result, lists);
+        }
+
+        #[test]
+        fn list_of_boards_find_by_name_returns_board() {
+            //given
+            let board = TrelloBoard::new(
+                TrelloBoardId::new("2"),
+                TrelloBoardName::new("beta"),
+                vec![],
+            );
+            let boards = vec![
+                TrelloBoard::new(
+                    TrelloBoardId::new("1"),
+                    TrelloBoardName::new("alpha"),
+                    vec![],
+                ),
+                board.clone(),
+                TrelloBoard::new(
+                    TrelloBoardId::new("3"),
+                    TrelloBoardName::new("gamma"),
+                    vec![],
+                ),
+            ];
+
+            //when
+            let result = boards.find_by_name(board.name());
+
+            //then
+            assert_eq!(result, Some(&board));
+        }
+    }
 }

src/trello/types/auth.rs

@@ -16,12 +16,12 @@ newtype!(
 );
 
 #[derive(Debug, Clone)]
-pub struct TrelloAuth {
+pub struct TrelloAuth<'cfg> {
-    api_key: TrelloApiKey,
+    pub(crate) api_key: &'cfg TrelloApiKey,
-    api_secret: TrelloApiSecret,
+    pub(crate) api_secret: &'cfg TrelloApiSecret,
 }
-impl TrelloAuth {
+impl<'cfg> TrelloAuth<'cfg> {
-    pub const fn new(api_key: TrelloApiKey, api_secret: TrelloApiSecret) -> Self {
+    pub const fn new(api_key: &'cfg TrelloApiKey, api_secret: &'cfg TrelloApiSecret) -> Self {
         Self {
             api_key,
             api_secret,
@@ -35,8 +35,8 @@ impl TrelloAuth {
         &self.api_secret
     }
 }
-impl From<&TrelloAuth> for HashMap<String, String> {
+impl<'cfg> From<TrelloAuth<'cfg>> for HashMap<String, String> {
-    fn from(value: &TrelloAuth) -> Self {
+    fn from(value: TrelloAuth) -> Self {
         HashMap::from([(
             "Authorization".into(),
             format!(

src/trello/types/board.rs

@@ -1,11 +1,8 @@
 //
-use derive_more::derive::Constructor;
-
 use crate::trello::types::list::TrelloList;
+use crate::trello::types::{TrelloBoardId, TrelloBoardName};
 
-use super::{TrelloBoardId, TrelloBoardName};
+#[derive(Clone, Debug, PartialEq, Eq, serde::Deserialize)]
-
-#[derive(Clone, Debug, PartialEq, Eq, serde::Deserialize, Constructor)]
 pub(crate) struct TrelloBoard {
     pub(crate) id: TrelloBoardId,
     pub(crate) name: TrelloBoardName,

src/trello/types/list.rs

@@ -1,6 +1,7 @@
+//
 use derive_more::derive::Constructor;
 
-use super::{TrelloListId, TrelloListName};
+use crate::trello::types::{TrelloListId, TrelloListName};
 
 #[derive(Debug, Clone, PartialEq, Eq, serde::Deserialize, Constructor)]
 pub(crate) struct TrelloList {

src/trello/types/mod.rs

@@ -1,14 +1,14 @@
 pub(crate) mod auth;
-// pub(crate) mod board;
+pub(crate) mod board;
 // mod card;
-// mod list;
+mod list;
 // mod new_card;
 
 use derive_more::derive::Display;
 
 use crate::newtype;
 
-// newtype!(TrelloBoardId, String, Display, "Board ID");
+newtype!(TrelloBoardId, String, Display, "Board ID");
 newtype!(
     TrelloBoardName,
     String,
@@ -17,14 +17,14 @@ newtype!(
     Ord,
     "Board Name"
 );
-// newtype!(TrelloListId, String, "List ID");
+newtype!(TrelloListId, String, "List ID");
-// newtype!(
+newtype!(
-//     TrelloListName,
+    TrelloListName,
-//     String,
+    String,
-//     Display,
+    Display,
-//     PartialOrd,
+    PartialOrd,
-//     Ord,
+    Ord,
-//     "List Name"
+    "List Name"
-// );
+);
 // newtype!(TrelloCardId, String, Display, "Card ID");
 // newtype!(TrelloCardName, String, Display, "Card Name");