FEDX-522 Add APIs to help test suggestors with resolved AST

Workiva / dart_codemod

A library that makes it easy to write and run automated code modifications on a codebase.

Other

61 stars 11 forks source link

FEDX-522 Add APIs to help test suggestors with resolved AST #74

Closed evanweible-wf closed 12 months ago

evanweible-wf commented 1 year ago

Motivation

AstVisitingSuggestor makes it easy to write suggestors that resolve the AST by overriding shouldResolveAst(), but we don't make it very easy to test those suggestors, especially on files that have imports.

Changes

Add PackageContextForTest to help with this use case. See readme for details.

aviary-wf commented 1 year ago

Security Insights

No security relevant content was detected by automated scans.

Action Items

Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

evanweible-wf commented 12 months ago

QA +1

[x] Verified usage here: https://github.com/Workiva/w_flux/blob/ActionV2-type-migration/w_flux_codemod/test/action_v2_suggestor_test.dart#L25