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

Cleanup tests using Java records #3261

Merged
merged 4 commits into from
Dec 4, 2024

Conversation

LorenzoBettini
Copy link
Contributor

Closes #3258

Besides what's described in the issue, I also added Java records to ITs of our Maven plugins, since now we required Java 17. The ITs related to Java 21 are still useful because in those ITs we generate Java 21 bytecode.

Copy link

github-actions bot commented Nov 27, 2024

Test Results

  6 460 files  ±0    6 460 suites  ±0   3h 12m 14s ⏱️ + 3m 33s
 43 238 tests ±0   42 654 ✅ +1    584 💤 ±0   0 ❌  - 1 
170 253 runs   - 7  167 904 ✅ +1  2 336 💤 ±0  12 ❌  - 8  1 🔥 ±0 

Results for commit aeb4cfd. ± Comparison against base commit 8190316.

This pull request removes 2 and adds 2 tests. Note that renamed tests count towards both.
org.eclipse.xtext.builder.standalone.incremental.IncrementalStandaloneBuilderWithJava21Test ‑ testCleanBuildAccessingJava21Record
org.eclipse.xtext.java.tests.JavaSourceLanguageTest ‑ testJava21Record
org.eclipse.xtext.builder.standalone.incremental.IncrementalStandaloneBuilderWithJava21Test ‑ testCleanBuildAccessingJavaRecord
org.eclipse.xtext.java.tests.JavaSourceLanguageTest ‑ testJavaRecord

♻️ This comment has been updated with latest results.

@LorenzoBettini
Copy link
Contributor Author

@szarnekow I rebased on top of main.
Is it OK to merge?

@LorenzoBettini
Copy link
Contributor Author

@cdietrich @szarnekow can I merge?

@LorenzoBettini LorenzoBettini merged commit 373b1ac into eclipse-xtext:main Dec 4, 2024
9 checks passed
@LorenzoBettini LorenzoBettini deleted the lb_3258 branch December 4, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup tests using Java records
2 participants