Stream: t-compiler/wg-nll

Topic: #53351-review-ignored-tests

pnkfelix (Aug 14 2018 at 20:43, on Zulip):

Just as a heads up for @davidtwco : My first task is going to be to put in links to the locations where each file actually ended up

pnkfelix (Aug 14 2018 at 20:44, on Zulip):

(you can see how I've been encoding it so far; my technique has been a pretty simple minded `find src/test/ui -name '*' invocation.)

pnkfelix (Aug 14 2018 at 20:45, on Zulip):

but also, I think at least some of these cases are instances where the test is/was weak, in the sense that a lot of these tests assume lexical lifetimes for no reason: e.g. one should add a use of the borrow, rather than let it have an artificially short life time.

pnkfelix (Aug 14 2018 at 20:46, on Zulip):

I'll add either add notes about such cases, or put up a PR fixing them.

pnkfelix (Aug 14 2018 at 20:46, on Zulip):

(I think you should be able to find similar updates to tests if you look at the PR where we originally landed compare-mode ... I think ...)

davidtwco (Aug 14 2018 at 20:47, on Zulip):

Sounds good. If you've not got time to fix them but leave some notes then I can put a PR in.

davidtwco (Aug 15 2018 at 08:45, on Zulip):

Had a look over your PR and it seemed fine to me @pnkfelix. Might be worth getting another pair of eyes on it though.

pnkfelix (Aug 15 2018 at 09:15, on Zulip):

@davidtwco I was curious : what technique/methodology did you use to validate the .nll.stderr files when you portes compile-fail to ui/?

davidtwco (Aug 15 2018 at 09:16, on Zulip):

A quick glance at each of them to see if it looked right. Not particularly rigorous but there were a lot of them.

pnkfelix (Aug 15 2018 at 09:16, on Zulip):

Because even with this relatively small set of tests, i found my eyes quickly glazed over ...

pnkfelix (Aug 15 2018 at 14:31, on Zulip):

I guess we should just plan a third round of comparison between .stderr and .nll.stderr. Maybe I’ll try to make a tool this time for equating the ones with only cosmetic deltas ...

Last update: Jul 14 2020 at 12:30UTC