Skip to content

Updates tests to use assertThat instead of assertEquals#2511

Merged
sanderploegsma merged 5 commits into
exercism:mainfrom
pavanbaloju:update-tests-to-use-assertj
Oct 6, 2023
Merged

Updates tests to use assertThat instead of assertEquals#2511
sanderploegsma merged 5 commits into
exercism:mainfrom
pavanbaloju:update-tests-to-use-assertj

Conversation

@pavanbaloju
Copy link
Copy Markdown
Contributor

pull request for (#2147 )

In this Pr, all the below tests have been updated to use assertj

  1. ComplexNumberTest.java
  2. EtlTest.java
  3. FlattenerTest.java
  4. FoodChainTest.java
  5. GigasecondTest.java
  6. GoCountingTest.java
  7. GrainsTest.java
  8. GrepToolTest.java
  9. IsbnVerifierTest.java
  10. LargestSeriesProductCalculatorTest.java
  11. MatrixTest.java
  12. MeetupTest.java
  13. MicroBlogTest.java
  14. MinesweeperBoardTest.java
  15. NucleotideCounterTest.java
  16. OctalTest.java
  17. PascalsTriangleGeneratorTest.java
  18. NaturalNumberTest.java
  19. PigLatinTranslatorTest.java
  20. PokerTest.java
  21. PovTest.java
  22. PrimeFactorsCalculatorTest.java
  23. PythagoreanTripletTest.java

Reviewer Resources:

Track Policies

Comment thread exercises/practice/octal/src/test/java/OctalTest.java
Copy link
Copy Markdown
Contributor

@sanderploegsma sanderploegsma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for your contribution 🙏

@sanderploegsma sanderploegsma merged commit 7ad49a1 into exercism:main Oct 6, 2023
@pavanbaloju pavanbaloju deleted the update-tests-to-use-assertj branch October 6, 2023 17:10
@pavanbaloju
Copy link
Copy Markdown
Contributor Author

LGTM! Thanks for your contribution 🙏

Thank you!

pavanbaloju added a commit to pavanbaloju/exercism-java that referenced this pull request Oct 6, 2023
* Updates tests to use assertThat instead of assertEquals

* Fix style check

* Revert deprecated exercise changes

* Fix style check

* Revert imports change
sanderploegsma pushed a commit that referenced this pull request Oct 6, 2023
* Update tests to use assertj for classes starting with letter R-Z

* Revert deprecated exercise changes

* Updates tests to use assertThat instead of assertEquals (#2511)

* Updates tests to use assertThat instead of assertEquals

* Fix style check

* Revert deprecated exercise changes

* Fix style check

* Revert imports change

* Use assertj pattern matching in test
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.

2 participants