Составить unit-тесты на описания для lexical_analyzer

Issue #294 new
Former user created an issue

Originally reported on Google Code with ID 294

Должен быть протестирован как обычный ход работы, так и bypass.
Должна быть протестирована генерация соответствий между исправленной строкой и строкой
студента, а также список ошибок, по одному тесту, на каждый вид ошибок.

Reported by mamontov.dp on 2014-07-08 11:29:34

Comments (18)

  1. Former user Account Deleted
    Код вытянут и слит. Необходимо начать писать тесты. Когда приступите к задаче?
    

    Reported by mamontov.dp on 2014-07-16 18:56:20

  2. Oleg Sychev repo owner
    Маша, жлем ответа. Для вас будет сильно лучше если в сентябре мы статью отправим...
    

    Reported by oasychev on 2014-07-16 21:27:11

  3. Former user Account Deleted
    Если функция best_string_pairs, то сегодня начну. Если другая - напишите название.
    

    Reported by mariabirvg on 2014-07-17 05:05:20

  4. Former user Account Deleted
    На best_string_pairs у нас уже есть тексты. Я имею ввиду сам qtype_correctwriting_lexical_analyzer.
    

    Reported by mamontov.dp on 2014-07-17 08:06:48

  5. Former user Account Deleted
    Возможно привести один тестовый пример? так как мне не очень понятно как там создавать
    

    Reported by mariabirvg on 2014-07-17 10:36:49

  6. Oleg Sychev repo owner
    Речь идет о тестах на сам класс анализатора - как в режиме анализа, так и (1-2 теста)
    - байпасса.
    

    Reported by oasychev on 2014-07-17 10:49:27

  7. Oleg Sychev repo owner
    Создаете объект qtype_correctwriting_lexical_analyzer используя конструктор с параметрами
    и проверяете в тесте корректность содержимого поля resultstringpairs - что одна или
    несколько (если есть эквивалентные по весу группы ошибок) пар там находятся, что в
    эти парах строк корректно заполнены поля, которые ваш анализатор должен заполнять (главным
    образом поле $matches в классе пары строк) и что генерируются нормальные $mistakes.
    Какие тут нужны примеры?
    
    Дмитрий - я все перечислил?
    

    Reported by oasychev on 2014-07-17 11:03:34

  8. Former user Account Deleted
    Да. Желательно по одному тесту на каждый вид ошибки и один - на комплексный случай.
    

    Reported by mamontov.dp on 2014-07-17 12:50:47

  9. Oleg Sychev repo owner
    Issue 295 has been merged into this issue.
    

    Reported by oasychev on 2014-12-03 00:40:12

  10. Former user Account Deleted
    Если тесты на составление описаний являются отдельным issue, то да, fixed.
    

    Reported by mamontov.dp on 2015-03-03 12:42:21

  11. Oleg Sychev repo owner
    Дмитрий, оставляю на вас решение - как легче/быстрее, сделать тесты на описания самому
    или ждать Марию. Можно пока заняться картинкой...
    

    Reported by oasychev on 2015-03-03 15:57:17

  12. Former user Account Deleted
    Сейчас у задачи на описания есть слелдующие измерения
    
    Есть/нет описания у лексемы в ответе преподавателя - варианты "да", "нет"
    Вид лексической ошибки - опечатка, пропуск разделителя, разный разделитель, спец. ошибка
    Работа с перечислениями - есть перемещение лексемы правильного ответа/нет
    Вид ошибки последовательности - пропуск лексемы, удаление лексемы, перемещение лексемы
    
    Все измерения, кроме того, что зависит от спец. ошибки - могу взять на себя.
    Скорее всего так и поступлю. Пусть Мария дополнит, когда такие ошибки появятся.
    

    Reported by mamontov.dp on 2015-03-04 14:30:22

  13. Oleg Sychev repo owner
    Нужны обязательно тесты на сочетания ошибок:
    а) когда в одной лексеме несколько ошибок (например опечатка + перемещена)
    б) когда из-за ошибки в предыдущих лексемах типа лишний или отсутствующий разделитель,
    номер лексемы с другой ошибкой меняется при исправлении строки.
    

    Reported by oasychev on 2015-03-09 02:09:03

  14. Former user Account Deleted
    Вылил 6 тестов, в которых присутствуют комбинации данных ошибок + сдвиг по индексов.
    Также учтено присутствие/отстутсиве описаний для лексем. Смотреть в общем с Марией
    репозитории. В ближайшее время планирую интегрировать новый расчет расстояния Дамерау-Левенштейна
    (добавить t)
    

    Reported by mamontov.dp on 2015-03-13 19:48:44

  15. Former user Account Deleted
    Вылил в formallangs вычисление redaction на основании Дамерау-Левенштейна. Судя по тестам,
    вычисление транспозиции проходит нормально. Заодно - залил небольшой фикс.
    

    Reported by mamontov.dp on 2015-03-13 20:34:57

  16. Log in to comment