test(reports): update Open Debtors e2e tests

This commit updates the end to end tests to work with the new report
layout.

client/src/i18n/fr/report.json

@@ -19,7 +19,9 @@
       "EXPLOITATION_RESULT": "Resultat d'exploitation"
     },
     "CASH_EXPENSE":"Depenses",
-    "CASHFLOW_BY_SERVICE": {"TITLE":"Rapports de flux de trésorerie par Service", "DESCRIPTION":"Ce rapport affiche les variations des comptes et des revenus du bilan sur la trésorerie"},
+    "CASHFLOW_BY_SERVICE": {
+      "TITLE":"Rapports de flux de trésorerie par Service", "DESCRIPTION":"Ce rapport affiche les variations des comptes et des revenus du bilan sur la trésorerie"
+    },
     "CASHFLOW":"Cashflow",
     "CASH_INCOME":"Recettes",
     "CHART_OF_ACCOUNTS":"Plan Comptable",
@@ -32,10 +34,16 @@
     "INCOME_EXPENSE":"Rapport  des recettes et des dépenses",
     "INCOME_REPORT":"Rapport des recettes",
     "MONTHLY_BALANCE":"Balance mensuelle",
-    "OPEN_DEBTORS":{
-      "DESCRIPTION":"Ce rapport affiche les débiteurs endettés",
+    "OPEN_DEBTORS": {
       "TITLE":"Débiteurs endettés",
-      "TREE":"Dettes des Débiteurs"
+      "TREE":"Dettes des Débiteurs",
+      "DESCRIPTION": "Ce rapport montre les débiteurs qui ont une dette avec l'institution et donne un certain contrôle sur l'ordre de tri. Pour une analyse avancée, le rapport peut ajouter la dernière date de facturation et la dernière date de paiement, ainsi que limiter la portée de la recherche par une date.",
+      "SHOW_DATE_LIMIT" : "Limité par Date",
+      "DATE_LIMIT_HELP" : "La limite de la date ne fera que numériser les enregistrements de cette date au présent.",
+      "SHOW_DETAILED_VIEW" : "Inclure les colonnes pour les dernières dates de facturation et de paiement",
+      "SHOW_DETAILED_VIEW_HELP" : "L'activation de cette option comprendra la dernière date à laquelle le débiteur a payé et la dernière date où ils ont été facturés. Ceci est utile pour comprendre quand ils étaient à l'établissement. Cependant, cela rendra la requête plus lent.",
+      "SHOW_UNVERIFIED_TRANSACTIONS" : "Inclure les transactions non vérifiées",
+      "SHOW_UNVERIFIED_TRANSACTIONS_HELP" : "L'activation de cette option comprend les enregistrements du Journal de Saisie qui n'ont pas été vérifiés par le comptable ainsi que les enregistrements du Grand Livre que le comptable a vérifié. Cette vue doit être traitée comme une projection jusqu'à ce que le comptable ait approuvé les transactions du journal et les a posté vers le Grand Livre."
     },
     "OPENING_BALANCE": "Balance d'ouverture",
     "ORDER": {

client/src/modules/reports/generate/open_debtors/open_debtors.html

@@ -32,20 +32,20 @@ <h3 class="text-capitalize" translate>REPORT.OPEN_DEBTORS.TITLE</h3>
                 name="order"
                 ng-model="ReportConfigCtrl.reportDetails.order">
                 <ui-select-match placeholder="{{ 'FORM.LABELS.ORDER_BY' | translate }}">
-                  <span>
+                  <span class="text-capitalize">
                     <i class="fa" ng-class="{
                       'fa-caret-down': $select.selected.direction === ReportConfigCtrl.DESC,
                       'fa-caret-up' : $select.selected.direction === ReportConfigCtrl.ASC
                       }" aria-hidden="true"></i>
-                    <strong translate class="text-capitalize">{{$select.selected.label}}</strong>
+                    {{$select.selected.hrLabel}}
                   </span>
                 </ui-select-match>
-                <ui-select-choices ui-select-focus-patch repeat="column.value as column in (ReportConfigCtrl.orders | filter:$select.search)">
+                <ui-select-choices ui-select-focus-patch repeat="column.value as column in ReportConfigCtrl.orders | filter: { 'hrLabel' : $select.search }">
                   <i class="fa" ng-class="{
                     'fa-caret-down': column.direction === ReportConfigCtrl.DESC,
                     'fa-caret-up' : column.direction === ReportConfigCtrl.ASC
                   }" aria-hidden="true"></i>
-                  <span ng-bind-html="column.label | translate | highlight:$select.search" class="text-capitalize"></span>
+                  <span ng-bind-html="column.hrLabel | highlight:$select.search" class="text-capitalize"></span>
                 </ui-select-choices>
               </ui-select>
               <div class="help-block" ng-messages="ConfigForm.order.$error" ng-show="ConfigForm.$submitted">

client/src/modules/reports/generate/open_debtors/open_debtors.service.js

@@ -1,14 +1,16 @@
 angular.module('bhima.services')
   .service('OpenDebtorsReportService', OpenDebtorsReportService);
 
+OpenDebtorsReportService.$inject = ['$translate'];
+
 /**
  * @class OpenDebtorsReportService
  *
  * @description
  * An object that contains keys/properties relevant to rendering the Open
  * Debtors Report.
  */
