Peter Gafert comments

Results 223 comments of


                                            Peter Gafert

run tests with JUnit Platform

I've compared the Gradle test xml report of all the executed tests before and after. From what I see the output coincides for both standard and slow tests (except for...

How can I find out which gradle module a class belongs to

@GeeJoe I'm wondering, if you have two Gradle modules and one class in each, why would both end up in the same build output folder afterwards? That doesn't seem normal...

@AnalyzeClasses breaks JUnit 5 tag filtering

Glad you like ArchUnit 😄 `@Tag` is actually from the JUnit Jupiter API, while ArchUnit is a completely different JUnit 5 engine that just builds on the JUnit platform. As...

@AnalyzeClasses breaks JUnit 5 tag filtering

Did this solve your problem?

@AnalyzeClasses breaks JUnit 5 tag filtering

Closing this for now, feel free to reopen if your issue persists!

Enum classes not recognized as enums and with with wrong access modifiers

> This looks like a bug in `JavaPackage.getClasses()` AFAIR this is by design, you can argue if it's the best design of course 🤔 Cause at some point we decided...

fix `OnionArchitecture` losing `optionalLayers` after `as`

> Thanks for fixing what I overlooked in #273! Well... 🤷‍♂️ "codecholeric approved these changes on Jan 9, 2020"

Add possibility to check for overriding methods to domain and lang

Thanks a lot for your contribution and sorry for the slow process 🙈 I see that there is another PR going in the same direction, but from my point of...

Package information is lost in the Surefire XML reports when using JUnit 5 with Maven

If the report is correct using Gradle, but broken using the Maven Surefire plugin, doesn't that point to a bug in the Maven Surefire plugin's JUnit 5 support? 🤔 Why...

OnionArchitecture: "withOptionalLayers" and "because" in the same statement casues that withOptionalLayers has no effect

Thanks for raising the issue and providing the analysis!! You're right, this must have been an oversight when the flag was added 🙈 I tried to fix it in https://github.com/TNG/ArchUnit/pull/1226...