Merge pull request #27 from pat-karakun/master
replace hardcoded package names with (unused) parameters
This commit is contained in:
@@ -23,10 +23,10 @@ abstract class ArchitectureElement {
|
|||||||
static void denyDependency(String fromPackageName, String toPackageName, JavaClasses classes) {
|
static void denyDependency(String fromPackageName, String toPackageName, JavaClasses classes) {
|
||||||
noClasses()
|
noClasses()
|
||||||
.that()
|
.that()
|
||||||
.resideInAPackage("io.reflectoring.reviewapp.domain..")
|
.resideInAPackage(matchAllClassesInPackage(fromPackageName))
|
||||||
.should()
|
.should()
|
||||||
.dependOnClassesThat()
|
.dependOnClassesThat()
|
||||||
.resideInAnyPackage("io.reflectoring.reviewapp.application..")
|
.resideInAnyPackage(matchAllClassesInPackage(toPackageName))
|
||||||
.check(classes);
|
.check(classes);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -34,13 +34,7 @@ abstract class ArchitectureElement {
|
|||||||
List<String> fromPackages, List<String> toPackages, JavaClasses classes) {
|
List<String> fromPackages, List<String> toPackages, JavaClasses classes) {
|
||||||
for (String fromPackage : fromPackages) {
|
for (String fromPackage : fromPackages) {
|
||||||
for (String toPackage : toPackages) {
|
for (String toPackage : toPackages) {
|
||||||
noClasses()
|
denyDependency(fromPackage, toPackage, classes);
|
||||||
.that()
|
|
||||||
.resideInAPackage(matchAllClassesInPackage(fromPackage))
|
|
||||||
.should()
|
|
||||||
.dependOnClassesThat()
|
|
||||||
.resideInAnyPackage(matchAllClassesInPackage(toPackage))
|
|
||||||
.check(classes);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user