Removed logs to console

Carlos Purificação
1 parent 0716bd47
Showing 4 changed files with 61 additions and 13 deletions Show diff stats
src/app/environment/environment-home.component.ts
src/app/environment/environment.component.spec.ts
src/app/environment/environment.component.ts
src/lib/ng-noosfero-api/http/environment.service.ts
@@ -22,12 +22,9 @@ export class EnvironmentHomeComponent {
     environment: noosfero.Environment;
     constructor(private environmentService: EnvironmentService, private $sce: ng.ISCEService) {
-        console.debug("Constructor from EnvironmentHomeComponent");
-        
         environmentService.getByIdentifier("default").then((result: noosfero.Environment) => {
             this.environment = result;
-            console.debug("Environment ::", result);
-        })
+        });
     }
     getEnvironmentDescription() {
@@ -0,0 +1,60 @@
+import {quickCreateComponent} from "../../spec/helpers";
+import {EnvironmentComponent} from "./environment.component";
+
+describe("Components", () => {
+    describe("Environment Component", () => {
+
+        let $rootScope: ng.IRootScopeService;
+        let $q: ng.IQService;
+        let environmentServiceMock: any;
+        let notificationMock: any;
+        let $state: any;
+
+        beforeEach(inject((_$rootScope_: ng.IRootScopeService, _$q_: ng.IQService) => {
+            $rootScope = _$rootScope_;
+            $q = _$q_;
+        }));
+
+        beforeEach(() => {
+            $state = jasmine.createSpyObj("$state", ["transitionTo"]);
+            environmentServiceMock = jasmine.createSpyObj("environmentServiceMock", ["getByIdentifier", "getBoxes"]);
+            notificationMock = jasmine.createSpyObj("notificationMock", ["error"]);
+
+            let environmentResponse = $q.defer();
+            environmentResponse.resolve({ id: 1 });
+            let getBoxesResponse = $q.defer();
+            getBoxesResponse.resolve({ data: { boxes: [{ id: 2 }] } });
+
+            environmentServiceMock.getByIdentifier('default').and.returnValue(environmentResponse.promise);
+            environmentServiceMock.getBoxes = jasmine.createSpy("getBoxes").and.returnValue(getBoxesResponse.promise);
+        });
+        
+        it("get the default environment", done => {
+            let component: EnvironmentComponent = new EnvironmentComponent(environmentServiceMock, $state, notificationMock);
+            $rootScope.$apply();
+            expect(component.environment).toEqual({ id: 1 });
+            done();
+        });
+
+        it("get the environment boxes", done => {
+            let component: EnvironmentComponent = new EnvironmentComponent(environmentServiceMock, $state, notificationMock);
+            $rootScope.$apply();
+            expect(environmentServiceMock.getBoxes).toHaveBeenCalled();
+            expect(component.boxes).toEqual([{ id: 3 }]);
+            done();
+        });
+
+        it("display notification error when the environment wasn't found", done => {
+            let environmentResponse = $q.defer();
+            environmentResponse.reject();
+
+            let component: EnvironmentComponent = new EnvironmentComponent(environmentServiceMock, $state, notificationMock);
+            $rootScope.$apply();
+
+            expect(notificationMock.error).toHaveBeenCalled();
+            expect(component.environment).toBeUndefined();
+            done();
+        });
+
+    });
+});
@@ -38,13 +38,10 @@ export class EnvironmentComponent {
     environment: noosfero.Environment;
     constructor(environmentService: EnvironmentService, $state: ng.ui.IStateService, notificationService: NotificationService) {
-        //console.debug("Creating EnvironmentComponent...");
         let boxesPromisse = environmentService.getByIdentifier("default").then((environment: noosfero.Environment) => {
-            //console.debug("Set current environment by identifier callback.: ", environment);
             this.environment = environment;
             return environmentService.getBoxes(this.environment.id);
         }).then((boxes: noosfero.Box[]) => {
-            //console.debug("Set environment boxes in callback: ", boxes);
             this.boxes = boxes;
         }).catch(() => {
             $state.transitionTo('main');
@@ -19,10 +19,7 @@ export class EnvironmentService {
     }
     getByIdentifier(identifier: string): ng.IPromise<noosfero.Environment> {
-        console.debug("Getting the current environment by identifier in service: " + identifier);
         let p = this.restangular.one('environment').customGET(identifier);
-        console.debug("Return promise: ", p);
-                
         let deferred = this.$q.defer<noosfero.Environment>();
         p.then(this.getHandleSuccessFunction<noosfero.Environment>(deferred));
         p.catch(this.getHandleErrorFunction<noosfero.Environment>(deferred));
@@ -30,10 +27,7 @@ export class EnvironmentService {
     }
     getBoxes(id: number) {
-        console.debug("Getting the environment [${id}] boxes in service", id);
         let p = this.restangular.one('environments', id).customGET("boxes");
-        console.debug("Return boxes promise in service: ", p);
-                
         let deferred = this.$q.defer<noosfero.Box[]>();
         p.then(this.getHandleSuccessFunctionKeyArray<noosfero.Box[]>("boxes", deferred));
         p.catch(this.getHandleErrorFunction<noosfero.Box[]>(deferred));
@@ -0,0 +1,60 @@		@@ -0,0 +1,60 @@
	1	+import {quickCreateComponent} from "../../spec/helpers";
	2	+import {EnvironmentComponent} from "./environment.component";
	3	+
	4	+describe("Components", () => {
	5	+ describe("Environment Component", () => {
	6	+
	7	+ let $rootScope: ng.IRootScopeService;
	8	+ let $q: ng.IQService;
	9	+ let environmentServiceMock: any;
	10	+ let notificationMock: any;
	11	+ let $state: any;
	12	+
	13	+ beforeEach(inject((_$rootScope_: ng.IRootScopeService, _$q_: ng.IQService) => {
	14	+ $rootScope = _$rootScope_;
	15	+ $q = _$q_;
	16	+ }));
	17	+
	18	+ beforeEach(() => {
	19	+ $state = jasmine.createSpyObj("$state", ["transitionTo"]);
	20	+ environmentServiceMock = jasmine.createSpyObj("environmentServiceMock", ["getByIdentifier", "getBoxes"]);
	21	+ notificationMock = jasmine.createSpyObj("notificationMock", ["error"]);
	22	+
	23	+ let environmentResponse = $q.defer();
	24	+ environmentResponse.resolve({ id: 1 });
	25	+ let getBoxesResponse = $q.defer();
	26	+ getBoxesResponse.resolve({ data: { boxes: [{ id: 2 }] } });
	27	+
	28	+ environmentServiceMock.getByIdentifier('default').and.returnValue(environmentResponse.promise);
	29	+ environmentServiceMock.getBoxes = jasmine.createSpy("getBoxes").and.returnValue(getBoxesResponse.promise);
	30	+ });
	31	+
	32	+ it("get the default environment", done => {
	33	+ let component: EnvironmentComponent = new EnvironmentComponent(environmentServiceMock, $state, notificationMock);
	34	+ $rootScope.$apply();
	35	+ expect(component.environment).toEqual({ id: 1 });
	36	+ done();
	37	+ });
	38	+
	39	+ it("get the environment boxes", done => {
	40	+ let component: EnvironmentComponent = new EnvironmentComponent(environmentServiceMock, $state, notificationMock);
	41	+ $rootScope.$apply();
	42	+ expect(environmentServiceMock.getBoxes).toHaveBeenCalled();
	43	+ expect(component.boxes).toEqual([{ id: 3 }]);
	44	+ done();
	45	+ });
	46	+
	47	+ it("display notification error when the environment wasn't found", done => {
	48	+ let environmentResponse = $q.defer();
	49	+ environmentResponse.reject();
	50	+
	51	+ let component: EnvironmentComponent = new EnvironmentComponent(environmentServiceMock, $state, notificationMock);
	52	+ $rootScope.$apply();
	53	+
	54	+ expect(notificationMock.error).toHaveBeenCalled();
	55	+ expect(component.environment).toBeUndefined();
	56	+ done();
	57	+ });
	58	+
	59	+ });
	60	+});