search

Prior99 / jest-screenshot

A jest extension to deal with screenshots and all sorts of images.
MIT License
71 stars 21 forks source link

Possibility to add subtests link in reporter #4

Open Jack89ita opened 6 years ago

Jack89ita commented 6 years ago

Hi! I would like to know if it is possible to display in the report sidebar, under every test file, a list of all test in that file with a scrolling link. Actually i have just a couple of test files, but each file contain 10~ tests, so it will be useful to have a set of link instead of scrolling the entire page. Thank you very much.

Prior99 commented 6 years ago

I guess that would be easy to accomplish, when rendering the sidebar one could iterate over all tests and add an entry and put an id to the headline. I can see whether I can do it this weekend.

Jack89ita commented 6 years ago

It would be perfect for me! Let me know. Thank you!

Jack89ita commented 6 years ago

Hi! Any news about this? Thanks!

Prior99 commented 6 years ago

Hi, I am super busy currently and can't find the time to implement this. I would review a Pullrequest if you could find the time. Otherwise I'll see whether I can find the time on some weekend.

On Thu, May 24, 2018, 08:31 Giacomo Mottin notifications@github.com wrote:

Hi! Any news about this? Thanks!

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/Prior99/jest-screenshot/issues/4#issuecomment-391604222, or mute the thread https://github.com/notifications/unsubscribe-auth/ABbCdm--fjhKNtQK4cwvjvRWdyAgR612ks5t1lPZgaJpZM4T_H52 .

Jack89ita commented 6 years ago

I really would like to help you develop this feature, but i'm not so familiar with react, ts and tsx. I'm sorry for this, i will wait untill you find some free time to implement this. Thanks again.

Prior99 commented 6 years ago

Sorry for the delay.

On Thu, May 24, 2018, 12:53 Giacomo Mottin notifications@github.com wrote:

I really would like to help you develop this feature, but i'm not so familiar with react, ts and tsx. I'm sorry for this, i will wait untill you find some free time to implement this. Thanks again.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/Prior99/jest-screenshot/issues/4#issuecomment-391672188, or mute the thread https://github.com/notifications/unsubscribe-auth/ABbCdhHWwmw2n04R_iv7-xpXuRR7NyEqks5t1pEYgaJpZM4T_H52 .

Jack89ita commented 6 years ago

Hello @Prior99 Have you got any news about this? Thank you very much!

Prior99 commented 6 years ago

I am still very busy with my daytime job. I will see whether I can get to this once I have less on my desk. In the meantime I'd happily accept a PR ;)