Carlos Purificação · Ábner Oliveira · Ábner Oliveira · ABNER SILVA DE OLIVEIRA · Carlos Purificação · Carlos Purificação
Showing 5 changed files Show diff stats
src/app/article/article-default-view-component.spec.ts
src/app/article/article-default-view.component.ts
src/languages/en.json
src/languages/pt.json
src/spec/mocks.ts
@@ -19,6 +19,7 @@ describe(&quot;Components&quot;, () =&gt; {
     describe("Article Default View Component", () => {
         let helper: ComponentTestHelper<ArticleDefaultViewComponent>;
         const defaultViewTemplate: string = '<noosfero-default-article [article]="ctrl.article" [profile]="ctrl.profile"></noosfero-default-article>';
+        let notificationService = helpers.mocks.notificationService;
         let articleService: any = helpers.mocks.articleService;
         let article = <noosfero.Article>{
             id: 1,
@@ -29,7 +30,8 @@ describe(&quot;Components&quot;, () =&gt; {
         let state = <ng.ui.IStateService>jasmine.createSpyObj("state", ["go", "transitionTo"]);
         let providers = [
             provide('$state', { useValue: state }),
-            provide('ArticleService', { useValue: articleService })
+            provide('ArticleService', { useValue: articleService }),
+            helpers.createProviderToValue('NotificationService', notificationService),
         ].concat(helpers.provideFilters("translateFilter"));
         /**
@@ -67,13 +69,19 @@ describe(&quot;Components&quot;, () =&gt; {
          * Execute the delete method on the target component
          */
         function doDeleteArticle() {
+            // Create a mock for the notification service confirmation
+            spyOn(helper.component.notificationService, 'confirmation').and.callFake(function (params: Function) {
+
+            });
             // Create a mock for the ArticleService removeArticle method
-            spyOn(helper.component.articleService, 'removeArticle').and.callFake(function(param: noosfero.Article) {
+            spyOn(helper.component.articleService, 'removeArticle').and.callFake(function (param: noosfero.Article) {
                 return {
-                    catch: () => {}
+                    catch: () => { }
                 };
             });
             helper.component.delete();
+            expect(notificationService.confirmation).toHaveBeenCalled();
+            helper.component.doDelete();
             expect(articleService.removeArticle).toHaveBeenCalled();
             // After the component delete method execution, fire the
             // ArticleEvent.removed event
@@ -81,9 +89,11 @@ describe(&quot;Components&quot;, () =&gt; {
         }
         /**
-         * Simulate the ArticleService ArticleEvent.removed event
+         * Simulate the Notification Service confirmation and ArticleService 
+         * notifyArticleRemovedListeners event
          */
         function simulateRemovedEvent() {
+            helper.component.notificationService["confirmation"]({ title: "Title", message: "Message" }, () => { });
             helper.component.articleService["notifyArticleRemovedListeners"](article);
         }
     });
@@ -5,6 +5,7 @@ import {MacroDirective} from &quot;./macro/macro.directive&quot;;
 import {ArticleToolbarHotspotComponent} from "../hotspot/article-toolbar-hotspot.component";
 import {ArticleContentHotspotComponent} from "../hotspot/article-content-hotspot.component";
 import {ArticleService} from "./../../lib/ng-noosfero-api/http/article.service";
+import { NotificationService } from "./../shared/services/notification.service";
 /**
  * @ngdoc controller
@@ -17,13 +18,13 @@ import {ArticleService} from &quot;./../../lib/ng-noosfero-api/http/article.service&quot;;
     selector: 'noosfero-default-article',
     templateUrl: 'app/article/article.html'
 })
-@Inject("$state", ArticleService)
+@Inject("$state", ArticleService, NotificationService)
 export class ArticleDefaultViewComponent {
     @Input() article: noosfero.Article;
     @Input() profile: noosfero.Profile;
-    constructor(private $state: ng.ui.IStateService, public articleService: ArticleService) {
+    constructor(private $state: ng.ui.IStateService, public articleService: ArticleService, public notificationService: NotificationService) {
         // Subscribe to the Article Removed Event
         this.articleService.subscribeToArticleRemoved((article: noosfero.Article) => {
             if (this.article.parent) {
@@ -31,15 +32,21 @@ export class ArticleDefaultViewComponent {
             } else {
                 this.$state.transitionTo('main.profile.info', { profile: this.article.profile.identifier });
             }
+            this.notificationService.success({ title: "article.remove.success.title", message: "article.remove.success.message" });
         });
     }
     delete() {
-        this.articleService.removeArticle(this.article).catch((cause: any) => {
-            throw new Error(`Problem removing the article: ${cause}`);
+        this.notificationService.confirmation({ title: "article.remove.confirmation.title", message: "article.remove.confirmation.message" }, () => {
+            this.doDelete();
         });
     }
+    doDelete() {
+        this.articleService.removeArticle(this.article).catch((cause: any) => {
+                throw new Error(`Problem removing the article: ${cause}`);
+        });        
+    }
 }
 /**
@@ -56,6 +56,10 @@
   "article.basic_editor.cancel": "Cancel",
   "article.basic_editor.success.title": "Good job!",
   "article.basic_editor.success.message": "Article saved!",
+  "article.remove.success.title": "Good job!",
+  "article.remove.success.message": "Article removed!",
+  "article.remove.confirmation.title": "Are you sure?",
+  "article.remove.confirmation.message": "You will not be able to recover this article!",
   "article.basic_editor.visibility": "Visibility",
   "article.basic_editor.visibility.public": "Public",
   "article.basic_editor.visibility.private": "Private",
@@ -53,6 +53,10 @@
   "article.basic_editor.cancel": "Cancelar",
   "article.basic_editor.success.title": "Bom trabalho!",
   "article.basic_editor.success.message": "Artigo salvo com sucesso!",
+  "article.remove.success.title": "Bom trabalho!",
+  "article.remove.success.message": "Artigo removido!",
+  "article.remove.confirmation.title": "Tem certeza?",
+  "article.remove.confirmation.message": "Não será possível recuperar este artigo!",  
   "article.basic_editor.visibility": "Visibilidade",
   "article.basic_editor.visibility.public": "Público",
   "article.basic_editor.visibility.private": "Privado",
@@ -192,6 +192,7 @@ export var mocks: any = {
         translate: (text: string) => { return text; }
     },
     notificationService: {
-        success: () => { }
+        success: () => { },
+        confirmation: () => { }
     }
 };
          * Execute the delete method on the target component
          */
         function doDeleteArticle() {
+            // Create a mock for the notification service confirmation
+            spyOn(helper.component.notificationService, 'confirmation').and.callFake(function (params: Function) {
             }
+            this.notificationService.success({ title: "article.remove.success.title", message: "article.remove.success.message" });
         });
     }
  
     delete() {
-        this.articleService.removeArticle(this.article).catch((cause: any) => {
-            throw new Error(`Problem removing the article: ${cause}`);
+        this.notificationService.confirmation({ title: "article.remove.confirmation.title", message: "article.remove.confirmation.message" }, () => {
+            this.doDelete();
         });
     }
  
+    doDelete() {
+        this.articleService.removeArticle(this.article).catch((cause: any) => {
+                throw new Error(`Problem removing the article: ${cause}`);
   "article.basic_editor.cancel": "Cancelar",
   "article.basic_editor.success.title": "Bom trabalho!",
   "article.basic_editor.success.message": "Artigo salvo com sucesso!",
+  "article.remove.success.title": "Bom trabalho!",
67	69	* Execute the delete method on the target component
68	70	*/
69	71	function doDeleteArticle() {
	72	+ // Create a mock for the notification service confirmation
	73	+ spyOn(helper.component.notificationService, 'confirmation').and.callFake(function (params: Function) {
2		Link here Michel Felipe @mfdeveloper 2016-05-24 19:28:35 UTC Não seria mais indicado utilizar `jasmine.createSpy("confirmation")` ou `jasmine.createSpyObj("notificationService", ["confirmation"])` como já está amplamente utilizado em outros testes? As diversas formas para implementar coisas muito semelhantes pode causar confusão no futuro. Chegou a conversar rapidamente com @abner sobre isso? Write Preview Não seria mais indicado utilizar `jasmine.createSpy("confirmation")` ou `jasmine.createSpyObj("notificationService", ["confirmation"])` como já está amplamente utilizado em outros testes? As diversas formas para implementar coisas muito semelhantes pode causar confusão no futuro. Chegou a conversar rapidamente com @abner sobre isso? Choose File ... File name... Cancel Link here Carlos Purificação @carloseugenio 2016-05-24 19:40:57 UTC Não sei direito qual a diferença. Como não estava conseguindo utilizar o jasmine.createSpy, vi isto na documentação do jasmine e utilizei. Write Preview Não sei direito qual a diferença. Como não estava conseguindo utilizar o jasmine.createSpy, vi isto na documentação do jasmine e utilizei. Choose File ... File name... Cancel
33	34	}
	35	+ this.notificationService.success({ title: "article.remove.success.title", message: "article.remove.success.message" });
34	36	});
35	37	}
36	38
37	39	delete() {
38		- this.articleService.removeArticle(this.article).catch((cause: any) => {
39		- throw new Error(`Problem removing the article: ${cause}`);
	40	+ this.notificationService.confirmation({ title: "article.remove.confirmation.title", message: "article.remove.confirmation.message" }, () => {
	41	+ this.doDelete();
40	42	});
41	43	}
42	44
	45	+ doDelete() {
	46	+ this.articleService.removeArticle(this.article).catch((cause: any) => {
	47	+ throw new Error(`Problem removing the article: ${cause}`);
3		Link here Michel Felipe @mfdeveloper 2016-05-24 19:30:31 UTC Se ocorrer um problema, essa mensagem será exibida para o usuário final? Write Preview Se ocorrer um problema, essa mensagem será exibida para o usuário final? Choose File ... File name... Cancel Link here Carlos Purificação @carloseugenio 2016-05-24 19:39:00 UTC Está conforme foi feito nos outros callbacks de serviços. Ach que tem ser feita uma revisão geral de como será o padrão de funcionamento das mensagens de erro. Write Preview Está conforme foi feito nos outros callbacks de serviços. Ach que tem ser feita uma revisão geral de como será o padrão de funcionamento das mensagens de erro. Choose File ... File name... Cancel Link here Ábner Oliveira @abner 2016-05-25 00:20:49 UTC Como esse catch aí é num componente de tela, então deveria estar utilizando o NotificationService para exibir a mensagem de erro para o usuário. Os serviços não devem realmente tratar a exibição de mensagens, mas os componentes de tela devem. Write Preview Como esse catch aí é num componente de tela, então deveria estar utilizando o NotificationService para exibir a mensagem de erro para o usuário. Os serviços não devem realmente tratar a exibição de mensagens, mas os componentes de tela devem. Choose File ... File name... Cancel
53	53	"article.basic_editor.cancel": "Cancelar",
54	54	"article.basic_editor.success.title": "Bom trabalho!",
55	55	"article.basic_editor.success.message": "Artigo salvo com sucesso!",
	56	+ "article.remove.success.title": "Bom trabalho!",
1		Link here Michel Felipe @mfdeveloper 2016-05-24 19:33:15 UTC As chaves `article.remove.success.title` e `article.remove.confirmation.title` são específicas da tela de artigo? Outras telas não relacionadas com artigo poderão utilizar essas chaves? Write Preview As chaves `article.remove.success.title` e `article.remove.confirmation.title` são específicas da tela de artigo? Outras telas não relacionadas com artigo poderão utilizar essas chaves? Choose File ... File name... Cancel