wuffle 0.44.0 → 0.44.4

package/lib/apps/board-api-routes/board-api-filters.js

@@ -10,13 +10,47 @@ function filterIssueOrPull(issueOrPull) {
 module.exports.filterIssueOrPull = filterIssueOrPull;
 
 
-function filterRepository(githubRepository) {
+function filterCheckRun(checkRun) {
+  const {
+    status,
+    conclusion,
+    name,
+    html_url
+  } = checkRun;
+
+  return {
+    status,
+    conclusion,
+    name,
+    html_url
+  };
+}
+
+
+function filterStatus(status) {
+  const {
+    context,
+    description,
+    target_url,
+    state
+  } = status;
+
+  return {
+    context,
+    description,
+    target_url,
+    state
+  };
+}
+
+
+function filterRepository(repository) {
 
   const {
     name,
     owner,
     html_url
-  } = githubRepository;
+  } = repository;
 
   return {
     name,
@@ -28,13 +62,13 @@ function filterRepository(githubRepository) {
 module.exports.filterRepository = filterRepository;
 
 
-function filterUser(githubUser) {
+function filterUser(user) {
 
   const {
     login,
     avatar_url,
     html_url
-  } = githubUser;
+  } = user;
 
   return {
     login,
@@ -66,12 +100,14 @@ function filterLabel(githubLabel) {
 
   const {
     name,
-    color
+    color,
+    column_label
   } = githubLabel;
 
   return {
     name,
-    color
+    color,
+    ...(column_label ? { column_label } : {})
   };
 }
 
@@ -126,7 +162,11 @@ function filterPull(pullRequest) {
     merged,
     html_url,
     pull_request,
-    links = []
+    links = [],
+    order,
+    column,
+    check_runs = [],
+    statuses = []
   } = pullRequest;
 
   return {
@@ -135,18 +175,22 @@ function filterPull(pullRequest) {
     number,
     state,
     title,
-    user: filterUser(user),
+    user: user ? filterUser(user) : null,
     assignees: assignees.map(filterUser),
-    requested_reviewers: requested_reviewers.map(filterUser),
+    requested_reviewers: (requested_reviewers || []).map(filterUser),
     labels: labels.map(filterLabel),
-    comments: comments.map(filterComment),
+    comments: Array.isArray(comments) ? comments.map(filterComment) : [],
     milestone: milestone ? filterMilestone(milestone) : null,
     draft,
     merged,
     repository: filterRepository(repository),
     html_url,
     pull_request,
-    links: links.map(filterLink)
+    links: links.map(filterLink),
+    order,
+    column,
+    statuses: statuses.map(filterStatus),
+    check_runs: check_runs.map(filterCheckRun)
   };
 }
 
@@ -169,7 +213,9 @@ function filterIssue(issue) {
     repository,
     pull_request,
     comments = [],
-    links = []
+    links = [],
+    order,
+    column
   } = issue;
 
   return {
@@ -178,15 +224,17 @@ function filterIssue(issue) {
     number,
     state,
     title,
-    user: filterUser(user),
+    user: user ? filterUser(user) : null,
     assignees: assignees.map(filterUser),
     labels: labels.map(filterLabel),
     milestone: milestone ? filterMilestone(milestone) : null,
     repository: filterRepository(repository),
-    comments: comments.map(filterComment),
+    comments: Array.isArray(comments) ? comments.map(filterComment) : [],
     pull_request,
     html_url,
-    links: links.map(filterLink)
+    links: links.map(filterLink),
+    order,
+    column
   };
 
 }