handle projects without default_branch correctly

Author: larscom

api/src/model/mod.rs

@@ -99,7 +99,7 @@ pub mod test {
             id: 456,
             name: "name".to_string(),
             web_url: "web_url".to_string(),
-            default_branch: "default_branch".to_string(),
+            default_branch: Some("default_branch".to_string()),
             topics: vec!["topic".to_string()],
         }
     }

api/src/model/project.rs

@@ -7,7 +7,7 @@ pub struct Project {
     pub id: u64,
     pub name: String,
     pub web_url: String,
-    pub default_branch: String,
+    pub default_branch: Option<String>,
     pub topics: Vec<String>,
 }
 

api/src/project/pipeline_aggregator.rs

@@ -44,10 +44,14 @@ impl PipelineAggregator {
         projects: Vec<Project>,
     ) -> Result<Vec<ProjectPipeline>, ApiError> {
         try_collect_with_buffer(projects, |project| async move {
-            let pipeline = self
-                .pipeline_service
-                .get_latest_pipeline(project.id, project.default_branch.clone())
-                .await?;
+            let pipeline = if let Some(default_branch) = project.clone().default_branch {
+                self.pipeline_service
+                    .get_latest_pipeline(project.id, default_branch)
+                    .await?
+            } else {
+                None
+            };
+
             Ok(ProjectPipeline {
                 group_id,
                 project,
@@ -75,10 +79,13 @@ impl PipelineAggregator {
         projects: Vec<Project>,
     ) -> Result<Vec<ProjectPipelines>, ApiError> {
         try_collect_with_buffer(projects, |project| async move {
-            let pipelines = self
-                .pipeline_service
-                .get_pipelines(project.id, None)
-                .await?;
+            let pipelines = if project.default_branch.is_some() {
+                self.pipeline_service
+                    .get_pipelines(project.id, None)
+                    .await?
+            } else {
+                Vec::default()
+            };
             Ok(ProjectPipelines {
                 group_id,
                 project,

api/src/schedule/pipeline_aggregator.rs

@@ -49,7 +49,11 @@ impl PipelineAggregator {
         projects: Vec<Project>,
     ) -> Result<Vec<ScheduleProjectPipeline>, ApiError> {
         let result = try_collect_with_buffer(projects, |project| async move {
-            let schedules = self.schedule_service.get_schedules(project.id).await?;
+            let schedules = if project.default_branch.is_some() {
+                self.schedule_service.get_schedules(project.id).await?
+            } else {
+                Vec::default()
+            };
             let result = self
                 .with_latest_pipeline(group_id, &project, schedules)
                 .await?;

proxy.conf.js

@@ -1,5 +1,5 @@
-const target = 'https://gitlab-ci-dashboard.larscom.nl'
-// const target = 'http://localhost:8080'
+// const target = 'https://gitlab-ci-dashboard.larscom.nl'
+const target = 'http://localhost:8080'
 
 module.exports = {
   '/api/**': {