Skip to content

Conversation

@stephen-carter-at-sf
Copy link
Contributor

STILL IN DRAFT

NEED TO ASK ROBERT ABOUT BROKEN TESTS

expect(countsPerRule).toEqual({
MissingDescription: 56,
MissingFaultHandler: 15,
MissingFaultHandler: 9, // TODO: Follow up with robert to ensure that this should be 9 (not 15 as it was before)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WAITING ON THIS TODO

const results: EngineRunResults = await engine.runRules(PreventPassingUserDataRules, createRunOptions(workspace));

expect(results.violations).toHaveLength(1);
expect(results.violations).toHaveLength(1); // TODO: This test is broken - need to check with Robert about this first!
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WAITING ON THIS TODO

const results: EngineRunResults = await engine.runRules(PreventPassingUserDataRules, createRunOptions(workspace));

expect(results.violations).toHaveLength(1);
expect(results.violations).toHaveLength(1); // TODO: This test is broken - need to check with Robert about this first!
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WAITING ON THIS TODO

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants