----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62958/#review187914 -----------------------------------------------------------
Master (2aee90d) is red with this patch. ./build-support/jenkins/build.sh :licenseTest UP-TO-DATE :license UP-TO-DATE :pmdJmh :pmdMain :pmdTest :testI1013 00:50:54.681 [ShutdownHook, SchedulerMain] Stopping scheduler services. FAILED :jacocoTestReport Coverage report generated: file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/jacoco/test/html/index.html :jacocoTestCoverageVerification[ant:jacocoReport] Rule violated for bundle aurora: instructions covered ratio is 0.76, but expected minimum is 0.87 [ant:jacocoReport] Rule violated for bundle aurora: branches covered ratio is 0.65, but expected minimum is 0.79 FAILED FAILURE: Build completed with 2 failures. 1: Task failed with an exception. ----------- * What went wrong: Execution failed for task ':test'. > Process 'Gradle Test Executor 6' finished with non-zero exit value 137 * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. ============================================================================== 2: Task failed with an exception. ----------- * What went wrong: Execution failed for task ':jacocoTestCoverageVerification'. > Rule violated for bundle aurora: instructions covered ratio is 0.76, but > expected minimum is 0.87 Rule violated for bundle aurora: branches covered ratio is 0.65, but expected minimum is 0.79 * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. ============================================================================== * Get more help at https://help.gradle.org BUILD FAILED in 10m 16s 48 actionable tasks: 39 executed, 9 up-to-date I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Oct. 13, 2017, 12:10 a.m., David McLaughlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62958/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2017, 12:10 a.m.) > > > Review request for Aurora, Kai Huang, Reza Motamedi, and Santhosh Kumar > Shanmugham. > > > Repository: aurora > > > Description > ------- > > This maintains the existing UI functionality where clicking on tabs updates > the browser location, to create shareable URLs to tab views. > > > Diffs > ----- > > ui/package.json cde8d106346d9ac498cfee9b5291ebc637fc6a2a > ui/src/main/js/components/Tabs.js 43b1950b11b80dc3017730801992341bc527c39c > ui/src/main/js/components/__tests__/Tabs-test.js > e028c2dd86739ed9762aa1d0be5c609d5487e06e > ui/src/main/js/pages/Job.js fc400f7442a1f8a5f0ebfe366dfe40ef40e7108e > ui/src/main/js/pages/__tests__/Job-test.js > 4cc76b8731d71ceca87a5d1e259360b2af8feba0 > > > Diff: https://reviews.apache.org/r/62958/diff/1/ > > > Testing > ------- > > ./gradlew ui:lint > ./gradlew ui:test > > Unfortunately the testing here is only on one side (restoring the URL state). > I cannot simulate the change events from the unit tests because of > limitations with shallow rendering. When I figure out how to add integration > tests that work with React Router, I'll add coverage. > > > Thanks, > > David McLaughlin > >
