fedora-commops / fedora-happiness-packets

#250 Add Test Cases for URLs
Merged by jflory7. Opened by alishapapun.
fedora-commops/ alishapapun/fedora-happiness-packets add/pytest-migrations  into  add/pytest-migrations

Download 250.patch

This PR addresses 3 issues:
1. Adding dependency pytest, pytest-Django and pytest-cov to requirements.
2. Adding a .dockerignore
- While running test cases, pycache was getting created thus failing docker to get build.
- So, pycache was added to .dockerignore
3. Adding Test Cases for the urls.

This is for the first time I am writing test-case. Do lemme know @jflory7 if I did any mistake.

Metadata Update from @jflory7:
- Pull-request tagged with: PASSED, needs testing, new change, type - summer coding
- Request assigned

rebased onto 888ae092f2651585fd513b0346fb0a6b57216f98

I'm having some dependency issues in my environment but I am pretty sure they are specific to me. It's taking me a little longer to review this but I'll try to have it done by Friday night (US CDT).

rebased onto 22a956ca1647cd002a69d7b63615b261d874e9a7

@alishapapun I finally had a chance to test and review. These look great! :raised_hands: These look great and follow best practices for tests: they are short, simple, and easy to understand. I also learned something new about testing in Django. Merging! :ocean:

BTW – the issue I was facing is explained in this Red Hat Bugzilla ticket, specific to Fedora 30 and later.

Metadata Update from @jflory7:
- Pull-request untagged with: needs testing

Pull-Request has been merged by jflory7
Metadata
jflory7

PASSED new change type - summer coding

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.