Set default values

[masterT]

• set default values
• extract the code that format the query params in a new function ‘formatQueryParams’
• add test for default values

Fix issue #20

[t3chnoboy]

All checks have passed

whoa. PR checker works now!

[masterT]

Now I don't see the point of having a variable defaultParams. I could refactor a little more and change this:

// Default
defaultParams['SearchIndex'] = 'All';
defaultParams['Condition'] = 'All';
defaultParams['ResponseGroup'] = 'ItemAttributes';
defaultParams['Keywords'] = '';
defaultParams['ItemPage'] = '1';

params = setDefaultParams(params, defaultParams);

to that:

// Default
params = setDefaultParams(params, {
  SearchIndex: 'All',
  Condition: 'All',
// ...
});

@t3chnoboy what do you think?

[t3chnoboy]

sweet! Thanks a lot for what you are doing to this project!

[masterT]

You are welcomed! :blush:

[masterT]

I think everything is good now.

[masterT]

@t3chnoboy I will merge/dump version/publish good ?

[t3chnoboy]

yep! shipit :ship:

[t3chnoboy]

that was fast :car:

[masterT]

:information_desk_person: