-
Notifications
You must be signed in to change notification settings - Fork 56
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
Dots from default reporter still show up #8
Comments
Actually, protractor's default jasmine reporter comes with the minijasminenode library and is not easy to deactivate for now. I proposed a PR to add a silent option a couple of weeks ago but it is not merged yet. |
cool, thanks again! |
I have just published a new version (0.3.1) with a hack to be able to remove dot reporter. You can give it a try if you want. |
Hi, I got new version, but dots still present. "jasmine-reporters" : "0.4.0", |
have you added the hack to your spec files ? |
ok, it works. I added wrong path to hack file. |
You could find a dedicated section on the Readme for the different ways to remove the dots. |
@bcaudan Hi. When using your jasmine-spec-reporter with multiCapabilities the dots still appear as the tests are running, but then it does output the tests in the correct spec style at the end. If you run only 1 browser you get each spec style output as each test is running. Is there a way to turn off the dots entirely, even for multi capability tests? |
I opened a dedicated issue for it (#17). |
can you add this section back? https://github.com/bcaudan/jasmine-spec-reporter/tree/v2.0.0#remove-protractor-dot-reporter still an issue with [email protected] + [email protected] + [email protected], but your fix works. |
This section has been moved to configuration > protractor |
Still not working for me even if I applied the workaround
And this is the output:
|
@ahmed-abdulmoniem if you upgrade to jasmine 2.5.2, then you can use |
@disjunction It can't be set in conf file? I should call it manually? |
@ahmed-abdulmoniem I saw a pull request about adding it to the options, but can't find it now. Anyway as far as I can see it not in the current code. |
This worked for me:
|
@valterkraemer This worked for me. Thanks. |
|
You need [email protected] to use |
@disjunction, |
@jeremyhewett I've just tested, and the exit code did work for me. Tested with [email protected] |
Hi. There is a simple solution described on https://github.com/bcaudan/jasmine-spec-reporter/blob/master/examples/protractor/README.md And it looks like this: |
The @xkamil suggestion doesn't seem to work in jasmine 3.4.0. :( |
@demisx I've just tested it with the protractor example of this repository with: and it works fine on my machine. |
The initial issue has been resolved by the hack described in the documentation. I'm locking the conversation to avoid spam in this old issue. |
Thanks for building the plugin!
In my tests the dots from the default reporter still show up. Take a look at them at Travis nghellostyle#L742. They break the indentation for list items.
Here is my config. Is it something on my side or is it a bug?
The text was updated successfully, but these errors were encountered: