You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Add an option to run/debug all the parametrizations of a tests when clicking on "Run/Debug Test (Multiple)" #5608
This would be an nice feature! Maybe if there are multiple tests provide another code lens option like "Run all X Tests (Multiple)" or something like that, where X is the number of parametrized tests.
EDIT:
If you provide some help on writing tests, or a hint where i can find existing tests that test the codelens feature, i would prepare a pull request.
Luciana Abud (@luabud)Brett Cannon (@brettcannon)
I think we'll end up with too many code lenses. For testing alone we'll have 3 code lenses on the one method.
Languages add code lenses for # of references, code complexity, & the like. We might want to re-consider adding yet another code lens
I think 3 code lenses for one single feature (testing) is too many.
Install gitlens and a couple of other extensions, then we have too many even now.
Suggestion: How about we drop (Multple) from the code lenses. Do we need it?
I actually like the codelenses for parametrized tests so that you can select single tests to run or debug. Maybe as you said Don Jayamanne (@DonJayamanne) drop the (Multiple) because you actually see that there is a parametrized test, which then would clean up the appearance. What bothered me is that you cannot run all single parametrized tests in a row with a quick action (like the code lens). This is the main reason for proposing the third option "Run All" in #8682.
Yeah could also be a solution! I thought it would be more intuitive and fast that you don't have to click on Run Test and then move the mouse up to the drop down list to select that Run All command. Just my five cents ;)
Woohoo 🎉 thanks Philipp Loose (@phloose) for taking interest in this PR!
I agree with Don, we already have enough code lenses as it is 😅 let's make "Run all" be the first item in the drop down list.
Activity
phloose commentedon Nov 14, 2019
This would be an nice feature! Maybe if there are multiple tests provide another code lens option like "Run all X Tests (Multiple)" or something like that, where X is the number of parametrized tests.
EDIT:
If you provide some help on writing tests, or a hint where i can find existing tests that test the codelens feature, i would prepare a pull request.
DonJayamanne commentedon Nov 20, 2019
Luciana Abud (@luabud) Brett Cannon (@brettcannon)
I think we'll end up with too many code lenses. For testing alone we'll have 3 code lenses on the one method.
Languages add code lenses for
# of references,code complexity, & the like. We might want to re-consider adding yet another code lensI think 3 code lenses for one single feature (testing) is too many.
Install
gitlensand a couple of other extensions, then we have too many even now.Suggestion: How about we drop
(Multple)from the code lenses. Do we need it?phloose commentedon Nov 20, 2019
I actually like the codelenses for parametrized tests so that you can select single tests to run or debug. Maybe as you said Don Jayamanne (@DonJayamanne) drop the
(Multiple)because you actually see that there is a parametrized test, which then would clean up the appearance. What bothered me is that you cannot run all single parametrized tests in a row with a quick action (like the code lens). This is the main reason for proposing the third option "Run All" in #8682.DonJayamanne commentedon Nov 20, 2019
This could also be another item (first item) in the drop down list, instead of adding another code lens.
phloose commentedon Nov 20, 2019
Yeah could also be a solution! I thought it would be more intuitive and fast that you don't have to click on Run Test and then move the mouse up to the drop down list to select that Run All command. Just my five cents ;)
phloose commentedon Nov 20, 2019
If you decide against the third code lens option i could work on the "Run All" drop down element!
luabud commentedon Nov 21, 2019
Woohoo 🎉 thanks Philipp Loose (@phloose) for taking interest in this PR!
I agree with Don, we already have enough code lenses as it is 😅 let's make "Run all" be the first item in the drop down list.
phloose commentedon Nov 21, 2019
OK, i will try and prepare something for that dropdown option!
phloose commentedon Nov 23, 2019
Don Jayamanne (@DonJayamanne) Luciana Abud (@luabud) i opened a pull request for the discussed option. I am keen for input and improvements from your side!
Add news entry for microsoft#5608
Add news entry for microsoft#5608
Add "Run All/Debug All" dropdown picker for parametrized tests (#8757)
phloose commentedon Dec 12, 2019
Luciana Abud (@luabud) as the pull request got merged shouldn't this be closed?
luabud commentedon Dec 12, 2019
Yes, thanks for bringing it up :)