diff options
author | Young Gyu Park <younggyu@google.com> | 2018-10-01 20:37:19 -0700 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2018-10-01 20:37:19 -0700 |
commit | 67063532ead2758c662051446e20c6c12b91a1dd (patch) | |
tree | ca1a2d505bdd155935a4182c3c2a00682d6b70e2 | |
parent | 21765143627e91fb1f9b4c022a5e45421f227b96 (diff) | |
parent | 70e4836af7373b0908809916eea811d00ce4610e (diff) | |
download | dashboard-67063532ead2758c662051446e20c6c12b91a1dd.tar.gz |
Merge "Bug fix for a badge color according to percentage."
am: 70e4836af7
Change-Id: I367c939354c3bb4748a73611f230289c0006d95c
-rw-r--r-- | src/main/webapp/WEB-INF/jsp/show_coverage_overview.jsp | 4 | ||||
-rw-r--r-- | src/main/webapp/js/test_results.js | 43 |
2 files changed, 35 insertions, 12 deletions
diff --git a/src/main/webapp/WEB-INF/jsp/show_coverage_overview.jsp b/src/main/webapp/WEB-INF/jsp/show_coverage_overview.jsp index 7d4e32a..6641b7e 100644 --- a/src/main/webapp/WEB-INF/jsp/show_coverage_overview.jsp +++ b/src/main/webapp/WEB-INF/jsp/show_coverage_overview.jsp @@ -404,10 +404,10 @@ <c:choose> <c:when test = "${covPct <= 20}"> - <c:set var="badgeColor" value="green" /> + <c:set var="badgeColor" value="red" /> </c:when> <c:when test = "${covPct >= 70}"> - <c:set var="badgeColor" value="yellow" /> + <c:set var="badgeColor" value="green" /> </c:when> <c:otherwise> <c:set var="badgeColor" value="orange" /> diff --git a/src/main/webapp/js/test_results.js b/src/main/webapp/js/test_results.js index 2811b0f..24d4c0b 100644 --- a/src/main/webapp/js/test_results.js +++ b/src/main/webapp/js/test_results.js @@ -67,6 +67,32 @@ function getNickname(testCaseResult) { } /** + * Get the badge color from ratio value. + * + * @param the percentage value. + * @returns the string of color for the badge. + */ +function getBadgeColor(ratio) { + var color = "orange"; + if (ratio <= 20) { + color = "red"; + } else if (ratio >= 70) { + color = "green"; + } + return color; +} + +/** + * Get the rounded value. + * + * @param the percentage value. + * @returns the rounded value from percentage value. + */ +function getRoundValue(ratio) { + return Math.round(ratio * 1000) / 10; +} + +/** * Display test data in the body beneath a test run's metadata. * @param container The jquery object in which to insert the test metadata. * @param data The json object containing the columns to display. @@ -249,16 +275,11 @@ function displayTestMetadata(container, metadataList, showTestNames = false) { var url = ('/show_coverage?testName=' + test + '&startTime=' + startTime); var covered = metadata.testRun.coveredLineCount; var total = metadata.testRun.totalLineCount; - var covPct = Math.round(covered / total * 1000) / 10; - var color = 'red'; - if (covPct > 20 && covPct < 70) { - color = 'orange'; - } else if (covPct >= 70) { - color = 'green'; - } + var covPct = getRoundValue(covered / total); + var color = getBadgeColor(covPct); var coverage = ('Coverage: ' + covered + '/' + total + ' (' + covPct + '%)'); - createClickableIndicator(div, coverage, color, function() { + createClickableIndicator(div, coverage, color, function(evt) { window.location.href = url; return false; }); @@ -267,8 +288,10 @@ function displayTestMetadata(container, metadataList, showTestNames = false) { metadata.testRun.totalApiCount != undefined) { var covered = metadata.testRun.coveredApiCount; var total = metadata.testRun.totalApiCount; - var apiCoverage = ('API Coverage: ' + covered + '/' + total); - createClickableIndicator(div, apiCoverage, 'orange', function(evt) { + var covPct = getRoundValue(covered / total); + var color = getBadgeColor(covPct); + var apiCoverage = ('API Coverage: ' + covered + '/' + total + ' (' + covPct + '%)'); + createClickableIndicator(div, apiCoverage, color, function(evt) { $('#apiCoverageModal') .data('urlSafeKeyList', metadata.testRun.apiCoverageKeyList); $('#apiCoverageModal').modal('open'); |