-function OpenDebtorsReportService() {
+function OpenDebtorsReportService($translate) {
   var service = this;
 
   var ASC = 1;
@@ -43,5 +45,10 @@ function OpenDebtorsReportService() {
     direction : DESC,
   }];
 
+  // ensure a human readable label
+  service.orders.forEach(function (order) {
+    order.hrLabel = $translate.instant(order.label);
+  });
+
   return service;
 }

client/src/modules/reports/reports.html

@@ -4,8 +4,10 @@
     <ol class="headercrumb">
       <li class="static" translate>TREE.REPORTS</li>
       <li ng-hide="ReportCtrl.isArchive()" class="title" translate>{{ ReportCtrl.report.title_key }}</li>
-      <li ng-show="ReportCtrl.isArchive()">
-        <a data-method="report-config" ui-sref="^.reportsBase.{{ReportCtrl.report.report_key}}" translate>{{ ReportCtrl.report.title_key }}</a>
+      <li ng-if="ReportCtrl.isArchive()">
+        <a data-method="report-config" ui-sref="^.reportsBase.{{ReportCtrl.report.report_key}}" translate>
+          {{ ReportCtrl.report.title_key }}
+        </a>
       </li>
       <li ng-show="ReportCtrl.isArchive()" class="title" translate>REPORT.UTIL.VIEW_ARCHIVE</li>
     </ol>

test/end-to-end/reports/open_debtors/open_debtors.page.js

@@ -1,5 +1,3 @@
-/* global browser, element, by */
-
 const chai = require('chai');
 const helpers = require('../../shared/helpers');
 
@@ -15,17 +13,22 @@ class OpenDebtorsReportPage {
   }
 
   // preview a OpenDebtors report
-  showOpenDebtorsReportPreview(column, orderBy) {
-    FU.uiSelect('ReportConfigCtrl.reportDetails.order', column);
+  showOpenDebtorsReportPreview(order) {
+    FU.uiSelect('ReportConfigCtrl.reportDetails.order', order);
+
+    const showUnverifiedTransactions = $('[name="showUnverifiedTransactions"]');
+    const showDetailedView = $('[name="showDetailedView"]');
 
-    element(by.id(orderBy)).click();
+    // enable the checkboxes
+    showUnverifiedTransactions.click();
+    showDetailedView.click();
 
     this.page.preview();
   }
 
   // save a OpenDebtors report
-  saveOpenDebtorsReport(column, orderBy, reportName, reportFormat) {
-    this.showOpenDebtorsReportPreview(column, orderBy);
+  saveOpenDebtorsReport(order, reportName, reportFormat) {
+    this.showOpenDebtorsReportPreview(order);
 
     // save report as PDF
     this.page.saveAs();
@@ -39,8 +42,8 @@ class OpenDebtorsReportPage {
   }
 
   // print a debtors report
-  printOpenDebtorsReport(column, orderBy, weekly, cashbox) {
-    this.showOpenDebtorsReportPreview(column, orderBy);
+  printOpenDebtorsReport(order) {
+    this.showOpenDebtorsReportPreview(order);
     this.page.printPreview();
   }
 
@@ -57,4 +60,4 @@ class OpenDebtorsReportPage {
   }
 }
 
-module.exports = OpenDebtorsReportPage;
+module.exports = OpenDebtorsReportPage;

test/end-to-end/reports/open_debtors/open_debtors.spec.js

@@ -1,4 +1,4 @@
-/* global browser, element, by */
+/* global browser */
 
 const chai = require('chai');
 const helpers = require('../../shared/helpers');
@@ -7,15 +7,14 @@ helpers.configure(chai);
 
 const ReportOpenDebtorsPage = require('./open_debtors.page');
 
-describe('Open Debtors Report ::', () => {
+describe('Open Debtors Report', () => {
   let Page;
   const key = 'open_debtors';
 
   const dataset = {
-    order : 'Total des dettes',
-    orderBy : 'desc',
-    report_name : 'Open Debtors Report, Order by Debts Desc',
-    renderer : 'PDF',    
+    order : 'Total',
+    report_name : 'Open Debtors Report, Order by Debts',
+    renderer : 'PDF',
   };
 
   before(() => {
@@ -24,23 +23,23 @@ describe('Open Debtors Report ::', () => {
     browser.refresh();
   });
 
-  it('preview a new Open Debtor Report: Order by balance Desc', () => {
-    Page.showOpenDebtorsReportPreview(dataset.order, dataset.orderBy);
+  it(`preview a new Open Debtors report - order by ${dataset.order}`, () => {
+    Page.showOpenDebtorsReportPreview(dataset.order);
   });
 
   it('close the previewed report', () => {
     Page.closeOpenDebtorsReportPreview();
   });
 
   it('save a previewed report', () => {
-    Page.saveOpenDebtorsReport(dataset.order, dataset.orderBy, dataset.report_name, dataset.renderer);
+    Page.saveOpenDebtorsReport(dataset.order, dataset.report_name, dataset.renderer);
   });
 
   it('report has been saved into archive', () => {
     Page.checkSavedOpenDebtorsReport(dataset.report_name);
   });
 
   it('print the previewed report', () => {
-    Page.printOpenDebtorsReport(dataset.order, dataset.orderBy);
+    Page.printOpenDebtorsReport(dataset.order);
   });
-});
+});