Merge pull request silverbux#25 from silverbux/angular-tests

Angular unit testing

Author: silverbux

angular/app/components/forgot-password/forgot-password.component.html

@@ -5,7 +5,7 @@ <h4>Error:</h4>
     </div>
     <div class="form-group has-feedback">
         <input type="email" class="form-control" placeholder="Please enter your email address" ng-model="vm.email">
-        <span class="glyphicon glyphicon-lock form-control-feedback"></span>
+        <span class="glyphicon glyphicon-envelope form-control-feedback"></span>
     </div>
     <div class="row">
         <div class="col-xs-8">

angular/config/routes.config.js

@@ -48,7 +48,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<comingSoon></comingSoon>'
+          template: '<coming-soon></coming-soon>'
         }
       }
     })
@@ -59,7 +59,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userProfile></userProfile>'
+          template: '<user-profile></user-profile>'
         }
       },
       params: {
@@ -73,7 +73,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userLists></userLists>'
+          template: '<user-lists></user-lists>'
         }
       }
     })
@@ -84,7 +84,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userEdit></userEdit>'
+          template: '<user-edit></user-edit>'
         }
       },
       params: {
@@ -99,7 +99,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userRoles></userRoles>'
+          template: '<user-roles></user-roles>'
         }
       }
     })
@@ -110,7 +110,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userPermissions></userPermissions>'
+          template: '<user-permissions></user-permissions>'
         }
       }
     })
@@ -121,7 +121,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userPermissionsAdd></userPermissionsAdd>'
+          template: '<user-permissions-add></user-permissions-add>'
         }
       },
       params: {
@@ -135,7 +135,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userPermissionsEdit></userPermissionsEdit>'
+          template: '<user-permissions-edit></user-permissions-edit>'
         }
       },
       params: {
@@ -150,7 +150,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userRolesAdd></userRolesAdd>'
+          template: '<user-roles-add></user-roles-add>'
         }
       },
       params: {
@@ -164,7 +164,7 @@ export function RoutesConfig ($stateProvider, $urlRouterProvider) {
       },
       views: {
         'main@app': {
-          template: '<userRolesEdit></userRolesEdit>'
+          template: '<user-roles-edit></user-roles-edit>'
         }
       },
       params: {

angular/index.components.js

@@ -21,17 +21,17 @@ import { RegisterFormComponent } from './app/components/register-form/register-f
 
 angular.module('app.components')
   .component('widgets', WidgetsComponent)
-  .component('userprofile', UserProfileComponent)
+  .component('userProfile', UserProfileComponent)
   .component('userVerification', UserVerificationComponent)
-  .component('comingsoon', ComingSoonComponent)
-  .component('useredit', UserEditComponent)
-  .component('userpermissionsedit', UserPermissionsEditComponent)
-  .component('userpermissionsadd', UserPermissionsAddComponent)
-  .component('userpermissions', UserPermissionsComponent)
-  .component('userrolesedit', UserRolesEditComponent)
-  .component('userrolesadd', UserRolesAddComponent)
-  .component('userroles', UserRolesComponent)
-  .component('userlists', UserListsComponent)
+  .component('comingSoon', ComingSoonComponent)
+  .component('userEdit', UserEditComponent)
+  .component('userPermissionsEdit', UserPermissionsEditComponent)
+  .component('userPermissionsAdd', UserPermissionsAddComponent)
+  .component('userPermissions', UserPermissionsComponent)
+  .component('userRolesEdit', UserRolesEditComponent)
+  .component('userRolesAdd', UserRolesAddComponent)
+  .component('userRoles', UserRolesComponent)
+  .component('userLists', UserListsComponent)
   .component('dashboard', DashboardComponent)
   .component('navSidebar', NavSidebarComponent)
   .component('navHeader', NavHeaderComponent)

tests/PasswordResetTest.php

@@ -86,13 +86,12 @@ private function checkEmailContent($checks)
 
         $mock->shouldReceive('send')
         ->withArgs([Mockery::on(function ($message) use ($checks) {
+            $this->assertEquals($checks['title'], $message->getSubject());
+            $this->assertSame([$checks['email'] => null], $message->getTo());
+            $this->assertContains($checks['content'], $message->getBody());
 
-                $this->assertEquals($checks['title'], $message->getSubject());
-                $this->assertSame([$checks['email'] => null], $message->getTo());
-                $this->assertContains($checks['content'], $message->getBody());
-
-                return true;
-            }), Mockery::any()])
-            ->once();
+            return true;
+        }), Mockery::any()])
+        ->once();
     }
 }

