gradle / gradle

Adaptable, fast automation for all
https://gradle.org
Apache License 2.0
16.19k stars 4.52k forks source link

All IDEA tests lost their display names, DX gone #28993

Closed TWiStErRob closed 1 week ago

TWiStErRob commented 2 weeks ago

Current Behavior

image


Additional problem, which might be a separate issue, but very likely related: failing tests never complete (on UI), Gradle build completed. image

Expected Behavior

image

Context (optional)

https://docs.gradle.org/8.8-rc-1/release-notes.html#ide-integration-improvements

It's great, but it shouldn't break all existing IDEs.

Steps to Reproduce

Bog standard @TestFactory with dynamicContainer and dynamicTest in a Kotlin JVM project. @Nested inner class and @ParameterizedTest are also affected.

I assume you have repros for this, because it's mentioned in the release notes, let me know if you need an example.

The very simple act of changing the wrapper version from 8.7 to 8.8-rc-1 and re-running the tests breaks the existing (good) behavior, therefore I raised it as regression.

Gradle version

Gradle 8.8-rc-1

Gradle version that used to work

Gradle 8.7

Build scan URL (optional)

No response

Your Environment (optional)

IntelliJ IDEA 2024.1 (Ultimate Edition)
Build #IU-241.14494.240, built on March 28, 2024
Runtime version: 17.0.10+8-b1207.12 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 6144M
Cores: 28
Registry:
  ide.windowSystem.autoShowProcessPopup=true
  ide.tooltip.initialDelay=105
  debugger.valueTooltipAutoShowOnSelection=true
  debugger.click.disable.breakpoints=true
  terminal.use.conpty.on.windows=false
  external.system.auto.import.disabled=true
  android.gradle.project.startup.activity.disabled=true
  android.sdk.local.properties.update.disabled=true
  unknown.sdk.auto=false
  maven.create.dummy.module.on.first.import=false
Non-Bundled Plugins:
  detekt (2.4.0)
  com.karateca.jasmineDescriber (3.5)
  com.jetbrains.colorful.toolwindow.icons (1.0)
  CMD Support (1.0.5)
  org.intellij.productivity-log (0.1.1)
  com.afcastano.intellij.autovalue (1.0.5)
  com.github.droibit.plugin.rxjava.postfix (0.1.0)
  com.intellij.mermaid (0.0.21+IJ.232)
  GrepConsole (12.25.0-IJ2023.3)
  com.github.adrienpessu.sarifviewer (1.2.3)
  com.github.copilot (1.5.2.5345)
  com.intellij.lang.jsgraphql (241.14494.150)
  MetricsReloaded (1.12)
  org.jetbrains.debugger.streams.kotlin (0.0.3)
  org.jetbrains.android (241.14494.240)
  com.android.tools.design (241.14494.158)
  com.dmarcotte.handlebars (241.14494.150)
  mobi.hsz.idea.gitignore (4.5.3)
  com.intellij.plugins.html.instantEditing (241.14494.140)
  com.jetbrains.php (241.14494.240)
  org.exbin.deltahex.intellij (0.2.10)
Kotlin: 241.14494.240-IJ
ov7a commented 2 weeks ago

Sorry for that.

This is a known issue that is supposed to be fixed in 8.8-RC2 by

ov7a commented 2 weeks ago

As for "failing tests never complete", see this issue:

it's not 100% the same, but closest to what we observe.

TWiStErRob commented 6 days ago

Thanks for the fix. The spinning one might be slightly different, but will have to see. My code doesn't have before/after. It might be related to two dynamic containers having the same name. Will watch out if I see it without this issue present.