Code Coverage for helpers.py about kitabe

[noobkid2411]

Hey,thanks for responding,but I was able to resolve it. Apparantly the system was failing in building wheel for the all-auth dependency. I just reinstalled django and ran it again.I'll ask if I have any more issues. Would try to add a pr for the same soon too and will keep you guys updated till then:)

[Sukriti-sood]

@Praful932 I want to work on this.

[Mukta-Sawant]

@Praful932 I would like to work on this issue as a part of GSSOC'21.

[Praful932]

@Mukta-Sawant Go ahead!

[maneprajakta]

@Mukta-Sawant are you working on this issue.

[Mukta-Sawant]

@maneprajakta Yes.

[Praful932]

@Mukta-Sawant You could open a draft PR even if it's incomplete so we know it is WIP

[maneprajakta]

@Mukta-Sawant no updates from your side.

[arpit456jain]

@Praful932 sir can you please guide me what's this (code coverge.py) and from where i can learn how to it.

[Praful932]

Sure @arpit456jain, Check out this article on what tests & coverage is. Then check out this article on django testing. Hope this helps.
Ping me if you have more doubts
Just @Praful932 is fine btw :)

[ayushjha952]

@Praful932 I want to contribute for this issue but I am not getting how to write unittest for methods present in helpers.py
can you help me with this?

[Praful932]

Hi @ayushjha952, for each function which isn't currently covered by tests you need to write tests which assert that the intended function returns what is expected

[ayushjha952]

@Praful932 I am not getting how to write the test case?

[Praful932]

@Praful932 I am not getting how to write the test case?

You can write it same as how django tests are written here, only difference is to assert the correct return from the function

[ayushjha952]

Okay So can you assign this to me I can try?

…

On Wed, Mar 31, 2021, 22:25 Praful932 ***@***.***> wrote: @Praful932 <https://github.com/Praful932> I am not getting how to write the test case? You can write it same as how django tests are written here <https://github.com/Praful932/Kitabe/blob/master/mainapp/tests.py>, only difference is to assert the correct return from the function — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#33 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AOKSI7XE7EYTO2JGDMEZSE3TGNHYFANCNFSM4YUVX7MA> .

[ayushjha952]

@Praful932 Do we have make any separate class or write in one of class provided in tests.py

[Praful932]

@Praful932 Do we have make any separate class or write in one of class provided in tests.py

Separate. Although you can combine all one-liner functions/similar functions into one class.
Do you want to be assigned to this issue?

[maneprajakta]

@Sukriti-sood Go for it, read once Starter Guide it will help in contributing to the project.

[maneprajakta]

@Sukriti-sood you can draft a WIP PR for changes done till now.

[jonathanalvares9009]

@maneprajakta can you assign me this issue?

[maneprajakta]

@jonathanalvares9009 Go ahead also once go through Starter Guide.

[maneprajakta]

@jonathanalvares9009 any updates?

[noobkid2411]

Hi,I would like to work on this issue,could you assign me the same!

[Praful932]

Sure go ahead @noobkid2411, do checkout the Guide :)

[noobkid2411]

Hey,I tried generating the results before adding code to helpers.py but the execution shows an error,
Module not found:no module named allauth and further goes on to say coverage.py warning no data collected. Can I get some help in figuring out what is the issue,I have never faced this issue before so would like some insights on the same!

[maneprajakta]

@noobkid2411 did you set up the env and installed all requirements.

[Praful932]

@noobkid2411 Could you add the ss

[noobkid2411]

Hey,unfortunately I have a health emergency in my family to deal with.Iwould still like to contribute but I might not be able to make a pr before the 20th.If this is an urgent work,you can unassign me,sorry for the inconvenience from my side.

[subhangi2731]

@Praful932 please assign me this I want to work on this

[maneprajakta]

@subhangi2731 you can take this up!

[maneprajakta]

@subhangi2731 any updates!

[Praful932]

Not prioritized right now