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.

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

Fixnut padajuce testy #280

Open
32 of 41 tasks
martinmacko47 opened this issue May 15, 2020 · 7 comments
Open
32 of 41 tasks

Fixnut padajuce testy #280

martinmacko47 opened this issue May 15, 2020 · 7 comments

Comments

@martinmacko47
Copy link
Owner

martinmacko47 commented May 15, 2020

  • V prvom PR si spust lokalne vsetky testy a padajuce testy s dekoratorom @skip. Testy sa pustaju pomocou env/bin/python manage.py test

  • V druhom PR rozbehaj CI na githube. Aby sa po kazdom pushnutom commite automaticky na githube spustili testy. Ak si nebudes vediet rady, googli. Na githube som CI este nerozbehaval. Ale v principe by malo stacit vytvorit subor .github/workflows/main.yml s popisom CI. Ciel je, aby v kazdom PR bol check, ci testy zbehli a nedovolito to PR mergnut ak testy nezbehli.

  • Potom dalej v samostatnych PR si zober jednu skupinu testov a padajuce testy fixni. V kazdom PR si zober napr. jeden adresar z poleno/, alebo jednu app z chcemvediet/apps/. Pripadne len jeden subor s testami, ak je tam vela parajucich testov.

    • poleno/utils/tests/test_forms.py
    • poleno/utils/tests/test_http.py
    • poleno/utils/tests/test_misc.py
    • poleno/utils/tests/test_models.py
    • poleno/utils/tests/test_views.py
    • poleno/attachments/tests/test_forms.py
    • poleno/attachments/tests/test_models.py
    • poleno/attachments/tests/test_views.py
    • poleno/mail/tests/test_backend.py
    • poleno/mail/tests/test_imap.py
    • poleno/mail/tests/test_mandrill.py
    • poleno/mail/tests/test_models.py
    • poleno/mail/tests/test_smtp.py
    • chcemvediet/apps/accounts/tests/test_forms.py
    • chcemvediet/apps/accounts/tests/test_views.py
    • chcemvediet/apps/inforequests/tests/test_cron.py
    • chcemvediet/apps/inforequests/tests/test_signals.py
    • chcemvediet/apps/inforequests/tests/test_models/test_action.py
    • chcemvediet/apps/inforequests/tests/test_models/test_action_draft.py
    • chcemvediet/apps/inforequests/tests/test_models/test_branch.py
    • chcemvediet/apps/inforequests/tests/test_models/test_inforequest.py
    • chcemvediet/apps/inforequests/tests/test_models/test_inforequest_draft.py
    • chcemvediet/apps/inforequests/tests/test_models/test_inforequest_email.py
    • chcemvediet/apps/inforequests/tests/test_views/test_add_smail.py
    • chcemvediet/apps/inforequests/tests/test_views/test_attachments.py
    • chcemvediet/apps/inforequests/tests/test_views/test_create.py
    • chcemvediet/apps/inforequests/tests/test_views/test_decide_email.py
    • chcemvediet/apps/inforequests/tests/test_views/test_decide_email_unknown.py
    • chcemvediet/apps/inforequests/tests/test_views/test_decide_email_unrelated.py
    • chcemvediet/apps/inforequests/tests/test_views/test_delete_draft.py
    • chcemvediet/apps/inforequests/tests/test_views/test_detail.py
    • chcemvediet/apps/inforequests/tests/test_views/test_extend_deadline.py
    • chcemvediet/apps/inforequests/tests/test_views/test_mine.py
    • chcemvediet/apps/inforequests/tests/test_views/test_new_action.py
    • chcemvediet/apps/obligees/tests/test_forms.py
    • chcemvediet/apps/obligees/tests/test_models.py
    • chcemvediet/apps/obligees/tests/test_views.py
    • chcemvediet/tests/test_views.py

Pri niektorych testoch bude jasne, co sa v kode zmenilo a ako testy treba upravit. Pri niektorych testoch to nemusi byt uplne jasne. Kde to nie je jasne, mozeme si k tomu zavolat a zamysliet sa nad tym.

@martinmacko47
Copy link
Owner Author

martinmacko47 commented Jun 11, 2020

Pri opravovani existujucich testov bude asi lahsie zacat od testov v adresari /poleno, kedze tam su vseobecnejsie utility.

Edit: ✔️

