Skip to content

Commit

Permalink
chore: join object renaming tests into parameterized ones (#3154)
Browse files Browse the repository at this point in the history
## Changes
Follow-up for
#3130 (comment).

Tests joined from standalone tests to parameterized ones. Two tests were
left, because:
- First represents the non-deterministic behavior of Terraform and it
would be good to have it separated
- Second, is also unique and tests the correlation of config order to
execution order

Commented test cases are there intentionally because normally they were
skipped due to errors that cannot be asserted with terraform testing
framework.

**Question:** The "important" tests are still not separated from the
regular object renaming tests because they're currently failing in a way
they're not assertable by the terraform provider testing framework (they
will always fail). In this case, should we just leave it as is or just
mark the "important" tests with TODO and ticket number?
  • Loading branch information
sfc-gh-jcieslak authored Nov 5, 2024
1 parent c3edb79 commit be13502
Showing 1 changed file with 177 additions and 610 deletions.
Loading

0 comments on commit be13502

Please sign in to comment.