Skip to content
This repository has been archived by the owner on May 15, 2023. It is now read-only.

Split handwritten tests by resource instead of CLI / read / unit #243

Open
melinath opened this issue Jun 28, 2021 · 2 comments
Open

Split handwritten tests by resource instead of CLI / read / unit #243

melinath opened this issue Jun 28, 2021 · 2 comments
Assignees
@ScottSuarez
Labels
tech debt

Comments

@melinath
Copy link
Member

melinath commented Jun 28, 2021 •
edited
Loading

The existing CLI / read / unit split is confusing for users, and each test file has too much magic in it, which can make it hard to understand what's actually happening. We should instead split tests by resource and prioritize readability. It might help to do this by generating the tests automatically, which would require marking all resources exclude_validator: true - similar to #232.

Blocked by #332
@melinath melinath changed the title Generate tests from Magic Modules Generate resource tests from Magic Modules Sep 14, 2021
@melinath melinath changed the title Generate resource tests from Magic Modules Copy handwritten per-resource tests from Magic Modules Sep 14, 2021
@melinath melinath changed the title Copy handwritten per-resource tests from Magic Modules Split handwritten tests by resource instead of CLI / read / unit Nov 1, 2021
@melinath
Copy link
Member Author

melinath commented Dec 1, 2021

sort of related: b/196008728

@marktru
Copy link

marktru commented Dec 23, 2021

Internal issue: b/196008728

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

@ScottSuarez ScottSuarez

Labels
tech debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@melinath @ScottSuarez @marktru