bug#31844: [cuirass] Invalid specs are ignored, should be reported instead

bug#31844: [cuirass] Invalid specs are ignored, should be reported instead

[Ricardo Wurmus]

When Cuirass processes a list of specifications it silently ignores
those that would result in erroneous SQL statements.

Take for example a specification that does not include an #:arguments
field.  The database schema says that “arguments” may not be NULL, but
db-add-specification will produce an INSERT statement that sets
“arguments” to NULL when it is not part of the specification alist.

Cuirass should either validate the specifications before passing them to
“db-add-specification” or report an error when “db-add-specification”
fails to add a table record.

--
Ricardo

bug#31844: [cuirass] Invalid specs are ignored, should be reported instead

[Mathieu Othacehe]

Hello,

Invalid specifications are now reported, see for example:

--8<---------------cut here---------------start------------->8---
(list (specification
       (namee "guix-master")
       (build 'hello)))
--8<---------------cut here---------------end--------------->8---

gives,

--8<---------------cut here---------------start------------->8---
2021-03-25T14:52:46 running Fibers on 4 kernel threads
/home/mathieu/cuirass/examples/hello.scm:1:6: error: (specification (namee "guix-master") (build (quote hello))): extraneous field initializers (namee)
--8<---------------cut here---------------end--------------->8---

Closing this one,

Thanks,

Mathieu