Merge pull request #25 from rcardin/18-narrow-the-scope-of-assertions-constructor

Fixed constructors scope

Author: rcardin

src/main/kotlin/in/rcard/assertj/arrowcore/AbstractEitherAssert.kt

@@ -23,8 +23,8 @@ import org.assertj.core.internal.StandardComparisonStrategy
  * @since 0.0.1
  */
 abstract class AbstractEitherAssert<
-    SELF : AbstractEitherAssert<SELF, LEFT, RIGHT>, LEFT : Any, RIGHT : Any,
-    >(
+        SELF : AbstractEitherAssert<SELF, LEFT, RIGHT>, LEFT : Any, RIGHT : Any,
+        > internal constructor(
     either: Either<LEFT, RIGHT>?,
 ) : AbstractObjectAssert<SELF, Either<LEFT, RIGHT>>(either, AbstractEitherAssert::class.java) {
 

src/main/kotlin/in/rcard/assertj/arrowcore/AbstractOptionAssert.kt

@@ -19,8 +19,8 @@ import org.assertj.core.internal.StandardComparisonStrategy
  * @since 0.0.1
  */
 abstract class AbstractOptionAssert<
-    SELF : AbstractOptionAssert<SELF, VALUE>, VALUE : Any,
-    >(
+        SELF : AbstractOptionAssert<SELF, VALUE>, VALUE : Any,
+        > internal constructor(
     option: Option<VALUE>?,
 ) : AbstractObjectAssert<SELF, Option<VALUE>>(option, AbstractOptionAssert::class.java) {
 

src/main/kotlin/in/rcard/assertj/arrowcore/AbstractRaiseAssert.kt

@@ -20,7 +20,7 @@ import org.assertj.core.internal.StandardComparisonStrategy
  */
 abstract class AbstractRaiseAssert<
         SELF : AbstractRaiseAssert<SELF, ERROR, VALUE>, ERROR : Any, VALUE : Any,
-        >(lambda: context(Raise<ERROR>) () -> VALUE) :
+        > internal constructor(lambda: context(Raise<ERROR>) () -> VALUE) :
     AbstractAssert<SELF, context(Raise<ERROR>) () -> VALUE>(lambda, AbstractRaiseAssert::class.java) {
 
     private val comparisonStrategy: ComparisonStrategy = StandardComparisonStrategy.instance()

src/main/kotlin/in/rcard/assertj/arrowcore/EitherAssert.kt

@@ -10,7 +10,7 @@ import arrow.core.Either
  * @author Riccardo Cardin
  * @since 0.0.1
  */
-class EitherAssert<LEFT : Any, RIGHT : Any>(either: Either<LEFT, RIGHT>?) :
+class EitherAssert<LEFT : Any, RIGHT : Any> private constructor(either: Either<LEFT, RIGHT>?) :
     AbstractEitherAssert<EitherAssert<LEFT, RIGHT>, LEFT, RIGHT>(either) {
     companion object {
         fun <LEFT : Any, RIGHT : Any> assertThat(actual: Either<LEFT, RIGHT>?): EitherAssert<LEFT, RIGHT> =

src/main/kotlin/in/rcard/assertj/arrowcore/OptionAssert.kt

@@ -9,7 +9,7 @@ import arrow.core.Option
  * @author Riccardo Cardin
  * @since 0.0.1
  */
-class OptionAssert<VALUE : Any>(option: Option<VALUE>?) :
+class OptionAssert<VALUE : Any> private constructor(option: Option<VALUE>?) :
     AbstractOptionAssert<OptionAssert<VALUE>, VALUE>(option) {
     companion object {
         fun <VALUE : Any> assertThat(option: Option<VALUE>?): OptionAssert<VALUE> =

src/main/kotlin/in/rcard/assertj/arrowcore/errors/EitherShouldBeLeft.kt

@@ -9,7 +9,7 @@ import org.assertj.core.error.BasicErrorMessageFactory
  * @author Riccardo Cardin
  * @since 0.0.1
  */
-internal class EitherShouldBeLeft(actual: Either<*, *>) :
+internal class EitherShouldBeLeft private constructor(actual: Either<*, *>) :
     BasicErrorMessageFactory("%nExpecting an Either to be left but was <$actual>.") {
     companion object {
         internal fun shouldBeLeft(actual: Either<*, *>): EitherShouldBeLeft = EitherShouldBeLeft(actual)

src/main/kotlin/in/rcard/assertj/arrowcore/errors/EitherShouldBeRight.kt

@@ -9,7 +9,7 @@ import org.assertj.core.error.BasicErrorMessageFactory
  * @author Riccardo Cardin
  * @since 0.0.1
  */
-internal class EitherShouldBeRight(actual: Either<*, *>) :
+internal class EitherShouldBeRight private constructor(actual: Either<*, *>) :
     BasicErrorMessageFactory("%nExpecting an Either to be right but was <$actual>.") {
     companion object {
         internal fun shouldBeRight(actual: Either<*, *>): EitherShouldBeRight = EitherShouldBeRight(actual)

src/main/kotlin/in/rcard/assertj/arrowcore/errors/EitherShouldContain.kt

@@ -9,7 +9,7 @@ import org.assertj.core.error.BasicErrorMessageFactory
  * @author Riccardo Cardin
  * @since 0.0.1
  */
-internal class EitherShouldContain(message: String, actual: Either<Any, Any>, expected: Any) :
+internal class EitherShouldContain private constructor(message: String, actual: Either<Any, Any>, expected: Any) :
     BasicErrorMessageFactory(message, actual, expected) {
 
     companion object {

src/main/kotlin/in/rcard/assertj/arrowcore/errors/EitherShouldContainInstanceOf.kt

@@ -10,7 +10,7 @@ import org.assertj.core.error.BasicErrorMessageFactory
  * @author Riccardo Cardin
  * @since 0.0.1
  */
-internal class EitherShouldContainInstanceOf(message: String) : BasicErrorMessageFactory(message) {
+internal class EitherShouldContainInstanceOf private constructor(message: String) : BasicErrorMessageFactory(message) {
     companion object {
         private const val EXPECTING_TO_CONTAIN_DIFFERENT_INSTANCE =
             "%nExpecting:%n <%s>%nto contain a value that is an instance of:%n <%s>%nbut did contain an instance of:%n <%s>"

src/main/kotlin/in/rcard/assertj/arrowcore/errors/OptionShouldBeEmpty.kt

@@ -9,7 +9,7 @@ import org.assertj.core.error.BasicErrorMessageFactory
  * @author Riccardo Cardin
  * @since 0.0.1
  */
-internal class OptionShouldBeEmpty(expected: Option<*>) :
+internal class OptionShouldBeEmpty private constructor(expected: Option<*>) :
     BasicErrorMessageFactory("%nExpecting an Option to be empty but was <%s>.", expected.orNull()) {
     companion object {
         internal fun shouldBeEmpty(actual: Option<*>): OptionShouldBeEmpty =