Commit 8627da5
committed
fix(algorithms): fix path-planting tests and add missing exports
- Fix path-generator.ts to validate source/target nodes exist before adding edges
- Fix test file to use correct Graph API (addNode takes single parameter)
- Add pathFollowsTemplate to evaluation exports
Test results:
- 18/23 path-planting tests passing (up from 12/23)
- 5 citation-planting tests have design issue (return Graph instead of PlantedPathResult)
- All 14 integration tests passing
- Core evaluation framework working correctly
Known limitation: Citation-planting tests need refactoring to return proper PlantedPathResult structure instead of raw Graph.
Note: Pre-existing lint errors in path-planting files remain (non-null assertions)1 parent 9ff78e2 commit 8627da5
File tree
3 files changed
+23
-11
lines changed- packages/algorithms
- __tests__/evaluation
- src/evaluation
- path-planting
3 files changed
+23
-11
lines changedLines changed: 6 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
40 | | - | |
| 40 | + | |
| 41 | + | |
41 | 42 | | |
42 | 43 | | |
43 | 44 | | |
| |||
89 | 90 | | |
90 | 91 | | |
91 | 92 | | |
92 | | - | |
| 93 | + | |
| 94 | + | |
93 | 95 | | |
94 | 96 | | |
95 | 97 | | |
| |||
173 | 175 | | |
174 | 176 | | |
175 | 177 | | |
176 | | - | |
| 178 | + | |
| 179 | + | |
177 | 180 | | |
178 | 181 | | |
179 | 182 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| 47 | + | |
47 | 48 | | |
48 | 49 | | |
49 | 50 | | |
| |||
Lines changed: 16 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
158 | 158 | | |
159 | 159 | | |
160 | 160 | | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
161 | 173 | | |
162 | 174 | | |
163 | 175 | | |
| |||
167 | 179 | | |
168 | 180 | | |
169 | 181 | | |
170 | | - | |
171 | | - | |
172 | | - | |
173 | | - | |
| 182 | + | |
| 183 | + | |
174 | 184 | | |
175 | 185 | | |
176 | 186 | | |
| |||
212 | 222 | | |
213 | 223 | | |
214 | 224 | | |
215 | | - | |
216 | | - | |
217 | | - | |
218 | | - | |
| 225 | + | |
| 226 | + | |
219 | 227 | | |
220 | 228 | | |
221 | 229 | | |
| |||
0 commit comments