Merge pull request #142 from filecoin-project/feat/github-id-tests-fixes

fix: test fixes

Author: AlexMCon

fplus-lib/src/external_services/github.rs

@@ -174,6 +174,7 @@ impl GithubWrapper {
             .issues(&self.owner, &self.repo)
             .get(number)
             .await?;
+        println!("{:?}", iid);
         Ok(iid)
     }
 

fplus-lib/src/parsers.rs

@@ -341,34 +341,35 @@ impl From<IssueValidData> for Datacap {
     }
 }
 
-// #[cfg(test)]
-// mod tests {
-//     use crate::external_services::github::github_async_new;
+#[cfg(test)]
+mod tests {
+    use crate::external_services::github::github_async_new;
 
-//     static OWNER: &str = "filecoin-project";
-//     static REPO: &str = "filplus-tooling-backend-test";
+    static OWNER: &str = "keyko-io";
+    static REPO: &str = "test-philip-second";
 
-//     #[tokio::test]
-//     async fn test_parser() {
-//         let gh = github_async_new(OWNER.to_string(), REPO.to_string()).await;
-//         let issue = gh.list_issue(706).await.unwrap();
-//         let parsed_ldn = super::ParsedIssue::from_issue_body(&issue.body.unwrap());
-//         dbg!(&parsed_ldn);
+    #[tokio::test]
+    async fn test_parser() {
+        let _ = fplus_database::setup().await;
+        let gh = github_async_new(OWNER.to_string(), REPO.to_string()).await;
+        let issue = gh.list_issue(37).await.unwrap();
+        let parsed_ldn = super::ParsedIssue::from_issue_body(&issue.body.unwrap());
+        dbg!(&parsed_ldn);
 
-//         assert_eq!(parsed_ldn.version, 1);
-//         assert!(!parsed_ldn.id.is_empty());
+        assert_eq!(parsed_ldn.version, 1);
+        assert!(!parsed_ldn.id.is_empty());
 
-//         assert!(!parsed_ldn.client.name.is_empty());
-//         assert!(!parsed_ldn.client.industry.is_empty());
-//         assert!(!parsed_ldn.client.region.is_empty());
-//         assert!(!parsed_ldn.client.website.is_empty());
-//         assert!(!parsed_ldn.client.social_media.is_empty());
-//         assert!(!parsed_ldn.client.social_media_type.is_empty());
-//         assert!(!parsed_ldn.client.role.is_empty());
-//         assert!(!parsed_ldn.project.project_id.is_empty());
-//         assert!(!parsed_ldn.project.history.is_empty());
-//         assert!(!parsed_ldn.project.associated_projects.is_empty());
+        assert!(!parsed_ldn.client.name.is_empty());
+        assert!(!parsed_ldn.client.industry.is_empty());
+        assert!(!parsed_ldn.client.region.is_empty());
+        assert!(!parsed_ldn.client.website.is_empty());
+        assert!(!parsed_ldn.client.social_media.is_empty());
+        assert!(!parsed_ldn.client.social_media_type.is_empty());
+        assert!(!parsed_ldn.client.role.is_empty());
+        assert!(!parsed_ldn.project.project_id.is_empty());
+        assert!(!parsed_ldn.project.history.is_empty());
+        assert!(!parsed_ldn.project.associated_projects.is_empty());
 
-//         assert!(!parsed_ldn.datacap.total_requested_amount.is_empty());
-//     }
-// }
+        assert!(!parsed_ldn.datacap.total_requested_amount.is_empty());
+    }
+}