js-x-ray icon indicating copy to clipboard operation
js-x-ray copied to clipboard

Add UT around probes

Open Kawacrepe opened this issue 3 years ago • 2 comments

The goal is to add UT about https://github.com/NodeSecure/js-x-ray/tree/master/src/probes/

It can help you to get into this project, feel free to submit a PR about 1 or multiples probes :)

Kawacrepe avatar May 02 '22 13:05 Kawacrepe

Neat!

tony-go avatar May 10 '22 15:05 tony-go

List of probes that still need contribution.

Note Please contribute and create merge request for one probe at a time.

Already done:

  • [x] isLiteralRegex (contributed by @fraxken)
  • [x] isUnsafeCallee (contributed by @Kawacrepe)
  • [x] isWeakCrypto (contributed by @tony-go)
  • [x] isFunctionDeclaration
  • [x] isObjectExpression
  • [x] isBinaryExpression
  • [x] isRegexObject
  • [x] isMemberExpression
  • [x] isAssignmentExpression
  • [x] isImportDeclaration
  • [x] isRequire
  • [x] isArrayExpression
  • [x] isUnaryExpression
  • [x] isVariableDeclaration
  • [x] isLiteral

If you are new to the project or we didn't had any opportunity to deal with an AST (Abstract Syntax Tree) i highly recommend you to contribute to the first group of probes.

fraxken avatar Jun 03 '22 19:06 fraxken