Skip to content

Conversation

@saeedfazal
Copy link
Contributor

@saeedfazal saeedfazal commented Nov 17, 2022

  • Added cost estimation acceptance tests
  • Refactor test to use common testData and testScripts paths
  • Added config git repository resource cleanup after test completion

@saeedfazal saeedfazal self-assigned this Nov 17, 2022
@saeedfazal saeedfazal force-pushed the test-cost-estimate-cluster branch 2 times, most recently from f020bea to a1325e3 Compare November 18, 2022 14:01
@saeedfazal saeedfazal requested a review from foot November 18, 2022 14:09
@saeedfazal saeedfazal force-pushed the test-cost-estimate-cluster branch from a1325e3 to 5ec6a78 Compare November 18, 2022 15:27
Copy link
Collaborator

@foot foot left a comment

Choose a reason for hiding this comment

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

💯

@saeedfazal saeedfazal merged commit 22e67b8 into main Nov 18, 2022
@saeedfazal saeedfazal deleted the test-cost-estimate-cluster branch November 18, 2022 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants