authenticationUrl fix (KEY-343) (#40)

* authenticationUrl fix + tests

* linting

package.json

@@ -1,6 +1,6 @@
 {
   "name": "auth0-sso-dashboard",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "This extension provides your users with a dashboard for all of their applications.",
   "engines": {
     "node": ">=4.x"
@@ -130,6 +130,7 @@
     "autoprefixer": "^6.5.1",
     "babel-eslint": "7.1.1",
     "babel-loader": "^8.0.0-beta",
+    "babel-polyfill": "^6.26.0",
     "chai": "^3.5.0",
     "classnames": "^2.2.5",
     "cross-env": "^3.1.3",

server/lib/authenticationUrl.js

@@ -23,7 +23,7 @@ module.exports = (app) => {
   }
 
   if (app.connection) {
-    const sep = loginUrl.indexOf('?') > 0 ? '?' : '&';
+    const sep = loginUrl.indexOf('?') < 0 ? '?' : '&';
     const type = authProtocol === 'wsfed' ? 'whr' : 'connection';
     loginUrl += `${sep}${type}=${app.connection}`;
   }

server/lib/multipartRequest.js

@@ -1,7 +1,7 @@
 import Promise from 'bluebird';
 import { ArgumentError } from 'auth0-extension-tools';
 
-export default function(client, entity, opts = {}, perPage = 100, concurrency = 5) {
+export default function (client, entity, opts = {}, perPage = 100, concurrency = 5) {
   if (client === null || client === undefined) {
     throw new ArgumentError('Must provide a auth0 client object.');
   }
@@ -41,7 +41,7 @@ export default function(client, entity, opts = {}, perPage = 100, concurrency =
         }
 
         const pages = [];
-        for (let i=1; i<=pageCount; i++) {
+        for (let i = 1; i <= pageCount; i++) {
           pages.push(i);
         }
 

tests/server/lib/authenticationUrl.tests.js

@@ -0,0 +1,47 @@
+/* eslint-disable import/no-extraneous-dependencies */
+import { expect } from 'chai';
+import { setProvider } from '../../../server/lib/config';
+import authenticationUrl from '../../../server/lib/authenticationUrl';
+
+describe('authenticationUrl', () => {
+  setProvider(() => 'auth0.example.com');
+
+  it('should return saml url with connection', () => {
+    const app = {
+      type: 'saml',
+      scope: 'scope',
+      client: 'client',
+      callback: 'callback',
+      connection: 'connection',
+      response_type: 'response_type'
+    };
+
+    expect(authenticationUrl(app)).to.equal('https://auth0.example.com/samlp/client?connection=connection');
+  });
+
+  it('should return wsfed url with connection', () => {
+    const app = {
+      type: 'wsfed',
+      scope: 'scope',
+      client: 'client',
+      callback: 'callback',
+      connection: 'connection',
+      response_type: 'response_type'
+    };
+
+    expect(authenticationUrl(app)).to.equal('https://auth0.example.com/wsfed/client?wreply=callback&whr=connection');
+  });
+
+  it('should return oidc url with connection', () => {
+    const app = {
+      type: 'oidc',
+      scope: 'scope',
+      client: 'client',
+      callback: 'callback',
+      connection: 'connection',
+      response_type: 'response_type'
+    };
+
+    expect(authenticationUrl(app)).to.equal('https://auth0.example.com/authorize?response_type=response_type&scope=scope&client_id=client&redirect_uri=callback&connection=connection');
+  });
+});

webtask.json

@@ -1,7 +1,7 @@
 {
   "title": "SSO Dashboard",
   "name": "auth0-sso-dashboard",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "preVersion": "1.1.6",
   "author": "Auth0",
   "useHashName": false,