I can remove the ```disabled``` part as that's kind of a language change.

But neither you nor Andrei gave any good reason why ```file``` and ```line``` should be removed. Quoting Andrei ```Never had a need for that.```. Well I have a need for that as explained before and ```file```/```line``` were the main reason I started this pull request two years ago. So I won't just accept ```I don't see that this adds value, it's redundant.```(Already proven wrong) ```Never had a need for that``` or ```Still disagree``` as an answer, you'll have to provide some kind of argument.

```file/line``` are trivial to implement, it's not a language change, there's no added complexity for druntime or user code (you can hardly argue that two more fields in a struct increase complexity). Only the size of the executable slightly increases. We have ```2753``` tests in phobos. So the test executable will be 43kB bigger. The current size of the phobos test executable is ```62MB```.
Even if nobody ever used this feature, there's no harm. Why can't you just accept that a feature  that you don't need is useful to somebody else? 

Also quoting @andralex:
> The test runner could and should do its own bookkeeping with what tests should be ran, skipped, tried once in a while etc.

How can the test runner do book-keeping if it can not identify the tests? Not all tests will be named, unnamed tests cannot be identified in any other way.

And quoting @andralex again:
> Overall this pull request seems to do a bunch of changes in one go; we should reduce them.

That's not a honest statement. This pull request initially did not add ```name/identifier``` support, then you asked for that. ```name``` requires more changes than ```file``` / ```line``` together and it's more complex to implement, so you don't seem to be opposed to ```a bunch of changes in one go``` you're just opposed to ```file/line```.