Commit 4e7ee47
Fix LineContentBoundsDrawingTest JUnit 5 parameterized test migration
This commit fixes a compilation error introduced during the JUnit 5 migration
of LineContentBoundsDrawingTest. The test was using a mix of JUnit 4
@RunWith(Parameterized.class) with JUnit 5 annotations, causing:
"No ParameterResolver registered for parameter [java.lang.String arg0]"
Changes:
- Converted from JUnit 4 constructor-based parameterization to JUnit 5
method parameter-based parameterization
- Removed @RunWith(Parameterized.class) and related JUnit 4 imports
- Changed @parameters to provide method source via @MethodSource
- Converted @test to @ParameterizedTest(name = "{0}")
- Removed constructor that accepted String parameter
- Modified test method to accept String parameter directly
- Added required JUnit 5 parameterized test imports to MANIFEST.MF:
- org.junit.jupiter.params
- org.junit.jupiter.params.provider
The test now compiles successfully with 0 errors. Note: 5 pre-existing test
failures remain from before the JUnit 5 migration (verified by checking git
history showing no logic changes to those tests).
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>1 parent 2c6cc72 commit 4e7ee47
File tree
2 files changed
+8
-15
lines changed- tests/org.eclipse.jface.text.tests
- META-INF
- src/org/eclipse/jface/text/tests/source/inlined
2 files changed
+8
-15
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| 33 | + | |
| 34 | + | |
33 | 35 | | |
Lines changed: 6 additions & 15 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
19 | | - | |
20 | | - | |
21 | | - | |
| 18 | + | |
| 19 | + | |
22 | 20 | | |
23 | 21 | | |
24 | 22 | | |
| |||
48 | 46 | | |
49 | 47 | | |
50 | 48 | | |
51 | | - | |
52 | 49 | | |
53 | | - | |
54 | 50 | | |
55 | 51 | | |
56 | 52 | | |
| |||
59 | 55 | | |
60 | 56 | | |
61 | 57 | | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | 58 | | |
69 | 59 | | |
70 | 60 | | |
| |||
113 | 103 | | |
114 | 104 | | |
115 | 105 | | |
116 | | - | |
117 | | - | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
118 | 109 | | |
119 | 110 | | |
120 | 111 | | |
121 | 112 | | |
122 | 113 | | |
123 | | - | |
| 114 | + | |
124 | 115 | | |
125 | 116 | | |
126 | 117 | | |
| |||
0 commit comments