-
Notifications
You must be signed in to change notification settings - Fork 12k
Target e2e suite #807
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Hi, Any ideas when this will be implemented? 👍 |
I don't have an update on when this will be available, but if someone is interested in doing a PR I'll review it. As a workaround, right now you should be able to do |
I'd really like to knock out this easy little feature (or any angular-cli development), but I'm having trouble with the typings in the angular-cli project. My recent fork appears to be missing something because I get a lot of errors like:
|
@hboylan where do you get those errors? If it's in your editor, it might mean that it doesn't support Typescript 2.0. You should be able to ignore them though, it won't affect much. |
@filipesilva the Any idea for another workaround or maybe introduce another way to pass extra parameters to protractors? |
@victornoel there isn't a way to pass arbitrary parameters but as far as this issue is concerned now there is a way to pass suites:
|
@filipesilva actually Unfortunately for me, my need was to pass an |
You're absolutely right, and I was confusing the two. |
Can you tell me more about your usecase? Is it that you need to exclude specs/suites sometimes only? |
@filipesilva here it is:
I see two ways of solving this need:
|
Ok I understand better. Have you considered adding a different protractor config for now?
I agree that it's not a replacement for a |
I did not think of that, I will see if I can do something with that for now :) Thanks! Do you want me to open an issue for the |
Yes please! |
Ditto! I want this! |
This would be great - I may be misreading, but it looks like there was already an update for this, marked as superseded though, but I don't think it actually was. |
Any update on this? Using the protractor --suite flag would be helpful in our development |
Another workaround:
define your suites in protractor.conf.js:
run the desired suite with the double npm option:
|
|
@aniruddhadas9 you cannot use it with e2e. Add an extra protractor script command to package.json
and run it with |
I wanted to run it using angular cli command like |
Which version of the cli is this feature (#8354) going to be available in ? |
Its already available in |
Closing as this will be available in 1.7.0. |
@filipesilva if I use |
resolves: 807 Override suite in the protractor config. Can send in multiple suite by comma seperated values (ng e2e --suite=suite1.ts, suite2.ts). Issue link github.com/angular/issues/807 github.com/angular/pull/3551
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Hey all, I've found that it'd be nice to target a particular test suite with angular-cli.
Something like
ng e2e auth
instead of editing theangular-cli.json
:Looks like it'd be pretty easy to pass the proper arguments to the e2e task
The text was updated successfully, but these errors were encountered: