refactor(trello): rename module types as model

src/config.rs

@@ -6,15 +6,7 @@ use crate::trello::TrelloConfig;
 use crate::{f, s, Ctx, NAME};
 
 #[derive(
-    Clone,
+    Clone, Debug, derive_more::From, PartialEq, Eq, derive_more::AsRef, serde::Deserialize,
-    Debug,
-    derive_more::From,
-    PartialEq,
-    Eq,
-    PartialOrd,
-    Ord,
-    derive_more::AsRef,
-    serde::Deserialize,
 )]
 pub(crate) struct AppConfig {
     pub(crate) trello: TrelloConfig,

src/trello/api/boards.rs

@@ -1,5 +1,16 @@
 //
-use crate::trello::types::{board::TrelloBoard, TrelloBoardName};
+use crate::trello::model::board::TrelloBoard;
+// use color_eyre::Result;
+// use kxio::net::Net;
+//
+// use crate::{
+//     f,
+//     trello::{
+//         model::{TrelloAuth, TrelloBoardId},
+//         url,
+//     },
+// };
+use crate::trello::model::TrelloBoardName;
 
 // pub(crate) async fn get_board(
 //     auth: &TrelloAuth,

src/trello/api/members.rs

@@ -1,12 +1,11 @@
 //
 use kxio::{net::Net, print::Printer};
 
-use crate::trello::TrelloConfig;
 use crate::{
     api_result::APIResult,
     trello::{
-        types::{auth::TrelloAuth, board::TrelloBoard},
+        model::{auth::TrelloAuth, board::TrelloBoard},
-        url,
+        url, TrelloConfig,
     },
 };
 
@@ -50,7 +49,7 @@ pub(crate) async fn get_boards_that_member_belongs_to(
     };
     APIResult::new(
         net.get(url("/members/me/boards?lists=open"))
-            .headers(auth.into())
+            .headers((&auth).into())
             .header("Accept", "application/json")
             .send()
             .await,

src/trello/api/tests/mod.rs

@@ -6,7 +6,9 @@ use serde_json::json;
 
 use crate::{
     s,
-    trello::{api::members::get_boards_that_member_belongs_to, types::board::TrelloBoard},
+    trello::{
+        api::members::get_boards_that_member_belongs_to, model::board::TrelloBoard, TrelloConfig,
+    },
 };
 
 mod given;
@@ -15,7 +17,6 @@ type TestResult = color_eyre::Result<()>;
 
 mod members {
     use super::*;
-    use crate::trello::TrelloConfig;
 
     #[tokio::test]
     async fn get_member_boards() -> TestResult {

src/trello/client.rs

@@ -1,6 +1,6 @@
 //
 use crate::api_result::APIResult;
-use crate::trello::types::board::TrelloBoard;
+use crate::trello::model::board::TrelloBoard;
 use crate::trello::TrelloConfig;
 use crate::FullCtx;
 

src/trello/mod.rs

@@ -2,16 +2,23 @@
 pub(crate) mod api;
 pub(crate) mod boards;
 pub(crate) mod client;
-pub(crate) mod types;
+pub(crate) mod model;
 
-// #[cfg(test)]
+#[cfg(test)]
-// mod tests;
+mod tests;
 
-use crate::execute::Execute;
+use crate::{
-use crate::trello::boards::TrelloBoardCommand;
+    execute::Execute,
-use crate::trello::types::auth::{TrelloApiKey, TrelloApiSecret};
+    f,
-use crate::trello::types::TrelloBoardName;
+    trello::{
-use crate::{f, FullCtx};
+        boards::TrelloBoardCommand,
+        model::{
+            auth::{TrelloApiKey, TrelloApiSecret},
+            TrelloBoardName,
+        },
+    },
+    FullCtx,
+};
 use clap::Parser;
 
 pub(crate) fn url(path: impl Into<String>) -> String {
@@ -34,7 +41,7 @@ impl Execute for TrelloCommand {
     }
 }
 
-#[derive(Clone, Debug, derive_more::From, PartialEq, Eq, PartialOrd, Ord, serde::Deserialize)]
+#[derive(Clone, Debug, derive_more::From, PartialEq, Eq, serde::Deserialize)]
 pub struct TrelloConfig {
     pub(crate) api_key: TrelloApiKey,
     pub(crate) api_secret: TrelloApiSecret,

src/trello/model/auth.rs

@@ -1,6 +1,6 @@
-//
 use std::collections::HashMap;
 
+//
 use derive_more::derive::Display;
 
 use crate::newtype;
@@ -16,12 +16,12 @@ newtype!(
 );
 
 #[derive(Debug, Clone)]
-pub struct TrelloAuth<'cfg> {
+pub(crate) struct TrelloAuth<'cfg> {
     pub(crate) api_key: &'cfg TrelloApiKey,
     pub(crate) api_secret: &'cfg TrelloApiSecret,
 }
-impl From<TrelloAuth<'_>> for HashMap<String, String> {
+impl From<&TrelloAuth<'_>> for HashMap<String, String> {
-    fn from(value: TrelloAuth) -> Self {
+    fn from(value: &TrelloAuth) -> Self {
         HashMap::from([(
             "Authorization".into(),
             format!(

src/trello/model/board.rs

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

src/trello/model/list.rs

@@ -0,0 +1,8 @@
+//
+use super::{TrelloListId, TrelloListName};
+
+#[derive(Debug, Clone, PartialEq, Eq, serde::Deserialize)]
+pub(crate) struct TrelloList {
+    pub(crate) id: TrelloListId,
+    pub(crate) name: TrelloListName,
+}

src/trello/model/mod.rs

@@ -9,14 +9,7 @@ use derive_more::derive::Display;
 use crate::newtype;
 
 newtype!(TrelloBoardId, String, Display, "Board ID");
-newtype!(
+newtype!(TrelloBoardName, String, Display, "Board Name");
-    TrelloBoardName,
-    String,
-    Display,
-    PartialOrd,
-    Ord,
-    "Board Name"
-);
 newtype!(TrelloListId, String, "List ID");
 newtype!(
     TrelloListName,
@@ -26,5 +19,5 @@ newtype!(
     Ord,
     "List Name"
 );
-// newtype!(TrelloCardId, String, Display, "Card ID");
+newtype!(TrelloCardId, String, Display, "Card ID");
-// newtype!(TrelloCardName, String, Display, "Card Name");
+newtype!(TrelloCardName, String, Display, "Card Name");

src/trello/tests.rs

@@ -1,146 +1,43 @@
-mod board {
+//
-    // use crate::trello::{
+use crate::trello::{
-    //     // api::boards::TrelloBoards as _,
+    api::boards::TrelloBoards,
-    //     types::{
+    api::boards::TrelloBoards as _,
-    //         board::TrelloBoard, TrelloBoardId, TrelloBoardName, TrelloListId, TrelloListName,
+    model::{board::TrelloBoard, TrelloBoardId, TrelloBoardName},
-    //     },
+};
-    // };
 
-    // #[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));
-    // }
 mod commands {
+    use super::*;
+
     mod board {
-            #[test]
+        use super::*;
-            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(
+            let board = TrelloBoard {
-                    TrelloBoardId::new("2"),
+                id: TrelloBoardId::new("2"),
-                    TrelloBoardName::new("beta"),
+                name: TrelloBoardName::new("beta"),
-                    vec![],
+                lists: vec![],
-                );
+            };
-                let boards = vec![TrelloBoard::new(
-                    TrelloBoardId::new("1"),
-                    TrelloBoardName::new("alpha"),
-                    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(
+                TrelloBoard {
-                        TrelloBoardId::new("1"),
+                    id: TrelloBoardId::new("1"),
-                        TrelloBoardName::new("alpha"),
+                    name: TrelloBoardName::new("alpha"),
-                        vec![],
+                    lists: vec![],
-                    ),
+                },
                 board.clone(),
-                    TrelloBoard::new(
+                TrelloBoard {
-                        TrelloBoardId::new("3"),
+                    id: TrelloBoardId::new("3"),
-                        TrelloBoardName::new("gamma"),
+                    name: TrelloBoardName::new("gamma"),
-                        vec![],
+                    lists: vec![],
-                    ),
+                },
             ];
 
             //when
-                let result = boards.find_by_name(board.name());
+            let result = boards.find_by_name(&board.name);
 
             //then
             assert_eq!(result, Some(&board));
         }
     }
 }
-}

src/trello/types/list.rs

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