Recipe Name:
Use AssertJ's hasSize() instead of is(hasSize/iterableWithSize/arrayWithSize/aMapWithSize)
Description:
Can be replaced with AssertJ style assertions
Level:
warning
Language:
  • java
Tags:
  • AssertJ
  • Hamcrest
Recipe
id: scw:assertj:hamcrest-is-has-size
version: 10
metadata:
  name: Use AssertJ's hasSize() instead of is(hasSize/iterableWithSize/arrayWithSize/aMapWithSize)
  shortDescription: Can be replaced with AssertJ style assertions
  level: warning
  language: java
  enabled: true
  tags: AssertJ; Hamcrest
search:
  methodcall:
    name: assertThat
    anyOf:
    - args:
        2:
          value:
            is:
              methodcall:
                args:
                  1:
                    value:
                      is:
                        methodcall:
                          anyOf:
                          - name: hasSize
                            type: org.hamcrest.Matchers
                          - name: hasSize
                            type: org.hamcrest.collection.IsCollectionWithSize
                          - name: iterableWithSize
                            type: org.hamcrest.Matchers
                          - name: iterableWithSize
                            type: org.hamcrest.collection.IsIterableWithSize
                          - name: arrayWithSize
                            type: org.hamcrest.Matchers
                          - name: arrayWithSize
                            type: org.hamcrest.collection.IsArrayWithSize
                          - name: aMapWithSize
                            type: org.hamcrest.Matchers
                          - name: aMapWithSize
                            type: org.hamcrest.collection.IsMapWithSize
                name: is
                type:
                  reference:
                    matches: org\.hamcrest\.(CoreMatchers|Matchers|core\.Is)
                  checkInheritance: true
      argCount: 2
    - args:
        1:
          type: String
        3:
          value:
            is:
              methodcall:
                args:
                  1:
                    value:
                      is:
                        methodcall:
                          anyOf:
                          - name: hasSize
                            type: org.hamcrest.Matchers
                          - name: hasSize
                            type: org.hamcrest.collection.IsCollectionWithSize
                          - name: iterableWithSize
                            type: org.hamcrest.Matchers
                          - name: iterableWithSize
                            type: org.hamcrest.collection.IsIterableWithSize
                          - name: arrayWithSize
                            type: org.hamcrest.Matchers
                          - name: arrayWithSize
                            type: org.hamcrest.collection.IsArrayWithSize
                          - name: aMapWithSize
                            type: org.hamcrest.Matchers
                          - name: aMapWithSize
                            type: org.hamcrest.collection.IsMapWithSize
                name: is
                type:
                  reference:
                    matches: org\.hamcrest\.(CoreMatchers|Matchers|core\.Is)
                  checkInheritance: true
      argCount: 3
    type: org.hamcrest.MatcherAssert
availableFixes:
- doStaticImports: true
  name: Change to assertThat(actual).hasSize(expectedSize)
  availableIf:
    markedElement:
      is:
        methodcall:
          argCount: 2
  actions:
  - rewrite:
      to: org.assertj.core.api.Assertions.assertThat({{{ arguments.0 }}}).hasSize({{{ arguments.1.arguments.0.arguments.0 }}})
- doStaticImports: true
  name: Change to assertThat(actual).as(reason).hasSize(expectedSize)
  availableIf:
    markedElement:
      is:
        methodcall:
          argCount: 3
  actions:
  - rewrite:
      to: org.assertj.core.api.Assertions.assertThat({{{ arguments.1 }}}).as({{{ arguments.0 }}}).hasSize({{{ arguments.2.arguments.0.arguments.0 }}})