From 316aab1c3ba9ee8dfcf4ab014dadc7e41aea64dc Mon Sep 17 00:00:00 2001 From: Matt Borland Date: Mon, 14 Dec 2015 08:17:12 -0700 Subject: [PATCH] Raise JS unit test coverage threshold As people have been raising the unit test coverage, we should make sure we're raising the thresholds as well. This keeps the new code from being lax in coverage. The metric used here was to set whole number percentage points that were more than one-half of a percentage point below the actual coverage. Change-Id: I348d4b8cafcf6c7d0dc8e770b66609984e89c20b Partial-Bug: 1506891 --- horizon/karma.conf.js | 8 ++++---- openstack_dashboard/karma.conf.js | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/horizon/karma.conf.js b/horizon/karma.conf.js index aaad05e9bb..caef3fb0c5 100644 --- a/horizon/karma.conf.js +++ b/horizon/karma.conf.js @@ -150,10 +150,10 @@ module.exports = function (config) { // Coverage threshold values. thresholdReporter: { - statements: 90, // target 100 - branches: 77, // target 100 - functions: 89, // target 100 - lines: 87 // target 100 + statements: 91, // target 100 + branches: 81, // target 100 + functions: 90, // target 100 + lines: 91 // target 100 } }); }; diff --git a/openstack_dashboard/karma.conf.js b/openstack_dashboard/karma.conf.js index bff15f4f2f..e63c88be0b 100644 --- a/openstack_dashboard/karma.conf.js +++ b/openstack_dashboard/karma.conf.js @@ -192,10 +192,10 @@ module.exports = function (config) { // Coverage threshold values. thresholdReporter: { - statements: 92, // target 100 - branches: 89, // target 100 + statements: 93, // target 100 + branches: 90, // target 100 functions: 92, // target 100 - lines: 92 // target 100 + lines: 93 // target 100 } }); };