Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(account): add account project resource #1376

Merged
merged 7 commits into from
Jul 20, 2022

Conversation

yfodil
Copy link
Contributor

@yfodil yfodil commented Jul 18, 2022

feat #1051

@yfodil yfodil marked this pull request as draft July 18, 2022 16:04
@remyleone remyleone marked this pull request as ready for review July 20, 2022 09:28
})
}

func TestAccScalewayAccountProject_NoUpdate(t *testing.T) {
Copy link
Member

Choose a reason for hiding this comment

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

What is the purpose of this test?

Copy link
Member

Choose a reason for hiding this comment

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

This is a simple test with no update that we made on other resources, @Monitob asked for it

Copy link
Member

Choose a reason for hiding this comment

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

What was the reason? When did such test broke? Terraform checks that redoing a read on a resource does not change the state 🤔

@codecov-commenter
Copy link

Codecov Report

Merging #1376 (efcb15a) into master (3e84b5a) will decrease coverage by 0.35%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1376      +/-   ##
==========================================
- Coverage   66.73%   66.38%   -0.36%     
==========================================
  Files         122      123       +1     
  Lines       19521    19624     +103     
==========================================
  Hits        13027    13027              
- Misses       5357     5460     +103     
  Partials     1137     1137              
Impacted Files Coverage Δ
scaleway/helpers_account.go 50.00% <0.00%> (-50.00%) ⬇️
scaleway/provider.go 70.78% <0.00%> (-0.27%) ⬇️
scaleway/resource_account_project.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e84b5a...efcb15a. Read the comment docs.

@remyleone remyleone merged commit ce0ae07 into scaleway:master Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants