Skip to content

Commit

Permalink
Add readme giving commands, expectations, actual and (small) triage.
Browse files Browse the repository at this point in the history
  • Loading branch information
Samuel Cox authored and crankydillo committed Aug 20, 2024
1 parent 81edd76 commit 1463230
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions hamcrest-to-assertj/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
Meant to show an issue with 'AddDependency' when used on modified LST.

```sh
mvn org.openrewrite.maven:rewrite-maven-plugin:5.39.0:run \
-Drewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-testing-frameworks:2.16.0 \
-Drewrite.activeRecipes=org.openrewrite.java.testing.hamcrest.MigrateHamcrestToAssertJ \
-Drewrite.exportDatatables=true
```

After this, I expect `App.java` should be changed and `assertj` dependency
should be added to the pom. However, only `App.java` is changed and code no
longer compiles. Debugging suggests a condition related to `onlyIfUsing` in
`AddDependency` is not seeing the modified LST.

0 comments on commit 1463230

Please sign in to comment.