Following the guide here: <a href="http://docs.stormpath.com/angularjs/guide/confi

I've tried to reproduce the error, but couldn't. Try running <code class="notranslate"

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

Tests broken following guide about stormpath-sdk-angularjs HOT 10 OPEN

schinkowitch commented on August 16, 2024

Tests broken following guide

from stormpath-sdk-angularjs.

Comments (10)

drpalaric commented on August 16, 2024

I've tried to reproduce the error, but couldn't. Try running bower install --save stormpath-sdk-angularjs again. There might have been an issue when you were downloading from bower. Also, there might be more to the stack trace than what you are finding, and adding files manually won't solve the issue. Since you're not too far head in the tutorial, I'd simply try to go through the instructions and install the dependencies again.

from stormpath-sdk-angularjs.

schinkowitch commented on August 16, 2024

I started the tutorial from scratch, and followed the steps carefully. After performing the step: Add Stormpath to the Angular Application,
the client tests still fail when executed from the command line (grunt test:client) .

Here is the error with the full stack trace:

from stormpath-sdk-angularjs.

drpalaric commented on August 16, 2024

Yes, I got the same error running grunt test:client, but I think this error has to do with Angular-Fullstack and not Stormpath. Have you tried simply running grunt serve? That should work.

from stormpath-sdk-angularjs.

schinkowitch commented on August 16, 2024

Yes, grunt serve works fine. The point is I'm evaluating using Stormpath for a production app and I need to write testable code. I would think this would be important for you all as well.

I've played around with this some more and found that the following steps seem to fix the issue:

add
'client/bower_components/stormpath-sdk-angularjs/dist/stormpath-sdk-angularjs.js',
'client/bower_components/stormpath-sdk-angularjs/dist/stormpath-sdk-angularjs.tpls.min.js',
to the files in karma.conf.js
Use Chrome instead of PhantomJS for the browser: browsers: ['Chrome'], in karma.conf.js

Then you get a more reasonable test failure:
Error: Unexpected request: GET /api/users/current
Expected GET /api/things

The stormpath-sdk-angular.js uses ES5.1 Function.prototype.bind which doesn't seem to be implemented in PhantomJS. (This is unfortunate when you recommend angular-fullstack and it is the default test browser).

from stormpath-sdk-angularjs.

drpalaric commented on August 16, 2024

Disclaimer: I don't work for Stormpath and I am a pretty new developer.

Anyway, what you stated about including all the libraries into Karma, is apparently what you need to do to get the tests working initially.

For the second issue, the error seems to be coming from trying to get the route /api/things, but /api/users/current (from stormpath-sdk-express) is getting in the way. Both are called when you get to the front page, and both are called when you log in. Anyway, I think that there might be an issue with the test itself, and looking at the tests for the other controllers, you'll see that they are very generic.

from stormpath-sdk-angularjs.

robertjd commented on August 16, 2024

Hi @schinkowitch , thank you for reporting this. At the moment we don't have a recommendation for a testing framework, so take anything you see created by Angular-Fullstack is merely an example. Please note that this module is still beta.

Perhaps you could use the ES5 shim with Phantom?

Regarding the 'Unexpected request' error, I believe that is coming from Angular's mock $http service

Hopefully these comments help, In lieu of formal instructions for testing.

from stormpath-sdk-angularjs.

schinkowitch commented on August 16, 2024

Ok, thanks. Using the ES5 shim with PhantomJS will probably work. I got the test to run on PhantomJS by using Mozilla's Polyfill here.

The "Error: Unexpected request: GET /api/users/current" test failure is reasonable as the sdk is checking to see if the user is logged in. It's a one line fix in the test.

from stormpath-sdk-angularjs.

kevin212 commented on August 16, 2024

Hi @schinkowitch, what was the "one line fix..." that resolved this issue for you?

As soon as I add stormpath and stormpath.templates as application dependencies, all but a few of my tests fail with that same error: "Error: Unexpected request:".... and it is because SP is checking to see if the user is loogged-in. But my tests are not expecting these calls.

Can you share the one-line fix that worked for you in your tests?

from stormpath-sdk-angularjs.

kevin212 commented on August 16, 2024

Actually, I was able to figure it out:

$httpBackend.expectGET(CURRENT_USER_URI).respond(200, {});

from stormpath-sdk-angularjs.

schinkowitch commented on August 16, 2024

Note: the angular fullstack generator has dropped support for vanilla JS in 3.x, so when following the guide, you need to install the generator with:

npm install -g [email protected]

from stormpath-sdk-angularjs.

Tests broken following guide about stormpath-sdk-angularjs HOT 10 OPEN

Comments (10)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent