Resolve "Реализовать интеграционные автотесты для проверки модулей" #217

Merged
VladimirZarayskiy merged 8 commits from 103-autotest-conf into dev 5 years ago
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)
There is no content yet.
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

changed milestone to %5

changed milestone to %5
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

  • 961fdf93 - #70 added conference class, added create method test

Compare with previous version

added 1 commit <ul><li>961fdf93 - #70 added conference class, added create method test</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41660489&start_sha=ff7f535980ed6a34695b0f36665f61d65e06cb57)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>1ad66d6f - #103 change name method test</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41665249&start_sha=961fdf939759265df386f113444593d9586b4b61)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>253f4b4a - #103 add check new deadline</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41672370&start_sha=1ad66d6fb17bcf14d12fc8ca7fa9218419ad1665)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>0cb7b05c - #103 add check take part</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41674237&start_sha=253f4b4aad96dad266fb62fb1180bea11055e5dd)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>7b4318a3 - #103 add delete deadline</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41676466&start_sha=0cb7b05c2556f64b277c1d6903229ea0f38e72be)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>d9a42243 - #103 add attach article</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41686488&start_sha=7b4318a3b528c93b0be5cbdcbc19cb4019485221)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>cad86820 - #103 add adding article</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41691550&start_sha=d9a4224317c2ef5c5dae969003c6342a530d4500)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>4e4e7ae1 - #103 add paper transition</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41696347&start_sha=cad8682088abff0a54035edbcd0baaa2c5fe26b1)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>46864069 - #103 add undock article</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41697391&start_sha=4e4e7ae1948ac8ccfe1c4c612358c6399f887b30)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 3 commits

  • 6edcf8f5 - #103 add sort and filter test, fixed delete conference, if it has fk with event
  • 3f32a61e - Merge remote-tracking branch 'origin/103-autotest-conf' into 103-autotest-conf
  • b2e8a43c - #103 delete conf test

Compare with previous version

added 3 commits <ul><li>6edcf8f5 - #103 add sort and filter test, fixed delete conference, if it has fk with event</li><li>3f32a61e - Merge remote-tracking branch &#39;origin/103-autotest-conf&#39; into 103-autotest-conf</li><li>b2e8a43c - #103 delete conf test</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41713723&start_sha=46864069543d5c6f69572b68ab91c8e209069b1c)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>fe53c861 - #103 fixed delete conf test</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41820978&start_sha=b2e8a43ceec6f280324bea07c33d838cfd9c32ef)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>70062992 - #103 clear code</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41821026&start_sha=fe53c86136284413d153e6fc34700851ce98926e)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

assigned to @romanov73

assigned to @romanov73
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>5cbbb6c3 - #103 fix code style</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41821087&start_sha=70062992ae44493b6ec871a8a53ccd3aac23d29e)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>451e7e5d - #103 fix code style</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41821107&start_sha=5cbbb6c3190e2a28d793f8865722abef100475d4)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

unmarked as a Work In Progress

unmarked as a **Work In Progress**
romanov73 commented 5 years ago (Migrated from gitlab.com)

Все взаимодействие со страницей в PageObject надо выносить. Если меняется страница, то меняться должен только он, а не тесты

Все взаимодействие со страницей в PageObject надо выносить. Если меняется страница, то меняться должен только он, а не тесты
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

changed this line in version 15 of the diff

changed this line in [version 15 of the diff](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41928546&start_sha=451e7e5d04728e85a627852b3340b2b313b8e804#783e29e4f5c6cec11385afe81e9b0a65edb9a6c9_97_93)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>1f8722e3 - #103 code refactoring</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41928546&start_sha=451e7e5d04728e85a627852b3340b2b313b8e804)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

Взаимодействие вынес, про "Если меняется страница, то меняться должен только он, а не тесты" не понял

Взаимодействие вынес, про "Если меняется страница, то меняться должен только он, а не тесты" не понял
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>ab59cb52 - #103 fix code style</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41929960&start_sha=1f8722e3ff6036da36a57f328ad2a7a722e92650)
VladimirZarayskiy commented 5 years ago (Migrated from gitlab.com)

added 8 commits

  • ab59cb52...a3afdeb9 - 7 commits from branch dev
  • 58a84958 - Merge remote-tracking branch 'remotes/origin/dev' into 103-autotest-conf

Compare with previous version

added 8 commits <ul><li>ab59cb52...a3afdeb9 - 7 commits from branch <code>dev</code></li><li>58a84958 - Merge remote-tracking branch &#39;remotes/origin/dev&#39; into 103-autotest-conf</li></ul> [Compare with previous version](https://gitlab.com/romanov73/ng-tracker/merge_requests/90/diffs?diff_id=41930115&start_sha=ab59cb524e4e3eecbbdd4bb56f475fe065445da5)
romanov73 commented 5 years ago (Migrated from gitlab.com)

Тесты взаимодействуют с элементами на странице. Во время разработки верстка, расположение элементов, их локаторы могут меняться. Поэтому логика здесь похожая на концепцию mvc: все что что нужно актуализировать связанное с версткой меняется в одном классе. В тестах вызовы методов и нахождение элементов могут многократно повторяться -> каждый раз актуализация будет занимать много времени. PageObject служит своего рода интерфейсом между тестами и приложением.

Тесты взаимодействуют с элементами на странице. Во время разработки верстка, расположение элементов, их локаторы могут меняться. Поэтому логика здесь похожая на концепцию mvc: все что что нужно актуализировать связанное с версткой меняется в одном классе. В тестах вызовы методов и нахождение элементов могут многократно повторяться -> каждый раз актуализация будет занимать много времени. PageObject служит своего рода интерфейсом между тестами и приложением.
romanov73 commented 5 years ago (Migrated from gitlab.com)

resolved all discussions

resolved all discussions
romanov73 commented 5 years ago (Migrated from gitlab.com)

changed the description

changed the description
romanov73 commented 5 years ago (Migrated from gitlab.com)

не самый лучший вариант проверки, если потерли рядом стоящую сущность, тесты не упадут

не самый лучший вариант проверки, если потерли рядом стоящую сущность, тесты не упадут
romanov73 commented 5 years ago (Migrated from gitlab.com)

resolved all discussions

resolved all discussions
romanov73 commented 5 years ago (Migrated from gitlab.com)

merged

merged
romanov73 commented 5 years ago (Migrated from gitlab.com)

mentioned in commit 51851baa48

mentioned in commit 51851baa48157a5a67da3b172ae9ab2bd20d3b77
The pull request has been merged as 51851baa48.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b 103-autotest-conf dev
git pull origin 103-autotest-conf

Step 2:

Merge the changes and update on Gitea.
git checkout dev
git merge --no-ff 103-autotest-conf
git push origin dev
Sign in to join this conversation.
No reviewers
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: romanov73/ng-tracker#217
Loading…
There is no content yet.