Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
TWiStErRob opened this issue Apr 29, 2024 · 3 comments · Fixed by #28975
Closed

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

TWiStErRob opened this issue Apr 29, 2024 · 3 comments · Fixed by #28975
Assignees
Labels
Milestone

Comments

@TWiStErRob
Copy link
Contributor

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
@TWiStErRob TWiStErRob added a:regression This used to work to-triage labels Apr 29, 2024
@ov7a
Copy link
Member

ov7a commented Apr 29, 2024

Sorry for that.

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

@ov7a ov7a self-assigned this Apr 29, 2024
@ov7a ov7a added in:testing and removed to-triage labels Apr 29, 2024
@ov7a ov7a added this to the 8.8 milestone Apr 29, 2024
@ov7a
Copy link
Member

ov7a commented Apr 29, 2024

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

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

@ljacomet ljacomet modified the milestones: 8.8, 8.8 RC2 May 7, 2024
@ov7a ov7a closed this as completed May 7, 2024
@TWiStErRob
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants