Renamed retrieveAuthMethods to retrieveAuthMethodsFromAuthStatus

This commit is contained in:
Giuseppe Digilio
2020-01-10 15:30:26 +01:00
parent 3132da9b3d
commit 6f8f4b31bf
3 changed files with 4 additions and 4 deletions

View File

@@ -196,7 +196,7 @@ export class AuthEffects {
.pipe(
ofType(AuthActionTypes.RETRIEVE_AUTH_METHODS),
switchMap((action: RetrieveAuthMethodsAction) => {
return this.authService.retrieveAuthMethods(action.payload)
return this.authService.retrieveAuthMethodsFromAuthStatus(action.payload)
.pipe(
map((authMethodModels: AuthMethod[]) => new RetrieveAuthMethodsSuccessAction(authMethodModels)),
catchError((error) => observableOf(new RetrieveAuthMethodsErrorAction()))

View File

@@ -139,13 +139,13 @@ describe('AuthService test', () => {
it('should return the authentication methods available', () => {
const authStatus = new AuthStatus();
authService.retrieveAuthMethods(authStatus).subscribe((authMethods: AuthMethod[]) => {
authService.retrieveAuthMethodsFromAuthStatus(authStatus).subscribe((authMethods: AuthMethod[]) => {
expect(authMethods).toBeDefined();
expect(authMethods.length).toBe(0);
});
authStatus.authMethods = authMethodsMock;
authService.retrieveAuthMethods(authStatus).subscribe((authMethods: AuthMethod[]) => {
authService.retrieveAuthMethodsFromAuthStatus(authStatus).subscribe((authMethods: AuthMethod[]) => {
expect(authMethods).toBeDefined();
expect(authMethods.length).toBe(2);
});

View File

@@ -211,7 +211,7 @@ export class AuthService {
* Retrieve authentication methods available
* @returns {User}
*/
public retrieveAuthMethods(status: AuthStatus): Observable<AuthMethod[]> {
public retrieveAuthMethodsFromAuthStatus(status: AuthStatus): Observable<AuthMethod[]> {
let authMethods: AuthMethod[] = [];
if (isNotEmpty(status.authMethods)) {
authMethods = status.authMethods;