test: add tests for array and object destructure in class methods

[buehler]

• Closes Handles parsing of destructuring parameters (eg. 'foo( { bar, test } ) #70

This PR adds tests for array and object destructured parameters in class methods.

[codecov-io]

Codecov Report

Merging #72 into develop will increase coverage by 0.63%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #72      +/-   ##
===========================================
+ Coverage     89.4%   90.04%   +0.63%     
===========================================
  Files           63       63              
  Lines          944      944              
  Branches       167      185      +18     
===========================================
+ Hits           844      850       +6     
+ Misses          93       87       -6     
  Partials         7        7

Impacted Files	Coverage Δ
src/node-parser/class-parser.ts	94.36% <0%> (+8.45%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b96650f...e639894. Read the comment docs.

[jpike88]

@buehler any idea when next release is due? would like to ditch my fork

[buehler]

https://github.com/buehler/node-typescript-parser/pull/72/files#diff-16e398003280769544ce2edf5ca63b56R59

is this what you meant?

[jpike88]

Wait I'm not understanding why those tests are written like that. If you're parsing destructuring parameters, wouldn't you want to retain that information instead of outputting a flat array?

foo (a, b, c){}

Should be treated differently from

foo ({a, b, c}){}

But those tests seems to treat them the same way.

[buehler]

actually as I understand the matter, it's parsed into the given variables.
...
OH. I think i get what you mean.

So you would rather see the combined variant instead of each of them?

examples:

function(a, b) {}
// --> [p1: a, p2: b]

function({a, b}){}
// --> [p1: {a, b}]

[jpike88]

Exactly

[jpike88]

Can you do me a favour and modify the tests to look for that output? I'll tweak a PR to suit them if you like

[buehler]

@jpike88
https://github.com/buehler/node-typescript-parser/releases/tag/v2.5.0

[jpike88]

@buehler thanks you're a legend