Merge pull request #121 from hmrc/SAR-1042

Rename name for feature switch
hmrc · Nov 2, 2017 · 065da46 · 065da46
2 parents e578f18 + 6ae9fa9
commit 065da46
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/app/config/featureswitch/FeatureSwitch.scala b/app/config/featureswitch/FeatureSwitch.scala
@@ -21,7 +21,7 @@ import testonly.controllers.featureswitch.FeatureSwitchSetting
 
 sealed trait FeatureSwitch {
   val name: String
-  val shortName: String
+  val displayName: String
 }
 
 object FeatureSwitch {
@@ -36,13 +36,13 @@ object FeatureSwitch {
     }
 
   def apply(setting: FeatureSwitchSetting): FeatureSwitch =
-    switches find (_.shortName == setting.feature) match {
+    switches find (_.displayName == setting.feature) match {
       case Some(switch) => switch
       case None => throw new IllegalArgumentException("Invalid feature switch: " + setting.feature)
     }
 }
 
 object StubDESFeature extends FeatureSwitch {
-  val shortName = s"stub-des"
-  val name = s"$prefix.$shortName"
+  val displayName = s"Use stub for DES connection"
+  val name = s"$prefix.stub-des"
 }
diff --git a/app/testonly/controllers/featureswitch/FeatureSwitchSetting.scala b/app/testonly/controllers/featureswitch/FeatureSwitchSetting.scala
@@ -27,6 +27,6 @@ object FeatureSwitchSetting {
   implicit val format: OFormat[FeatureSwitchSetting] = Json.format[FeatureSwitchSetting]
 
   def apply(featureSwitch: FeatureSwitch, enable: Boolean): FeatureSwitchSetting =
-    FeatureSwitchSetting(featureSwitch.shortName, enable)
+    FeatureSwitchSetting(featureSwitch.displayName, enable)
 
 }