viliambalaz added a commit that referenced this issue Jun 11, 2020
* #280 Skip obligee tests

* #280 Skip inforequests tests

* #280 Skip accounts tests

* #280 Skip chcemvediet tests

* #280 Skip poleno tests
viliambalaz added a commit that referenced this issue Jun 13, 2020
viliambalaz added a commit that referenced this issue Jun 25, 2020
viliambalaz added a commit that referenced this issue Jun 30, 2020
viliambalaz added a commit that referenced this issue Jun 30, 2020
viliambalaz added a commit that referenced this issue Jun 30, 2020
viliambalaz added a commit that referenced this issue Jul 1, 2020
@martinmacko47
Copy link
Owner Author

martinmacko47 commented Jul 2, 2020

@viliambalaz Doplnil som do tasku zoznam testov, kde mame skip. Nech mame prehlad, kolko z toho je uz fixnute.

Btw, preco sme premenovali poleno/utils/test.py na poleno/utils/skip_test.py? Ved to nie su testy, ale utility k testom. Potom jasne, ze ti nefungovali importy v testoch, ked si premenoval importovany modul. Pls najprv, predtym nez budes fixovat ostatne testy, premenuj ten modul naspat a odznac skip_ prefixom moduly, ktorym uz importy potom budu fungovat.

Edit: ✔️

viliambalaz added a commit that referenced this issue Jul 2, 2020
* #280 Add main.yml

* #280 Edit main.yml

* #280 Use raw_input instead getpass.getpass

* #280 Install requirements for unittesting

* #280 Install requirements for unittesting

* #280 Add review suggestions

* #280 Edit on

* #280 Edit on: push

* #280 Remove strategy matrix
viliambalaz added a commit that referenced this issue Jul 6, 2020
viliambalaz added a commit that referenced this issue Aug 10, 2020
viliambalaz added a commit that referenced this issue Aug 11, 2020
viliambalaz added a commit that referenced this issue Aug 16, 2020
viliambalaz added a commit that referenced this issue Jul 15, 2021
viliambalaz added a commit that referenced this issue Jul 24, 2021
martinmacko47 added a commit that referenced this issue Jul 29, 2021
martinmacko47 added a commit that referenced this issue Jul 29, 2021
#415)

* #280 Fix ApplicantDeadlineReminderCronJobTest and CloseInforequestsCronJobTest tests

* #280 Remove unused import

* #280 Create reload_for_context

* #280 Review

Co-authored-by: Martin Macko <[email protected]>
viliambalaz added a commit that referenced this issue Aug 23, 2021
viliambalaz added a commit that referenced this issue Oct 30, 2021
viliambalaz added a commit that referenced this issue Nov 16, 2021
viliambalaz added a commit that referenced this issue Nov 23, 2021
viliambalaz added a commit that referenced this issue Nov 23, 2021
viliambalaz added a commit that referenced this issue Nov 23, 2021
viliambalaz added a commit that referenced this issue Nov 28, 2021
* #280 Fix inforequests view mine

* #280 Set correct path to mocked render and render_to_string

* #280 Add review suggestions

* #280 Set global password_hasher

* #280 Use assertTrue instead raising exception
viliambalaz added a commit that referenced this issue Dec 15, 2021
viliambalaz added a commit that referenced this issue Jan 9, 2022
viliambalaz added a commit that referenced this issue Apr 6, 2022
* #280 Fix test_detail view

* #280 Add inforequest test_detail tests

* #280 Refactroring tests names
viliambalaz added a commit that referenced this issue Apr 12, 2022
* #280 Fix test_detail view

* #280 Add inforequest test_detail tests

* #280 Refactroring tests names
viliambalaz added a commit that referenced this issue Apr 20, 2022
viliambalaz added a commit that referenced this issue Apr 20, 2022
viliambalaz added a commit that referenced this issue Apr 21, 2022
viliambalaz added a commit that referenced this issue Apr 25, 2022
viliambalaz added a commit that referenced this issue Apr 25, 2022
* #280 Fix test_create view

* #280 Compare attachment content
viliambalaz added a commit that referenced this issue Apr 27, 2022
viliambalaz added a commit that referenced this issue May 31, 2022
* #280 Fix DownloadAttachmentViewTest

* #280 Add tests for anonymous user to download attachment

* #280 Add tests for attachments assigned to action of anonymized inforequest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants