Release 0.6.3

Use `List` for hooks to maintain order

Signed-off-by: Gopal S Akshintala <[email protected]>

Author: overfullstack

.idea/misc.xml

@@ -18,7 +18,7 @@ endif::[]
 :pmtemplates: src/integrationTest/resources/pm-templates
 :imagesdir: docs/images
 :prewrap!:
-:revoman-version: 0.6.2
+:revoman-version: 0.6.3
 
 '''
 

README.adoc

@@ -6,6 +6,6 @@
  * ************************************************************************************************
  */
 const val GROUP_ID = "com.salesforce.revoman"
-const val VERSION = "0.6.2"
+const val VERSION = "0.6.3"
 const val ARTIFACT_ID = "revoman"
 const val STAGING_PROFILE_ID = "1ea0a23e61ba7d"

buildSrc/src/main/kotlin/Config.kt

@@ -129,7 +129,7 @@ public void accept(@NotNull StepReport stepReport, @NotNull Rundown rundown) {
 										post(afterStepContainingHeader("postLog"), postLogHook))
 								.dynamicEnvironment(dynamicEnvironment)
 								.off());
-    
+
 		assertThat(pokeRundown.firstUnIgnoredUnsuccessfulStepReport().failure)
 				.containsOnLeft(new PostStepHookFailure(RUNTIME_EXCEPTION));
 		assertThat(pokeRundown.stepReports).hasSize(5);

src/integrationTest/java/com/salesforce/revoman/integration/pokemon/PokemonTest.java

@@ -54,7 +54,7 @@ internal interface KickDef {
 
   fun skipSteps(): List<ExeStepPick>
 
-  fun hooks(): Set<HookConfig>
+  fun hooks(): List<HookConfig>
 
   @Value.Derived fun preHooks(): List<HookConfig> = hooks().filter { it.pick is PreTxnStepPick }
 

src/main/kotlin/com/salesforce/revoman/input/config/KickDef.kt

@@ -33,7 +33,8 @@ import org.http4k.format.withStandardMappings
 open class MoshiReVoman(builder: Moshi.Builder) {
   private var moshi = builder.build()
 
-  @get:JvmName("internalMoshiCopy") val moshiCopy: Moshi by lazy { moshi.newBuilder().build() }
+  @get:JvmName("internalMoshiCopy")
+  val internalMoshiCopy: Moshi by lazy { moshi.newBuilder().build() }
 
   @Synchronized
   fun addAdapters(
@@ -51,9 +52,6 @@ open class MoshiReVoman(builder: Moshi.Builder) {
   fun <PojoT : Any> lenientAdapter(targetType: Type): JsonAdapter<PojoT> =
     adapter<PojoT>(targetType).lenient()
 
-  private inline fun <reified PojoT : Any> lenientAdapter(): JsonAdapter<PojoT> =
-    moshi.adapter<PojoT>(PojoT::class.java).lenient()
-
   fun <PojoT : Any> fromJson(input: String?, targetType: Type = Any::class.java): PojoT? =
     input?.let { lenientAdapter<PojoT>(targetType).fromJson(it) }
 

src/main/kotlin/com/salesforce/revoman/internal/json/MoshiReVoman.kt

@@ -20,8 +20,7 @@ data class PostmanEnvironment<ValueT : Any?>
 @JvmOverloads
 constructor(
   val mutableEnv: MutableMap<String, ValueT> = mutableMapOf(),
-  @get:JvmName("moshiReVoman")
-  val moshiReVoman: MoshiReVoman = initMoshi(),
+  @get:JvmName("moshiReVoman") val moshiReVoman: MoshiReVoman = initMoshi(),
 ) : MutableMap<String, ValueT> by mutableEnv {
 
   @get:JvmName("immutableEnv") val immutableEnv: Map<String, ValueT> by lazy { mutableEnv.toMap() }