tests/angular/app/components/forgot-password.component.spec.js

@@ -11,6 +11,14 @@ ngDescribe({
     }
   },
   tests: function (deps) {
+    it('Should have email', () => {
+      var inputs = deps.element.find('input')
+      expect(inputs.length).toBe(1)
+
+      var email = deps.element.find('input')[0]
+      expect(email.attributes['type'].value).toBe('email')
+    })
+
     it('should request email verification successfully', () => {
       var component = deps.element.isolateScope().vm
 

tests/angular/app/components/reset-password.component.spec.js

@@ -16,8 +16,15 @@ ngDescribe({
     }
   },
   tests: function (deps) {
-    it('should expect verification on init', () => {
-      //
+    it('Should have password and password confirmation', () => {
+      var inputs = deps.element.find('input')
+      expect(inputs.length).toBe(2)
+
+      var password = deps.element.find('input')[0]
+      expect(password.attributes['type'].value).toBe('password')
+
+      var password_confirmation = deps.element.find('input')[1]
+      expect(password_confirmation.attributes['type'].value).toBe('password')
     })
 
     it('should submit password reset successfully', () => {

tests/angular/app/components/routeCssClassnames.component.spec.js

@@ -1,7 +1,7 @@
 ngDescribe({
   name: 'Test routeCssClassnames component',
   modules: 'app',
-  element: '<routeCssClassnames></routeCssClassnames>',
+  element: '<route-bodyclass></route-bodyclass>',
   tests: function (deps) {
     it('basic test', () => {
       //

tests/angular/app/components/user-edit.component.spec.js

@@ -2,9 +2,26 @@ ngDescribe({
   name: 'Test user-edit component',
   modules: 'app',
   element: '<user-edit></user-edit>',
+  http: {
+    get: {
+      '/api/users/roles': {
+        data: true
+      },
+      '/api/users/show': {
+        data: true
+      }
+    }
+  },
   tests: function (deps) {
-    it('basic test', () => {
-      //
+    it('Should have name, email', () => {
+      var inputs = deps.element.find('input')
+      expect(inputs.length).toBe(2)
+
+      var name = deps.element.find('input')[0]
+      expect(name.attributes['type'].value).toBe('text')
+
+      var email = deps.element.find('input')[1]
+      expect(email.attributes['type'].value).toBe('email')
     })
   }
 })

tests/angular/app/components/user-lists.component.spec.js

@@ -2,9 +2,17 @@ ngDescribe({
   name: 'Test user-lists component',
   modules: 'app',
   element: '<user-lists></user-lists>',
+  http: {
+    get: {
+      '/api/users': {
+        data: true
+      }
+    }
+  },
   tests: function (deps) {
-    it('basic test', () => {
-      //
+    it('delete is a function', () => {
+      var component = deps.element.isolateScope().vm
+      la(typeof component.delete === 'function')
     })
   }
 })

tests/angular/app/components/user-permissions-add.component.spec.js

@@ -3,8 +3,15 @@ ngDescribe({
   modules: 'app',
   element: '<user-permissions-add></user-permissions-add>',
   tests: function (deps) {
-    it('basic test', () => {
-      //
+    it('Should have name, slug and description', () => {
+      var inputs = deps.element.find('input')
+      expect(inputs.length).toBe(2)
+
+      var name = deps.element.find('input')[0]
+      expect(name.attributes['type'].value).toBe('text')
+
+      var textarea = deps.element.find('textarea')
+      expect(textarea.length).toBe(1)
     })
   }
 })