----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63337/ -----------------------------------------------------------
(Updated Oct. 27, 2017, 5:01 p.m.) Review request for Aurora and David McLaughlin. Changes ------- Change my approach slightly in the version. Instead on using `setInterval`, I use `setTimeout` which is more appropriate here! - Moved logic around and addressed review feedback. - Also added tests. Repository: aurora Description ------- # Polling updates page if in progress in UI This will update the job update page every 5 seconds if the update is in progress. It was also stop refreshing the page if the update is not in progress anymore. Diffs (updated) ----- ui/src/main/js/pages/Update.js c900269f0e47ccde616f021a7ecd51e13c57b734 ui/src/main/js/pages/__tests__/Update-test.js 570a999ccd60c646a1206941e968714e5457b1a4 Diff: https://reviews.apache.org/r/63337/diff/3/ Changes: https://reviews.apache.org/r/63337/diff/2-3/ Testing ------- Checked in vagrant with a tunneled live scheduler and it works. Current tests pass. ``` # rmotamedi@tw-mbp-rmotamedi:~/oss/aurora on git:ui-update-polling ? [13:31:26] ? ./gradlew ui:lint > Task :ui:lint > apache-aurora@1.0.0 lint /Users/rmotamedi/oss/aurora/ui > eslint src/main/js plugin/ --ext .js BUILD SUCCESSFUL in 15s 4 actionable tasks: 1 executed, 3 up-to-date # rmotamedi@tw-mbp-rmotamedi:~/oss/aurora on git:ui-update-polling ? [13:31:45] ? ./gradlew ui:test > Task :ui:test > apache-aurora@1.0.0 test /Users/rmotamedi/oss/aurora/ui > jest src/ plugin/ PASS src/main/js/pages/__tests__/Job-test.js PASS src/main/js/components/__tests__/InstanceViz-test.js PASS src/main/js/components/__tests__/ConfigDiff-test.js PASS src/main/js/components/__tests__/Diff-test.js PASS src/main/js/components/__tests__/InstanceHistoryItem-test.js PASS src/main/js/utils/__tests__/Update-test.js PASS src/main/js/components/__tests__/Pagination-test.js PASS src/main/js/pages/__tests__/Update-test.js PASS src/main/js/components/__tests__/UpdateList-test.js PASS src/main/js/components/__tests__/UpdateInstanceEvents-test.js PASS src/main/js/components/__tests__/JobStatus-test.js PASS src/main/js/pages/__tests__/Instance-test.js PASS src/main/js/components/__tests__/UpdateStatus-test.js PASS src/main/js/components/__tests__/UpdateDiff-test.js PASS src/main/js/components/__tests__/JobHistory-test.js PASS src/main/js/components/__tests__/TaskNeighbors-test.js PASS src/main/js/pages/__tests__/Updates-test.js PASS src/main/js/pages/__tests__/Home-test.js PASS src/main/js/components/__tests__/JobConfig-test.js PASS src/main/js/test-utils/__tests__/Builder-test.js PASS src/main/js/components/__tests__/JobList-test.js PASS src/main/js/components/__tests__/Breadcrumb-test.js PASS src/main/js/utils/__tests__/Task-test.js PASS src/main/js/components/__tests__/Tabs-test.js PASS src/main/js/components/__tests__/InstanceHistory-test.js PASS src/main/js/components/__tests__/TaskList-test.js PASS src/main/js/pages/__tests__/Jobs-test.js PASS src/main/js/components/__tests__/RoleQuota-test.js PASS src/main/js/utils/__tests__/Common-test.js PASS src/main/js/components/__tests__/StateMachine-test.js Test Suites: 30 passed, 30 total Tests: 134 passed, 134 total Snapshots: 0 total Time: 6.118s Ran all test suites matching /src\/|plugin\//i. BUILD SUCCESSFUL in 9s 2 actionable tasks: 1 executed, 1 up-to-date ``` Thanks, Reza Motamedi