mevn-cli icon indicating copy to clipboard operation
mevn-cli copied to clipboard

Write unit tests

Open jamesgeorge007 opened this issue 6 years ago • 10 comments

There is an action handler for each command. You can also find a couple of helper functions consumed by the respective handler. We need to write unit tests to ensure that the helpers serve their purpose.

jamesgeorge007 avatar Oct 01 '19 16:10 jamesgeorge007

@jamesgeorge007 Hey is this up for grabs

anooppoommen avatar Oct 03 '19 11:10 anooppoommen

@anooppoommen yeah, it is :+1: Let me know the command that you're willing to work on.

jamesgeorge007 avatar Oct 03 '19 11:10 jamesgeorge007

@jamesgeorge007 I'll work on the mevn init function for now

anooppoommen avatar Oct 04 '19 11:10 anooppoommen

I want to work on this issue.

cherryWood55 avatar Oct 05 '19 01:10 cherryWood55

@cherryWood55 you may take up a command other than mevn init

jamesgeorge007 avatar Oct 05 '19 05:10 jamesgeorge007

Where is the mevn init unit test?

evaporei avatar Oct 05 '19 18:10 evaporei

@otaviopace as of now we are missing out on unit tests (testing out the workflow in isolation). Feel free to pick up a command that you wish to work upon.

jamesgeorge007 avatar Oct 06 '19 07:10 jamesgeorge007

@jamesgeorge007 i am taking mev info.Is it okay?

ThakurKarthik avatar Oct 24 '19 05:10 ThakurKarthik

@jamesgeorge007 I shall take generate if no one else has https://github.com/madlabsinc/mevn-cli/blob/develop/src/commands/basic/generate.js. But I did notice that there is no jest setup for unit tests only e2e. I can include the setup in my PR as well, but wondering what's the plan on that part.

meroware avatar Oct 04 '20 02:10 meroware

@meroware thanks for showing interest, we don't have any unit tests at the moment. Feel free to shoot a PR.

jamesgeorge007 avatar Oct 12 '20 15:10 jamesgeorge007