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

Added names to token position tests #1961

Merged
merged 1 commit into from
Sep 4, 2024
Merged

Conversation

TwoOfTwelve
Copy link
Contributor

This PR adds a toString method to TokenPositionTestData, so JUnit shows a human readable name for each test.

This is the new output:
[1] Anno_2.java
[2] Anno_3.java
[3] JAnnoTBegin-End_1.java
[4] JAnnoTBegin-End_2.java
[5] VarDef_1.java
[6] Anno_4.java
[7] Anno_1.java

This is the old output (See #1960):
[1] de.jplag.testutils.datacollector.TokenPositionTestData@2fd6b6c7
[2] de.jplag.testutils.datacollector.TokenPositionTestData@5bfa9431
[3] de.jplag.testutils.datacollector.TokenPositionTestData@5db250b4
[4] de.jplag.testutils.datacollector.TokenPositionTestData@223f3642
[5] de.jplag.testutils.datacollector.TokenPositionTestData@38c5cc4c
[6] de.jplag.testutils.datacollector.TokenPositionTestData@37918c79
[7] de.jplag.testutils.datacollector.TokenPositionTestData@78e94dcf

Copy link

sonarcloud bot commented Sep 3, 2024

@tsaglam tsaglam added enhancement Issue/PR that involves features, improvements and other changes minor Minor issue/feature/contribution/change labels Sep 4, 2024
@tsaglam tsaglam merged commit ca2a891 into develop Sep 4, 2024
44 checks passed
@tsaglam tsaglam deleted the feature/tokenPositionTestNames branch September 4, 2024 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue/PR that involves features, improvements and other changes minor Minor issue/feature/contribution/change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants