-
-
Notifications
You must be signed in to change notification settings - Fork 197
tns test from a fresh project creation doesn't work #4119
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
Have managed to get the tests to at least execute by switching If this is known to not work, why is it default to true? Regardless, this is still a fairly broken default installation as after switching this, I'm still getting errors:
|
The only relation I see in that issue is that they're also trying to run Their unexpected behaviour is different (their tests run immediately, mine don't run at all). I am reporting that the default, out of the box installation, does not work with tests as per the documentation. As well as that, the suggested change provided by yourself makes no difference in my example. |
@Fatme I too think this is unrelated to #4071 :) This is completely out of the box v5 behaviour, and a fairly big impediment in my team using NS for serious enterprise applications. Are there any additional reproduction steps / information we can provide? We are getting the exact same issue as @Fedelaus |
@Fedelaus, @poppahorse, Sorry guys, my bad! I didn't notice the following: "No reachable hosts." It seems your android emulators are with android 28. If this is the case you need to add
|
@Fatme thank you so much, your solutions save my headache |
I came to this issue while setting up a CI build. I also found this helpful, they seem to be related: #2017 |
Thanks for the reply @Fatme. You are correct my emu for that specific output was 28 but I was seeing the same from a default installation using 24, and I managed to get them to execute as per here: #4119 (comment) but then immediately see that output when utilizing typescript files about exports, even though the documentation just says I need to import reflect-metadata and install jasmine types. It definitely feels like the out of the box setup requires tweaking if the documentation is actually correct. |
@Fatme Why!? Why need to add this workround. |
This workaround was needed because the default value of
As the default value was changed to However, this workaround is no more needed as we fixed it directly in |
@Fatme I am using Is this intentional or a bug? |
I made a pul request, to display an error message when this error happen |
Environment
Describe the bug
When initializing tests for a fresh installation on V5, they immediately do not work and just don't even start.
To Reproduce
tns create <appName> --ng
cd <appName> && tns test init
(I picked Jasmine for the test init)tns test android
The tests don't run.
Output here: https://hastebin.com/oluvahequl
You can see I edit the
example.js
file, in hopes a trigger of compilation could trigger tests.Meanwhile my android device is reporting
No reachable hosts
.Expected behavior
The tests to run.
The text was updated successfully, but these errors were encountered: