The generated name of the test method is too long

[amandelpie]

Description

For the method "com.alibaba.fastjson.parser.ParserConfig#isPrimitive2" we generate an odd name

To Reproduce

TODO: Need to feel this

Expected behavior

I think it's totally correct (you can literally write implementation of the function using its name), but it seems too much – too long and does not have information about unique condition (which one of the disjunct was satisfied)

Actual behavior

/**
 * <pre>
 * Test returns from: {@code return clazz.isPrimitive() || clazz == Boolean.class || clazz == Character.class || clazz == Byte.class || clazz == Short.class || clazz == Integer.class || clazz == Long.class || clazz == Float.class || clazz == Double.class || clazz == BigInteger.class || clazz == BigDecimal.class || clazz == String.class || clazz == java.util.Date.class || clazz == java.sql.Date.class || clazz == java.sql.Time.class || clazz == java.sql.Timestamp.class || clazz.isEnum(); }
 *
 * throws TimeoutException in: return clazz.isPrimitive() || clazz == Boolean.class || clazz == Character.class || clazz == Byte.class || clazz == Short.class || clazz == Integer.class || clazz == Long.class || clazz == Float.class || clazz == Double.class || clazz == BigInteger.class || clazz == BigDecimal.class || clazz == String.class || clazz == java.util.Date.class || clazz == java.sql.Date.class || clazz == java.sql.Time.class || clazz == java.sql.Timestamp.class || clazz.isEnum();
 * </pre>
 */
@Test
@DisplayName("isPrimitive2: return clazz.isPrimitive() || clazz == Boolean.class || clazz == Character.class || clazz == Byte.class || clazz == Short.class || clazz == Integer.class || clazz == Long.class || clazz == Float.class || clazz == Double.class || clazz == BigInteger.class || clazz == BigDecimal.class || clazz == String.class || clazz == java.util.Date.class || clazz == java.sql.Date.class || clazz == java.sql.Time.class || clazz == java.sql.Timestamp.class || clazz.isEnum() -> ThrowTimeoutException")
@Timeout(value = 100L, unit = TimeUnit.MILLISECONDS)
public void testIsPrimitive2_ClazzIsPrimitiveOrClazzNotEqualsBooleanClassOrClazzNotEqualsCharacterClassOrClazzNotEqualsByteClassOrClazzNotEqualsShortClassOrClazzNotEqualsIntegerClassOrClazzNotEqualsLongClassOrClazzNotEqualsFloatClassOrClazzNotEqualsDoubleClassOrClazzNotEqualsBigIntegerClassOrClazzNotEqualsBigDecimalClassOrClazzNotEqualsStringClassOrClazzNotEqualsJavautilDateClassOrClazzNotEqualsJavasqlDateClassOrClazzNotEqualsJavasqlTimeClassOrClazzNotEqualsJavasqlTimestampClassOrClazzIsEnum() throws Throwable {
    Class class1 = Object.class;
    
    /* This execution may take longer than the 100 ms timeout
     and therefore fail due to exceeding the timeout. */
    ParserConfig.isPrimitive2(class1);
}

Additional context

We need to reproduce this bug on the copied snippet of code to the our codebase (see utbot-sample)

[onewhl]

Method to test:

```public static boolean isPrimitive2(Class<?> clazz) {
    return clazz.isPrimitive()
            || clazz == Boolean.class
            || clazz == Character.class
            || clazz == Byte.class
            || clazz == Short.class
            || clazz == Integer.class
            || clazz == Long.class
            || clazz == Float.class
            || clazz == Double.class
            || clazz == BigInteger.class
            || clazz == BigDecimal.class
            || clazz == String.class
            || clazz == java.util.Date.class
            || clazz == java.sql.Date.class
            || clazz == java.sql.Time.class
            || clazz == java.sql.Timestamp.class
            || clazz.isEnum();
}`

isPrimitive() checks if a clazz is instance of Double/Long/Boolean/Byte/Short/Integer/Float/Character.
I ran UTBot on the code above and got two generated tests, here they are:

