search

LoginRadius / idx-auto-tester

A test suite to do test automation of LoginRadius IDX implementation
MIT License
12 stars 19 forks source link

Code level improvements #21

Closed rkpareek closed 2 years ago

rkpareek commented 3 years ago

Welcome for any Code level improvements, restructuring, or performance improvements Test Cases should not impact Run complete suite before PR

aerabi commented 3 years ago

I would like to refactor mail4.js a bit.

rkpareek commented 3 years ago

Your contribution will be appreciated

On Wed, Oct 6, 2021, 4:23 PM aerabi @.***> wrote:

I would like to refactor mail4.js https://github.com/LoginRadius/idx-auto-tester/blob/master/helpers/mail7.js a bit.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/LoginRadius/idx-auto-tester/issues/21#issuecomment-935963747, or unsubscribe https://github.com/notifications/unsubscribe-auth/AECX75W4SKMVWEHK42YO3NLUFQTC7ANCNFSM5FERUP3A .

aerabi commented 3 years ago

So, I'm running on Ubuntu and the repo is Windows-based. Are there any plans to support Linux as well? So far I had to change the path of the Chrome driver to make Selenium run (it still doesn't work). I can document the steps I took after I finished.

rkpareek commented 3 years ago

Yeah, it will be helpful