1323140 - run test-stylo just after generating bindings at build time

Status: RESOLVED DUPLICATE of bug 1373878

(Core :: CSS Parsing and Computation, defect, P2)

Description

[Cameron McCormack (:heycam)]

It would be great if we could run test-stylo automatically from the build, just after bindings are generated, and to have their failure be a build failure.  Ideally it would use the just-built style and geckolib crates, unlike doing a |cd servo/ && ./mach test-stylo|, which builds everything again under servo/.

Comment 1

[Bobby Holley (:bholley)]

Getting this on Nathan's radar. Nathan, do you think you'll have cycles for this sometime in Q2? It would be good to have this before we flip stylo on in Nightly.

Comment 2

[Nathan Froyd [:froydnj]]

I might.  bug 1331022 is similar, but this bug is slightly different insofar as we want to run it on vendored crates, not just first-party ones.  Are the servo/servo test bots not sufficient in this case because they're not running bindgen'd code?

Comment 3

[Bobby Holley (:bholley)]

(In reply to Nathan Froyd [:froydnj] from comment #2)
> I might.  bug 1331022 is similar, but this bug is slightly different insofar
> as we want to run it on vendored crates, not just first-party ones.

Are you counting the servo stuff as vendored?

It's worth noting that the only way in which the bindings are at all "vendored" is that build_gecko.rs lives in servo. But bug 1336540 proposes moving it to Gecko, which could kill two birds with one stone.


>  Are the
> servo/servo test bots not sufficient in this case because they're not
> running bindgen'd code?

The primary purpose of the test-stylo stuff is to run all the memory-layout tests that get generated with the bindings (i.e. [1]). That needs to run with every Gecko push, because it's possible to change Gecko headers to break bindgen without touching servo/ at all.


[1] http://searchfox.org/mozilla-central/rev/ee7cfd05d7d9f83b017dd54c2052a2ec19cbd48b/servo/components/style/gecko_bindings/structs_debug.rs#12150