[no subject]

[Ricardo Wurmus]

branch: master
commit dde1a9356493884b21b6b1dd585c5d6b9797cbb0
Author: Ricardo Wurmus <address@hidden>
AuthorDate: Wed Apr 15 19:50:25 2020 +0200

    database: Support "status:" keyword in queries.
    
    * src/cuirass/database.scm (query->bind-arguments): Handle "status" query.
    (db-get-builds-by-search, db-get-builds-query-min, db-get-builds-query-max):
    Add status filter.
---
 src/cuirass/database.scm | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/cuirass/database.scm b/src/cuirass/database.scm
index 40c4b92..25ef527 100644
--- a/src/cuirass/database.scm
+++ b/src/cuirass/database.scm
@@ -4,7 +4,7 @@
 ;;; Copyright © 2018, 2020 Ludovic Courtès <address@hidden>
 ;;; Copyright © 2018 Clément Lassieur <address@hidden>
 ;;; Copyright © 2018 Tatiana Sholokhova <address@hidden>
-;;; Copyright © 2019 Ricardo Wurmus <address@hidden>
+;;; Copyright © 2019, 2020 Ricardo Wurmus <address@hidden>
 ;;;
 ;;; This file is part of Cuirass.
 ;;;
@@ -624,6 +624,12 @@ WHERE derivation =" derivation ";"))
 
 (define (query->bind-arguments query-string)
   "Return a list of keys to query strings by parsing QUERY-STRING."
+  (define status-values
+    `(("success" . ,(build-status succeeded))
+      ("failed"  . ,(build-status failed))
+      ("failed-dependency" . ,(build-status failed-dependency))
+      ("failed-other" . ,(build-status failed-other))
+      ("canceled" . ,(build-status canceled))))
   (let ((args (append-map
                (lambda (token)
                  (match (string-split token #\:)
@@ -631,6 +637,8 @@ WHERE derivation =" derivation ";"))
                     `(#:system ,system))
                    (("spec" spec)
                     `(#:spec ,spec))
+                   (("status" status)
+                    `(#:status ,(assoc-ref status-values status)))
                    ((_ invalid) '())    ; ignore
                    ((query)
                     `(#:query
@@ -668,6 +676,8 @@ FROM Builds
 INNER JOIN Evaluations ON Builds.evaluation = Evaluations.id
 INNER JOIN Specifications ON Evaluations.specification = Specifications.name
 WHERE (Builds.nix_name LIKE :query)
+AND (:status IS NULL
+ OR (Builds.status = :status))
 AND (:spec IS NULL
  OR (Specifications.name = :spec))
 AND (:system IS NULL
@@ -973,6 +983,8 @@ ORDER BY E.id ASC;")))
 INNER JOIN Evaluations ON Builds.evaluation = Evaluations.id
 INNER JOIN Specifications ON Evaluations.specification = Specifications.name
 WHERE (Builds.nix_name LIKE :query)
+AND (:status IS NULL
+ OR (Builds.status = :status))
 AND (:spec IS NULL
  OR (Specifications.name = :spec))
 AND (:system IS NULL
@@ -992,6 +1004,8 @@ AND (:system IS NULL
 INNER JOIN Evaluations ON Builds.evaluation = Evaluations.id
 INNER JOIN Specifications ON Evaluations.specification = Specifications.name
 WHERE (Builds.nix_name LIKE :query)
+AND (:status IS NULL
+ OR (Builds.status = :status))
 AND (:spec IS NULL
  OR (Specifications.name = :spec))
 AND (:system IS NULL