From 4cf787eb7bbecb24b79a27d208e08c42b91e9274 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20W=C3=B3jcik?= Date: Wed, 26 May 2021 11:20:36 +0200 Subject: [PATCH 1/7] Working on pager --- .../app/js/views/failed_messages/controller.js | 16 ++++++---------- .../failed_messages/failed-messages-view.html | 8 +++++++- .../app/modules/shell/views/events-view.html | 2 +- 3 files changed, 14 insertions(+), 12 deletions(-) diff --git a/src/ServicePulse.Host/app/js/views/failed_messages/controller.js b/src/ServicePulse.Host/app/js/views/failed_messages/controller.js index a5f4e2327..45b845b3c 100644 --- a/src/ServicePulse.Host/app/js/views/failed_messages/controller.js +++ b/src/ServicePulse.Host/app/js/views/failed_messages/controller.js @@ -34,14 +34,13 @@ vm.selectedIds = []; vm.multiselection = {}; vm.sortButtonText = ''; - vm.allMessagesLoaded = false; vm.loadingData = false; vm.lastAction = selectActions.Selection; vm.page = 1; - vm.total = vm.stats.number_of_failed_messages; + vm.total = parseInt(vm.stats.number_of_failed_messages); notifier.subscribe($scope, function (event, data) { - vm.total = data; + vm.total = parseInt(data); }, 'MessageFailuresUpdated'); var setSortButtonText = function(sort, direction) { @@ -62,8 +61,6 @@ }); vm.failedMessages = vm.failedMessages.concat(exgroups); - vm.allMessagesLoaded = (vm.failedMessages.length >= vm.selectedExceptionGroup.count); - vm.page++; } vm.loadingData = false; @@ -277,7 +274,6 @@ vm.failedMessages = []; vm.selectedExceptionGroup = group; - vm.allMessagesLoaded = false; vm.page = 1; vm.loadMoreResults(group, sort, direction); @@ -287,14 +283,13 @@ selectGroupInternal(group, sort, direction, true); }; - vm.loadMoreResults = function(group, isInfiniteScrolling) { - vm.allMessagesLoaded = vm.failedMessages.length >= group.count; + vm.loadMoreResults = function(group) { - if (!group.initialLoad && (vm.allMessagesLoaded || vm.loadingData)) { + if (!group.initialLoad && vm.loadingData) { return; } - if (group.initialLoad && isInfiniteScrolling) { + if (group.initialLoad) { return; } @@ -314,6 +309,7 @@ group.count = response.total; } + vm.total = parseInt(response.total); processLoadedMessages(response.data); if (group.initialLoad) { diff --git a/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html b/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html index fd695934b..2703d238d 100644 --- a/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html +++ b/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html @@ -64,7 +64,7 @@

{{vm.selectedExceptionGroup.c -
+
@@ -101,6 +101,12 @@

{{vm.selectedExceptionGroup.c

+
+
+
    +
    +
    +
    diff --git a/src/ServicePulse.Host/app/modules/shell/views/events-view.html b/src/ServicePulse.Host/app/modules/shell/views/events-view.html index a674357f3..3d99ac1b5 100644 --- a/src/ServicePulse.Host/app/modules/shell/views/events-view.html +++ b/src/ServicePulse.Host/app/modules/shell/views/events-view.html @@ -65,7 +65,7 @@

    Events

    - +
    From bb6aecfed9328e6b9fa9d7f5797ae3183be429a5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20W=C3=B3jcik?= Date: Wed, 26 May 2021 12:58:43 +0200 Subject: [PATCH 2/7] Adding a pager to failed messages screen --- .../js/views/failed_messages/controller.js | 32 ++++++++++--------- .../failed_messages/failed-messages-view.html | 12 +++---- 2 files changed, 23 insertions(+), 21 deletions(-) diff --git a/src/ServicePulse.Host/app/js/views/failed_messages/controller.js b/src/ServicePulse.Host/app/js/views/failed_messages/controller.js index 45b845b3c..886a15485 100644 --- a/src/ServicePulse.Host/app/js/views/failed_messages/controller.js +++ b/src/ServicePulse.Host/app/js/views/failed_messages/controller.js @@ -36,11 +36,16 @@ vm.sortButtonText = ''; vm.loadingData = false; vm.lastAction = selectActions.Selection; - vm.page = 1; - vm.total = parseInt(vm.stats.number_of_failed_messages); + vm.pager = { + page: 1, + total: parseInt(vm.stats.number_of_failed_messages), + perPage: 50 + } + // vm.page = 1; + // vm.total = parseInt(vm.stats.number_of_failed_messages); notifier.subscribe($scope, function (event, data) { - vm.total = parseInt(data); + vm.pager.total = parseInt(data); }, 'MessageFailuresUpdated'); var setSortButtonText = function(sort, direction) { @@ -59,8 +64,8 @@ nObj.panel = 0; return nObj; }); - - vm.failedMessages = vm.failedMessages.concat(exgroups); + vm.selectedIds = []; + vm.failedMessages = exgroups; } vm.loadingData = false; @@ -93,7 +98,7 @@ vm.failedMessages = []; vm.selectedIds = []; - vm.page = 1; + vm.pager.page = 1; vm.sort = defaultSort.sort; vm.direction = defaultSort.direction; @@ -274,7 +279,7 @@ vm.failedMessages = []; vm.selectedExceptionGroup = group; - vm.page = 1; + vm.pager.page = 1; vm.loadMoreResults(group, sort, direction); }; @@ -285,11 +290,7 @@ vm.loadMoreResults = function(group) { - if (!group.initialLoad && vm.loadingData) { - return; - } - - if (group.initialLoad) { + if (vm.loadingData) { return; } @@ -299,9 +300,9 @@ var loadPromise; if (allExceptionsGroupSelected) { - loadPromise = serviceControlService.getFailedMessages(vm.sort, vm.page, vm.direction); + loadPromise = serviceControlService.getFailedMessages(vm.sort, vm.pager.page, vm.direction); } else { - loadPromise = serviceControlService.getFailedMessagesForExceptionGroup(group.id, vm.sort, vm.page, vm.direction); + loadPromise = serviceControlService.getFailedMessagesForExceptionGroup(group.id, vm.sort, vm.pager.page, vm.direction); } loadPromise.then(function (response) { @@ -309,7 +310,7 @@ group.count = response.total; } - vm.total = parseInt(response.total); + vm.pager.total = parseInt(response.total); processLoadedMessages(response.data); if (group.initialLoad) { @@ -317,6 +318,7 @@ } delete group.initialLoad; + $scope.$apply(); }); }; diff --git a/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html b/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html index 2703d238d..779828d20 100644 --- a/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html +++ b/src/ServicePulse.Host/app/js/views/failed_messages/failed-messages-view.html @@ -101,12 +101,12 @@

    {{vm.selectedExceptionGroup.c -
    -
    -
      -
      -
      - + + +
      +
      +
      +
        From 77a72e6d8baab921e276dd4e1d27c9c482fa559f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20W=C3=B3jcik?= Date: Wed, 26 May 2021 13:10:06 +0200 Subject: [PATCH 3/7] Replacing infinite scroll with pager for archives --- .../app/js/views/archive/archive-view.html | 8 +++++- .../app/js/views/archive/controller.js | 25 +++++++++++-------- 2 files changed, 22 insertions(+), 11 deletions(-) diff --git a/src/ServicePulse.Host/app/js/views/archive/archive-view.html b/src/ServicePulse.Host/app/js/views/archive/archive-view.html index 1977fb739..55fe7d75e 100644 --- a/src/ServicePulse.Host/app/js/views/archive/archive-view.html +++ b/src/ServicePulse.Host/app/js/views/archive/archive-view.html @@ -59,7 +59,7 @@

        {{vm.total | number}} message -
        +
        @@ -96,5 +96,11 @@

        {{vm.total | number}} message

        +
        +
        +
        +
          +
          +
          diff --git a/src/ServicePulse.Host/app/js/views/archive/controller.js b/src/ServicePulse.Host/app/js/views/archive/controller.js index 689d0e3f4..e07dd6a89 100644 --- a/src/ServicePulse.Host/app/js/views/archive/controller.js +++ b/src/ServicePulse.Host/app/js/views/archive/controller.js @@ -26,10 +26,15 @@ vm.selectedArchiveGroup = { 'id': $routeParams.groupId ? $routeParams.groupId : undefined, 'title': 'All deleted messages', 'count': 0, 'initialLoad': true }; + vm.pager = { + page: 1, + total: 1, + perPage: 50 + }; + vm.sort = { sortby: 'modified', direction: 'desc', - page: 1, start: undefined, end: undefined, buttonText: function () { @@ -66,9 +71,9 @@ return nObj; }); - vm.archives = vm.archives.concat(exgroups); + vm.selectedIds = []; + vm.archives = exgroups; vm.allMessagesLoaded = (vm.archives.length >= vm.total); - vm.sort.page++; } vm.loadingData = false; @@ -93,9 +98,9 @@ vm.configuration = sharedDataService.getConfiguration(); vm.error_retention_period = moment.duration(vm.configuration.data_retention.error_retention_period).asHours(); - vm.total = 1; + vm.pager.total = 1; vm.archives = []; - vm.sort.page = 1; + vm.pager.page = 1; var selectedArchiveGroup = getSelectedArchiveGroup(); @@ -209,8 +214,8 @@ vm.selectedIds = []; vm.archives = []; vm.allMessagesLoaded = false; - vm.total = 1; - vm.page = 1; + vm.pager.total = 1; + vm.pager.page = 1; vm.loadMoreResults(); }; @@ -258,7 +263,7 @@ }; vm.loadMoreResults = function () { - vm.allMessagesLoaded = vm.archives.length >= vm.total; + vm.allMessagesLoaded = vm.archives.length >= vm.pager.total; if (vm.allMessagesLoaded || vm.loadingData) { return; @@ -269,13 +274,13 @@ archivedMessageService.getArchivedMessages( vm.selectedArchiveGroup.id, vm.sort.sortby, - vm.sort.page, + vm.pager.page, vm.sort.direction, vm.sort.start, vm.sort.end).then(function (response) { notifier.notify('InitialLoadComplete'); - vm.total = response.total; + vm.pager.total = response.total; processLoadedMessages(response.data); }); }; From 9c22c3c75dfd8be4189161e078d9ee7716861edd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20W=C3=B3jcik?= Date: Wed, 26 May 2021 13:17:04 +0200 Subject: [PATCH 4/7] Adding a pager to custom checks --- .../custom_checks/customChecks.controller.js | 27 +++++++++---------- .../js/views/custom_checks/customChecks.html | 11 ++++++-- .../js/views/failed_messages/controller.js | 1 - 3 files changed, 22 insertions(+), 17 deletions(-) diff --git a/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.controller.js b/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.controller.js index 907b225f9..c7995b476 100644 --- a/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.controller.js +++ b/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.controller.js @@ -7,12 +7,15 @@ serviceControlService, notifyService) { - $scope.model = { data: [], total: 0 }; + $scope.model = { data: []}; $scope.loadingData = false; - $scope.disableLoadingData = false; $scope.reloadCount = 0; - var page = 1; + $scope.pager = { + page: 1, + total: 1, + perPage: 25 + } $scope.loadMoreResults = function () { if ($scope.loadingData) { @@ -20,7 +23,7 @@ } $scope.loadingData = true; - load(page++, $scope.reloadCount); + load($scope.pager.page, $scope.reloadCount); }; $scope.dismiss = function (row) { @@ -31,13 +34,13 @@ notifier.subscribe($scope, reloadData, 'CustomChecksUpdated'); function reloadData() { - page = 1; + $scope.pager.page = 1; + $scope.pager.total = 1; $scope.loadingData = true; - $scope.model = { data: [], total: 0 }; - $scope.disableLoadingData = false; + $scope.model = { data: [] }; $scope.reloadCount++; - load(page, $scope.reloadCount); + load($scope.pager.page, $scope.reloadCount); } function load(page, reloadCount) { @@ -50,12 +53,8 @@ $scope.loadingData = false; - $scope.model.data = $scope.model.data.concat(response.data); - $scope.model.total = response.total; - - if ($scope.model.data.length >= $scope.model.total) { - $scope.disableLoadingData = true; - } + $scope.model.data = response.data; + $scope.pager.total = response.total; }); } } diff --git a/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.html b/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.html index 069440604..994a0d3e3 100644 --- a/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.html +++ b/src/ServicePulse.Host/app/js/views/custom_checks/customChecks.html @@ -18,11 +18,11 @@

          Custom checks

          - +
          -
          +
          @@ -48,6 +48,13 @@

          Custom checks

          + +
          +
          +
          +
            +
            +
            \ No newline at end of file diff --git a/src/ServicePulse.Host/app/js/views/failed_messages/controller.js b/src/ServicePulse.Host/app/js/views/failed_messages/controller.js index 886a15485..78bb02e89 100644 --- a/src/ServicePulse.Host/app/js/views/failed_messages/controller.js +++ b/src/ServicePulse.Host/app/js/views/failed_messages/controller.js @@ -318,7 +318,6 @@ } delete group.initialLoad; - $scope.$apply(); }); }; From 5985d779b225e0eb001b7577c3db8370c343836f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20W=C3=B3jcik?= Date: Wed, 26 May 2021 14:07:07 +0200 Subject: [PATCH 5/7] Replace infinite scroll with pager --- src/ServicePulse.Host/app/js/app.constants.js | 2 +- .../js/views/pending_retries/controller.js | 34 +++++++++++-------- .../pending_retries/pending-retries-view.html | 9 ++++- 3 files changed, 28 insertions(+), 17 deletions(-) diff --git a/src/ServicePulse.Host/app/js/app.constants.js b/src/ServicePulse.Host/app/js/app.constants.js index d5fe34cde..b316b20d5 100644 --- a/src/ServicePulse.Host/app/js/app.constants.js +++ b/src/ServicePulse.Host/app/js/app.constants.js @@ -3,5 +3,5 @@ window.defaultConfig = { version: '1.2.0', service_control_url: 'http://localhost:33333/api/', monitoring_urls: ['http://localhost:33633/'], - showPendingRetry: false + showPendingRetry: true }; diff --git a/src/ServicePulse.Host/app/js/views/pending_retries/controller.js b/src/ServicePulse.Host/app/js/views/pending_retries/controller.js index 78d1f4d74..9e051de91 100644 --- a/src/ServicePulse.Host/app/js/views/pending_retries/controller.js +++ b/src/ServicePulse.Host/app/js/views/pending_retries/controller.js @@ -31,6 +31,13 @@ buttonText: 'All Pending Retries' }; + vm.pager = { + page: 1, + total: 1, + perPage: 50 + + }; + notifier.subscribe($scope, function (event, updatedTotalMessages) { if (vm.total !== updatedTotalMessages) { vm.total = updatedTotalMessages; @@ -107,9 +114,8 @@ return fillRedirect(nObj); }); - vm.pendingRetryMessages = vm.pendingRetryMessages.concat(exgroups); - vm.allMessagesLoaded = (vm.pendingRetryMessages.length >= vm.total); - vm.page++; + vm.selectedIds = []; + vm.pendingRetryMessages = exgroups; } vm.loadingData = false; }; @@ -118,13 +124,13 @@ vm.pendingRetryMessages = []; vm.selectedIds = []; vm.multiselection = {}; - vm.page = 1; + vm.pager.page = 1; vm.filter = { searchPhrase: undefined }; vm.filter.start = vm.filter.end = undefined; vm.total = sharedDataService.getstats().number_of_pending_retries; - vm.filteredTotal = vm.total; + vm.pager.total = vm.total; vm.sort = "time_of_failure"; vm.direction = "asc"; setSortButtonText(vm.sort, vm.direction); @@ -212,7 +218,7 @@ vm.retryAllConfirmationMessage = function() { if (!vm.isQueueFilterEmpty()) { - return "Are you sure you want to retry " + vm.filteredTotal + " out of " + vm.total + " previously retried messages? If the selected messages were processed in the meanwhile, then duplicate messages will be produced."; + return "Are you sure you want to retry " + vm.pager.total + " out of " + vm.total + " previously retried messages? If the selected messages were processed in the meanwhile, then duplicate messages will be produced."; } return "Bulk retry of messages can only be done for one queue at the time to avoid producing unwanted message duplicates."; @@ -239,7 +245,8 @@ vm.searchPhraseChanged = function() { vm.pendingRetryMessages = []; - vm.page = 1; + vm.pager.page = 1; + vm.pager.total = 1; vm.selectedIds = []; vm.multiselection = {}; vm.pendingRetryMessages.forEach(function (item) { @@ -257,7 +264,7 @@ vm.onSelect = function() { vm.pendingRetryMessages = []; - vm.page = 1; + vm.pager.page = 1; vm.loadMoreResults(); }; @@ -271,8 +278,7 @@ setSortButtonText(sort, direction); vm.pendingRetryMessages = []; - vm.allMessagesLoaded = false; - vm.page = 1; + vm.pager.page = 1; vm.loadTotalBasedOnFilters(); vm.loadMoreResults(sort, direction); @@ -313,20 +319,18 @@ vm.loadTotalBasedOnFilters = function() { pendingRetryService.getTotalPendingRetryMessages(vm.filter.searchPhrase ? vm.filter.searchPhrase.physical_address : '', vm.filter.start, vm.filter.end).then(function(response) { - vm.filteredTotal = response.total; + vm.pager.total = response.total; }); }; vm.loadMoreResults = function() { - vm.allMessagesLoaded = vm.pendingRetryMessages.length >= vm.total; - - if (vm.allMessagesLoaded || vm.loadingData) { + if (vm.loadingData) { return; } vm.loadingData = true; - pendingRetryService.getPendingRetryMessages(vm.filter.searchPhrase ? vm.filter.searchPhrase.physical_address : '', vm.sort, vm.page, vm.direction, vm.filter.start, vm.filter.end).then(function(response) { + pendingRetryService.getPendingRetryMessages(vm.filter.searchPhrase ? vm.filter.searchPhrase.physical_address : '', vm.sort, vm.pager.page, vm.direction, vm.filter.start, vm.filter.end).then(function(response) { processLoadedMessages(response.data); }); }; diff --git a/src/ServicePulse.Host/app/js/views/pending_retries/pending-retries-view.html b/src/ServicePulse.Host/app/js/views/pending_retries/pending-retries-view.html index c4dd65c79..b5d4aa7be 100644 --- a/src/ServicePulse.Host/app/js/views/pending_retries/pending-retries-view.html +++ b/src/ServicePulse.Host/app/js/views/pending_retries/pending-retries-view.html @@ -118,7 +118,7 @@ -
            +
            @@ -154,5 +154,12 @@
            + +
            +
            +
            +
              +
              +
              \ No newline at end of file From d6bba34721d77f551f4f201195e00c2e64d858fe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20W=C3=B3jcik?= Date: Fri, 28 May 2021 12:52:43 +0200 Subject: [PATCH 6/7] Revert the default value --- src/ServicePulse.Host/app/js/app.constants.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ServicePulse.Host/app/js/app.constants.js b/src/ServicePulse.Host/app/js/app.constants.js index b316b20d5..d5fe34cde 100644 --- a/src/ServicePulse.Host/app/js/app.constants.js +++ b/src/ServicePulse.Host/app/js/app.constants.js @@ -3,5 +3,5 @@ window.defaultConfig = { version: '1.2.0', service_control_url: 'http://localhost:33333/api/', monitoring_urls: ['http://localhost:33633/'], - showPendingRetry: true + showPendingRetry: false }; From e5e80331ad525a26213fa99cb075dc673f1f7e02 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20W=C3=B3jcik?= Date: Fri, 28 May 2021 14:10:40 +0200 Subject: [PATCH 7/7] Fixing tests, removing the old dependency --- .../views/failed_messages/controller.spec.js | 52 ------------------- .../views/pending_retries/controller.spec.js | 14 ++--- src/ServicePulse.Host/app/js/app.js | 1 - .../app/modules/shell/shell.js | 1 - src/ServicePulse.Host/package-lock.json | 5 -- src/ServicePulse.Host/package.json | 1 - 6 files changed, 7 insertions(+), 67 deletions(-) diff --git a/src/ServicePulse.Host.Tests/tests/js/views/failed_messages/controller.spec.js b/src/ServicePulse.Host.Tests/tests/js/views/failed_messages/controller.spec.js index 62bad145a..e30fc759f 100644 --- a/src/ServicePulse.Host.Tests/tests/js/views/failed_messages/controller.spec.js +++ b/src/ServicePulse.Host.Tests/tests/js/views/failed_messages/controller.spec.js @@ -66,56 +66,4 @@ expect(controller.selectedIds.length).toEqual(0); }); }); - - describe('when loading the data with infinite scroll', - function() { - var controller, serviceControlService, root, deferred, getFailedMessageSpy; - - beforeEach(inject(function ($rootScope, notifyService, $q) { - root = $rootScope; - this.notifyService = notifyService; - serviceControlService = { getExceptionGroups: function () { }, getFailedMessages: function(){ } }; - deferred = $q.defer(); - spyOn(serviceControlService, 'getExceptionGroups').and.callFake(function () { - - return deferred.promise; - }); - - getFailedMessageSpy = spyOn(serviceControlService, 'getFailedMessages').and.callFake(function () { - return deferred.promise; - }); - - controller = $controller('failedMessagesController', - { - $scope: root, - $timeout: null, - $interval: function(){}, - $location: null, - sharedDataService: { getstats: function() { return { number_of_pending_retries: 0 }; } }, - notifyService: notifyService, - serviceControlService: serviceControlService, - failedMessageGroupsService: null - }); - })); - - it('no load happens when initial load is in progress', - function () { - expect(getFailedMessageSpy).toHaveBeenCalledTimes(1); - - controller.loadMoreResults(controller.selectedExceptionGroup, true); - - expect(getFailedMessageSpy).toHaveBeenCalledTimes(1); - }); - - it('load happens when initial load is done', - function () { - root.$apply(function () { deferred.resolve({data: []}) }); - expect(getFailedMessageSpy).toHaveBeenCalledTimes(1); - - controller.loadMoreResults(controller.selectedExceptionGroup, true); - - expect(getFailedMessageSpy).toHaveBeenCalledTimes(2); - }); - - }); }); \ No newline at end of file diff --git a/src/ServicePulse.Host.Tests/tests/js/views/pending_retries/controller.spec.js b/src/ServicePulse.Host.Tests/tests/js/views/pending_retries/controller.spec.js index c04350721..b77e34821 100644 --- a/src/ServicePulse.Host.Tests/tests/js/views/pending_retries/controller.spec.js +++ b/src/ServicePulse.Host.Tests/tests/js/views/pending_retries/controller.spec.js @@ -17,7 +17,7 @@ $httpBackend.whenGET('http://localhost:33333/api/errors/queues/addresses').respond(null); $httpBackend.whenGET('http://localhost:33333/api/redirects').respond(null); - pendingRetryService = { retryPendingRetriedMessages: function () { } }; + pendingRetryService = { retryPendingRetriedMessages: function () { }, getPendingRetryMessages: function(){ return {then: function(){}}} }; controller = $controller('pendingRetriesController', { $scope: $scope, $timeout: null, @@ -40,7 +40,7 @@ controller.pendingRetryMessages = [{ id: 1, submittedForRetrial: false, selected: true }, { id: 2, submittedForRetrial: false, selected: false }, { id: 3, submittedForRetrial: false, selected: true }]; controller.selectedIds = [1, 3]; controller.retrySelected(); - + root.$apply(function () { deferred.resolve('Remote call result') }); expect(controller.selectedIds.length).toEqual(0); @@ -58,13 +58,13 @@ var $scope, controller, pendingRetryService, root, $httpBackend; beforeEach(inject(function ($rootScope, $injector) { - $scope = {}; + $scope = { }; root = $rootScope; $httpBackend = $injector.get('$httpBackend'); $httpBackend.whenGET('http://localhost:33333/api/errors/queues/addresses').respond(null); $httpBackend.whenGET('http://localhost:33333/api/redirects').respond(null); - pendingRetryService = { retryAllMessages: function () { } }; + pendingRetryService = { retryAllMessages: function () { }, getPendingRetryMessages: function(){ return {then: function(){}}} }; controller = $controller('pendingRetriesController', { $scope: $scope, $timeout: null, @@ -113,7 +113,7 @@ $httpBackend = $injector.get('$httpBackend'); $httpBackend.whenGET('http://localhost:33333/api/errors/queues/addresses').respond(null); $httpBackend.whenGET('http://localhost:33333/api/redirects').respond(null); - pendingRetryService = { markAsResolvedMessages: function () { } }; + pendingRetryService = { markAsResolvedMessages: function () { }, getPendingRetryMessages: function(){ return {then: function(){}}} }; controller = $controller('pendingRetriesController', { $scope: $scope, $timeout: null, @@ -158,7 +158,7 @@ $httpBackend = $injector.get('$httpBackend'); $httpBackend.whenGET('http://localhost:33333/api/errors/queues/addresses').respond(null); $httpBackend.whenGET('http://localhost:33333/api/redirects').respond(null); - pendingRetryService = { markAsResolvedAllMessages: function () { } }; + pendingRetryService = { markAsResolvedAllMessages: function () { }, getPendingRetryMessages: function(){ return {then: function(){}}} }; controller = $controller('pendingRetriesController', { $scope: $scope, $timeout: null, @@ -218,7 +218,7 @@ var message = { retried: true, number_of_processing_attempts: 1 }; var result = controller.noStatusPresent(message); - + expect(result).toEqual(false); })); diff --git a/src/ServicePulse.Host/app/js/app.js b/src/ServicePulse.Host/app/js/app.js index ab8d3fee3..9cf331ed7 100644 --- a/src/ServicePulse.Host/app/js/app.js +++ b/src/ServicePulse.Host/app/js/app.js @@ -11,7 +11,6 @@ 'toaster', 'toastService', 'ui.bootstrap', - 'infinite-scroll', 'services', 'ui.particular', 'ui.particular.reindexingstatus', diff --git a/src/ServicePulse.Host/app/modules/shell/shell.js b/src/ServicePulse.Host/app/modules/shell/shell.js index 95da3fa1b..4fc848250 100644 --- a/src/ServicePulse.Host/app/modules/shell/shell.js +++ b/src/ServicePulse.Host/app/modules/shell/shell.js @@ -11,7 +11,6 @@ require('jquery-csv'); require('zeroclipboard'); require('moment'); require('moment-duration-format'); -require('ng-infinite-scroll'); require('angularjs-toaster'); require('signalr'); require('ui-select'); diff --git a/src/ServicePulse.Host/package-lock.json b/src/ServicePulse.Host/package-lock.json index f3a53398d..70d1780d6 100644 --- a/src/ServicePulse.Host/package-lock.json +++ b/src/ServicePulse.Host/package-lock.json @@ -4348,11 +4348,6 @@ "integrity": "sha512-iyam8fBuCUpWeKPGpaNMetEocMt364qkCsfL9JuhjXX6dRnguRVOfk2GZaDpPjcOKiiXCPINZC1GczQ7iTq3Zw==", "dev": true }, - "ng-infinite-scroll": { - "version": "1.3.0", - "resolved": "https://registry.npmjs.org/ng-infinite-scroll/-/ng-infinite-scroll-1.3.0.tgz", - "integrity": "sha1-wumNj9E0sFJaTSz1jJXZtYN1URI=" - }, "ngstorage": { "version": "git://github.com/gsklee/ngStorage.git#3cd5ffaff77bebda910b94ca2a657ac44f4aac79", "from": "git://github.com/gsklee/ngStorage.git#0.3.10" diff --git a/src/ServicePulse.Host/package.json b/src/ServicePulse.Host/package.json index 76fe7da2f..0be2ae0fe 100644 --- a/src/ServicePulse.Host/package.json +++ b/src/ServicePulse.Host/package.json @@ -19,7 +19,6 @@ "jquery-csv": "^1.0.21", "moment": "^2.19.3", "moment-duration-format": "^1.3.0", - "ng-infinite-scroll": "^1.3.0", "ngstorage": "git://github.com/gsklee/ngStorage#0.3.10", "rx": "^4.1.0", "signalr": "^2.2.1",