Tests Bug

[romach]

Нашел баг в тестах:

Порядок воспроизведения

• обновить master;
• удалить базу данных;
• создать базу данных;
• запустить gradle (при этом запускается liquiBase, создаются таблицы и заполняются данные)

При первом запуске тестов возникают две ошибки:

org.itevents.service.CityServiceTest > testAddCity FAILED
    org.springframework.dao.DuplicateKeyException at CityServiceTest.java:55
        Caused by: org.postgresql.util.PSQLException at CityServiceTest.java:55

org.itevents.service.CityServiceTest > testRemoveCitySuccess FAILED
    org.springframework.dao.DuplicateKeyException at CityServiceTest.java:77
        Caused by: org.postgresql.util.PSQLException at CityServiceTest.java:77

При последующих запусках ошибки не возникают;

Анализ бага

• дело в том, что мы файле changeset-010-insert_data_to_city.xml явно указываем id. При этом у нас не обновляется сиквенс первичного ключа (http://stackoverflow.com/questions/9108833/postgres-autoincrement-not-updated-on-explicit-id-inserts). То-есть он равен 0.
• при первом запуске тестов :
• testAddExistingCity : инкремент сиквенса (city_pkey = 1) происходит ожидаемый exception:

    org.springframework.dao.DuplicateKeyException: 
    ### Error updating database.  Cause: org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
      Подробности: Key (id)=(1) already exists.
    ### The error may involve org.itevents.mybatis.mapper.CityMapper.addCity-Inline
    ### The error occurred while setting parameters
    ### SQL: INSERT INTO city(name, details, point) VALUES(?, ?, ST_MakePoint(?,?))
    ### Cause: org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
      Подробности: Key (id)=(1) already exists.
    ; SQL []; ERROR: duplicate key value violates unique constraint "city_pkey"
      Подробности: Key (id)=(1) already exists.; nested exception is org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
      Подробности: Key (id)=(1) already exists.

• testAddCity : инкремент сиквенса (city_pkey = 2) происходит exception:

org.springframework.dao.DuplicateKeyException: 
### Error updating database.  Cause: org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(2) already exists.
### The error may involve org.itevents.mybatis.mapper.CityMapper.addCity-Inline
### The error occurred while setting parameters
### SQL: INSERT INTO city(name, details, point) VALUES(?, ?, ST_MakePoint(?,?))
### Cause: org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(2) already exists.
; SQL []; ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(2) already exists.; nested exception is org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(2) already exists.

• testRemoveCitySuccess : инкремент сиквенса (city_pkey = 3) происходит exception:

org.springframework.dao.DuplicateKeyException: 
### Error updating database.  Cause: org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(3) already exists.
### The error may involve org.itevents.mybatis.mapper.CityMapper.addCity-Inline
### The error occurred while setting parameters
### SQL: INSERT INTO city(name, details, point) VALUES(?, ?, ST_MakePoint(?,?))
### Cause: org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(3) already exists.
; SQL []; ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(3) already exists.; nested exception is org.postgresql.util.PSQLException: ERROR: duplicate key value violates unique constraint "city_pkey"
  Подробности: Key (id)=(3) already exists.

При последующих запусках тестов исключений нет, так как сиквенс обновлен.

Вывод

Если явно не указывать id, бага не будет

[vaa25]

Замечательный отчет, Рома как всегда на высоте. Но данный баг легко исправился при рефакторинге тестов, так что issue 75 больше не актуальна.