databricks / cli

Databricks CLI
Other
148 stars 56 forks source link

Share test initializer in common helper function #1695

Closed pietern closed 3 months ago

pietern commented 3 months ago

Changes

These tests inadvertently re-ran mutators, the first time through loadTarget and the second time by running phases.Initialize() themselves. Some of the mutators that are executed in phases.Initialize() are also run as part of loadTarget. This is overdue a refactor to make it unambiguous what runs when. Until then, this removes the duplicated execution.

Tests

Unit tests pass.