1.   * <pre>
     * Test invokes:
     *     Class::isPrimitive once
     * returns from: {@code return clazz.isPrimitive() || clazz == Boolean.class || clazz == Character.class || clazz == Byte.class || clazz == Short.class || clazz == Integer.class || clazz == Long.class || clazz == Float.class || clazz == Double.class || clazz == BigInteger.class || clazz == BigDecimal.class || clazz == String.class || clazz == java.util.Date.class || clazz == java.sql.Date.class || clazz == java.sql.Time.class || clazz == java.sql.Timestamp.class || clazz.isEnum(); }
     * </pre>
     */
    @Test
    @DisplayName("isPrimitive2: ClassIsPrimitive -> return clazz.isPrimitive() || clazz == Boolean.class || clazz == Character.class || clazz == Byte.class || clazz == Short.class || clazz == Integer.class || clazz == Long.class || clazz == Float.class || clazz == Double.class || clazz == BigInteger.class || clazz == BigDecimal.class || clazz == String.class || clazz == java.util.Date.class || clazz == java.sql.Date.class || clazz == java.sql.Time.class || clazz == java.sql.Timestamp.class || clazz.isEnum()")
    public void testIsPrimitive2_ClazzIsPrimitiveOrClazzNotEqualsBooleanClassOrClazzNotEqualsCharacterClassOrClazzNotEqualsByteClassOrClazzNotEqualsShortClassOrClazzNotEqualsIntegerClassOrClazzNotEqualsLongClassOrClazzNotEqualsFloatClassOrClazzNotEqualsDoubleClassOrClazzNotEqualsBigIntegerClassOrClazzNotEqualsBigDecimalClassOrClazzNotEqualsStringClassOrClazzNotEqualsJavautilDateClassOrClazzNotEqualsJavasqlDateClassOrClazzNotEqualsJavasqlTimeClassOrClazzNotEqualsJavasqlTimestampClassOrClazzIsEnum() {
        Class class1 = Object.class;
   
        boolean actual = Main.isPrimitive2(class1);
   
        assertFalse(actual);
    }}`
   

In this case UTBot passes Object.class as a parameter to the tested method, therefore all the conditions are false, because all the listed classes extend from Object.class. So, the name should start with ClazzIsNotPrimitiveAnd...
This test covers the case when all the conditions are false, therefore the name should consist of a list of And instead of Or.

2.   * <pre>
     * Test invokes:
     *     Class::isPrimitive once
     *
     * throws NullPointerException in: return clazz.isPrimitive() || clazz == Boolean.class || clazz == Character.class || clazz == Byte.class || clazz == Short.class || clazz == Integer.class || clazz == Long.class || clazz == Float.class || clazz == Double.class || clazz == BigInteger.class || clazz == BigDecimal.class || clazz == String.class || clazz == java.util.Date.class || clazz == java.sql.Date.class || clazz == java.sql.Time.class || clazz == java.sql.Timestamp.class || clazz.isEnum();
     * </pre>
     */
    @Test
    @DisplayName("isPrimitive2: return clazz.isPrimitive() || clazz == Boolean.class || clazz == Character.class || clazz == Byte.class || clazz == Short.class || clazz == Integer.class || clazz == Long.class || clazz == Float.class || clazz == Double.class || clazz == BigInteger.class || clazz == BigDecimal.class || clazz == String.class || clazz == java.util.Date.class || clazz == java.sql.Date.class || clazz == java.sql.Time.class || clazz == java.sql.Timestamp.class || clazz.isEnum() : True -> ThrowNullPointerException")
    public void testIsPrimitive2_ClassIsPrimitive() {
        /* This test fails because executable under testing com.company.Main.isPrimitive2
        produces Runtime exception java.lang.NullPointerException */
        Main.isPrimitive2(null);
    }
   

`
Second test looks fine, we can get NPE in the first and in the last condition.

I added this example and corresponding summary test to our test base in the commit.