Stability checker should run for reference changes

[gsnedders]

cc/ @jgraham @jugglinmike

[bobholt]

@gsnedders: can you elaborate on this? Sounds like it was parrt of a discussion I missed.

[gsnedders]

Given a reftest a.html with <link rel=match href=a-ref.html>, the stability checker will run the test if there is a change to a.html, but won't run the test if there is a change to a-ref.html.

[gsnedders]

(Of course the problem is harder than that because of the possibility of chaining references, where a-ref.html would have another <link rel=match>.)

[foolip]

@lukebjerring is this something you could look into as part of your work on #7475? I'm calling this roadmap on the assumption that will make sense.

[kereliuk]

@lukebjerring any update on this?

[lukebjerring]

No, I haven't looked at it yet, will assign to self though (it's in the pile)

[kereliuk]

@lukebjerring have you had a chance to look at this yet?

[lukebjerring]

Not as yet.

…

On Wed, Nov 29, 2017, 3:52 PM Jonathon Kereliuk ***@***.***> wrote: @lukebjerring <https://github.com/lukebjerring> have you had a chance to look at this yet? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#5319 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAve5IHHMD8V1Ts1_fbZwoxMDPjeiTacks5s7cQEgaJpZM4MxxfU> .

[alijuma]

Re-ping about an update since this has prioirty:roadmap.

[lukebjerring]

Dropping that label; this is a side-issue barely related to a P2 OKR.

[foolip]

@gsnedders given changes in the manifest format since this issue was filed, is this now easier to implement?

[gsnedders]

Not sure. It's been a long time since I've touched the code that finds how we find affected tests.

[foolip]

It looks like this has been fixed actually. Here's how I tried it:

echo >> css/css-grid/table-grid-item-dynamic-001-ref.html
git commit -a -m bla
./wpt tests-affected HEAD~

That lists css/css-grid/table-grid-item-dynamic-001.html as affected!