Skip to content

Commit

Permalink
rename atrium-scala to atrium-scala2 and change package structure
Browse files Browse the repository at this point in the history
Don't prefix packages with scala but move it after ch.tutteli.atrium.
IMO it is not an extension like kotlin_1_3 and therefore should
not be at the end. Use scala2 as we are likely going to use atrium-scala
for scala 3 once it is out.
  • Loading branch information
robstoll committed Oct 20, 2020
1 parent 044edea commit 43cacae
Show file tree
Hide file tree
Showing 23 changed files with 25 additions and 19 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/java-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,9 @@ jobs:
run: ATRIUM_ANDROID_JAR="$PWD/android-jar-cache/android.jar" ./gradlew checkDexer
shell: bash

- name: composite build atrium-scala
- name: composite build atrium-scala2
run: ./gradlew build
working-directory: misc\tools\atrium-scala-test
working-directory: misc\tools\atrium-scala2-test

- name: Upload windows build code coverage
uses: codecov/codecov-action@v1
Expand Down
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
- set -e
- ./gradle/scripts/check-generateLogic-committed.sh
- ./gradlew buildAllWithoutJs
- misc/tools/atrium-scala-test/gradlew -p ./misc/tools/atrium-scala-test build
- misc/tools/atrium-scala2-test/gradlew -p ./misc/tools/atrium-scala2-test build
after_success: bash <(curl -s https://codecov.io/bash) -F current

- env: MODE=current_8
Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package ch.tutteli.atrium.api.fluent.en_GB.scala
package ch.tutteli.atrium.scala2.api.fluent.en_GB

import ch.tutteli.atrium.assertions.Assertion
import ch.tutteli.atrium.creating.Expect
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package ch.tutteli.atrium.api.fluent.en_GB.scala
package ch.tutteli.atrium.scala2.api.fluent.en_GB

import ch.tutteli.atrium.creating.Expect
import ch.tutteli.atrium.domain.creating.changers.PostFinalStep
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
package ch.tutteli.atrium.api.fluent.en_GB.scala
package ch.tutteli.atrium.scala2.api.fluent.en_GB

import scala.collection.generic.IsIterable
import ch.tutteli.atrium.creating.Expect
import java.lang.{Iterable => JIterable}

import scala.jdk.CollectionConverters._

object IsIterableHelpers {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
package ch.tutteli.atrium.api.fluent.en_GB.scala
package ch.tutteli.atrium.scala2.api.fluent.en_GB

import java.lang.{Iterable => JIterable}

import ch.tutteli.atrium.api.fluent.en_GB.scala.IsIterableHelpers._
import IsIterableHelpers._
import ch.tutteli.atrium.creating.Expect
import ch.tutteli.atrium.domain.builders.creating.IterableAssertionsBuilder

Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
package ch.tutteli.atrium.api.fluent.en_GB
package ch.tutteli.atrium.scala2.api.fluent

import ch.tutteli.atrium.creating.Expect
import ch.tutteli.atrium.domain.builders.ExpectImpl
import _root_.scala.annotation.unchecked.uncheckedVariance
import kotlin.reflect.KClass

package object scala extends ScalaKotlinConversions {
package object en_GB extends ScalaKotlinConversions {
import IsIterableHelpers._

@inline val ExpectImpl: ExpectImpl =
Expand All @@ -20,7 +20,7 @@ package object scala extends ScalaKotlinConversions {

}

package scala {
package en_GB {

import kotlin.jvm.internal.Reflection

Expand Down Expand Up @@ -105,4 +105,5 @@ package scala {
}

class KClassTag[T](val kClass: KClass[T]) extends AnyVal

}
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package ch.tutteli.atrium.api.fluent.en_GB.scala
package ch.tutteli.atrium.scala2.api.fluent.en_GB

import TestUtils._
import AnyAssertionsSpec._
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,12 @@
package ch.tutteli.atrium.api.fluent.en_GB.scala
package ch.tutteli.atrium.scala2.api.fluent.en_GB

import TestUtils._
import ch.tutteli.atrium.creating.Expect
import java.lang.{Iterable => JIterable}
import scala.reflect.ClassTag

import scala.reflect.ClassTag
import IterableFeatureAssertionsSpec._
import IsIterableHelpers._

// FIXME rewrite after fusing IterableFutureAssertionsSpec with IterableAssertionsSpecr
//class IterableFeatureAssertionsSpec
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package ch.tutteli.atrium.api.fluent.en_GB.scala
package ch.tutteli.atrium.scala2.api.fluent.en_GB

import ch.tutteli.atrium.creating.Expect

Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
rootProject.name = 'atrium-scala'
rootProject.name = 'atrium-scala2'

buildscript {
gradle.ext.tutteli_plugins_version = '0.32.2'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,11 @@ def dependsOnTask = { self, String projectName, String taskName ->
}

task build {
dependsOnTask(delegate, 'atrium-scala', 'build')
dependsOnTask(delegate, 'atrium-scala2', 'build')
}

task clean {
dependsOnTask(delegate, 'atrium-scala', 'clean')
dependsOnTask(delegate, 'atrium-scala2', 'clean')
}

task cleanAtrium {
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,4 @@ includeBuild('../../../') {
}
}
}
includeBuild '../../../atrium-scala/'
includeBuild '../../../atrium-scala2/'

0 comments on commit 43cacae

Please sign in to comment.