fixes new_cluster missing in the request

[stikkireddy]

Fixes: #75, error was introduced by a prior pull request: #36

[stikkireddy]

@gnarik this should resolve your issue.

[codecov-commenter]

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   47.27%   47.27%           
=======================================
  Files          53       53           
  Lines        6815     6815           
=======================================
  Hits         3222     3222           
  Misses       3539     3539           
  Partials       54       54           

Flag	Coverage Δ
#client	76.45% <ø> (ø)
#provider	38.74% <0.00%> (ø)

Impacted Files	Coverage Δ
databricks/resource_databricks_cluster.go	42.49% <0.00%> (ø)
databricks/resource_databricks_job.go	51.20% <0.00%> (ø)

[rparme]

This fixed my issue. That said, I would have added a small unit test to avoid further regression.

[stikkireddy]

@Gnarik hey thanks for the feedback, I will add an integration test to test this resource object, unfortunately AFAIK Terraform does not provide a great way to unit test with the schema resource object so I will instead create an integration test, to test the creation of this resource object.

[rparme]

Hi, what's the plan for the merge of this branch ? Just trying to understand your workflow :)

[stikkireddy]

@Gnarik please let me know the changes should fix #75 and #81

[rparme]

Hi @stikkireddy, thank for the quick answer.
This PR fixes both #75 and #81