Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

issues-147: add description to tests #151

Merged
merged 2 commits into from
Jan 9, 2023
Merged

issues-147: add description to tests #151

merged 2 commits into from
Jan 9, 2023

Conversation

Mania-c
Copy link

@Mania-c Mania-c commented May 1, 2022 •
edited by Cdayz
Loading

  • Добавлено поле "Описание" в: тест, тест-кейсы, а так-же в allure отчет

Resolve #147

@Mania-c
Copy link
Author

Mania-c commented May 1, 2022 •
edited
Loading

Открытые вопросы:

  1. Если в cases не указано поле "description" нужно ли брать родительский
    Сейчас выводится родительский
  2. Если поле "description" не указано, нужно ли в консоли выводить поле с пустым описанием или убирать его из вывода
    Сейчас выводится пустое поле

Осталось сделать:

  1. Написать тесты на PR (если нужно). Можно добавить поле description в существующий тест

@Mania-c Mania-c mentioned this pull request May 1, 2022
Closed
Mania-c
Mania-c commented May 1, 2022
View reviewed changes
testloader/yaml_file/parser.go
@@ -93,7 +94,11 @@ func makeTestFromDefinition(filePath string, testDefinition TestDefinition) ([]T
// produce as many tests as cases defined
for caseIdx, testCase := range testDefinition.Cases {
test := Test{TestDefinition: testDefinition, Filename: filePath}
test.Name = fmt.Sprintf("%s #%d", test.Name, caseIdx)
test.Name = fmt.Sprintf("%s #%d", test.Name, caseIdx+1)
Copy link
Author

Choose a reason for hiding this comment

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

Когда указывали cases нумерация в выводе отображалась с нуля, поправил.
Теперь отображается с 1

@keyclaim
Copy link
Contributor

keyclaim commented May 4, 2022

Отчет станет вот такой
image

Мое мнение по вопросам:

  1. Да, выводить родительский
  2. Мне больше нравится вариант - выводить "No description"

@keyclaim
Copy link
Contributor

keyclaim commented May 4, 2022 •
edited
Loading

Также нужно обновить Readme:

  • привести пример(или обновить существующий).
  • в описании нотации упомянуть, что переносы строки учитываются если в yaml использовать description: |

@Cdayz Cdayz added enhancement New feature or request patch release as patch labels May 26, 2022
@Cdayz Cdayz changed the title [WIP] issues-147: add description to tests issues-147: add description to tests May 26, 2022
@Cdayz Cdayz self-requested a review May 26, 2022 11:41
Cdayz
Cdayz reviewed May 26, 2022
View reviewed changes
Copy link
Contributor

@Cdayz Cdayz left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Как упомянул @keyclaim, еще нужно:

  • Добавить в README информацию о поле
  • Добавить в README информацию о том что поле попадает в Allure отчет
  • Добавить что при использовании пайпа (description: |) будет учитываться лишняя строчка (либо добавить strip на это поле)
  • Выводить "No Description" когда поле не заполнено

@Cdayz Cdayz self-requested a review May 26, 2022 11:45
@Cdayz
Copy link
Contributor

Cdayz commented Jun 1, 2022

@Mania-c , посмотри комментарии пожалуйста

@Mania-c
Copy link
Author

Mania-c commented Jun 2, 2022

@Mania-c , посмотри комментарии пожалуйста

Комментарии увидел, на этой недели запушу

@leorush
Copy link
Contributor

leorush commented Jul 5, 2022 •
edited
Loading

Можно использовать descrtiption: >- или descrtiption: |- тогда не будет лишней строчки

@keyclaim keyclaim mentioned this pull request Oct 11, 2022
Closed
@fetinin
Copy link
Contributor

fetinin commented Jan 9, 2023

Ещё надо:

  • Добавить description в readme-ru.md
  • Добавить description в gonkey.json

Докину отдельным PR

@fetinin fetinin added skip-release skip release on merge to master and removed patch release as patch labels Jan 9, 2023
@fetinin fetinin merged commit 53a1c84 into lamoda:master Jan 9, 2023
@fetinin fetinin mentioned this pull request Jan 14, 2023
Merged
@github-actions
Copy link

🚀 PR was released in v1.20.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maximdanilchenko maximdanilchenko maximdanilchenko approved these changes

@fetinin fetinin fetinin approved these changes

@vitkarpenko vitkarpenko Awaiting requested review from vitkarpenko

@Cdayz Cdayz Awaiting requested review from Cdayz

Assignees
No one assigned
Labels
enhancement New feature or request released skip-release skip release on merge to master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Добавить поле описания теста и отображать его в результатах
6 participants
@Mania-c @keyclaim @Cdayz @leorush @fetinin @maximdanilchenko