search

grapheo12 / iqps

Web-app meant for qp.metakgp.org
MIT License
20 stars 21 forks source link

Write Tests #11

Open grapheo12 opened 4 years ago

grapheo12 commented 4 years ago

Some basic API testing needs to be set up, so we don't accidentally break functionality. Maybe along with this a CI job (like a Github Action or Travis or CircleCI) can be set up to run on each PR.

grapheo12 commented 3 years ago

@Shrey-Sharma7 @shreyshik You too are assigned to this issue. We shall use Django tests to write our tests. Read the docs here: https://docs.djangoproject.com/en/3.1/topics/testing/overview/ In the iqps/ folder, you'll find many different sub-apps. Divide them into 2 groups and start writing tests. Before writing any test function, discuss in this thread what aspects are you testing for.

grapheo12 commented 3 years ago

Please self-assign yourself so that I get to know you have started working.

shreyshik commented 3 years ago

I do not have the permission to assign issues. Can you enable that please?

On Wed, 9 Dec 2020 at 11:16 PM Shubham Mishra notifications@github.com wrote:

Please self-assign yourself so that I get to know you have started working.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/grapheo12/iqps/issues/11#issuecomment-741940165, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQZQJF7LNATSG74TCV42XPDST6ZYNANCNFSM4OMSQ5AQ .

grapheo12 commented 3 years ago

@shreyshik Assigned you!