Skip to content

chore: test and example tech debt#466

Merged
lbajolet-hashicorp merged 1 commit intomainfrom
chore/test-debt
Sep 25, 2024
Merged

chore: test and example tech debt#466
lbajolet-hashicorp merged 1 commit intomainfrom
chore/test-debt

Conversation

@tenthirtyam
Copy link
Copy Markdown
Collaborator

Summary

Updates the tests and examples from the the original JetBrains content to generalized example content.

Testing

make test
?       github.com/hashicorp/packer-plugin-vsphere      [no test files]
ok      github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/clone        1.557s
ok      github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/common       2.338s
ok      github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/driver       5.569s
ok      github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/iso  1.557s
ok      github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/supervisor   4.876s
?       github.com/hashicorp/packer-plugin-vsphere/examples/driver      [no test files]
ok      github.com/hashicorp/packer-plugin-vsphere/post-processor/vsphere       1.399s
ok      github.com/hashicorp/packer-plugin-vsphere/post-processor/vsphere-template      1.527s

@tenthirtyam tenthirtyam added the technical-debt Technical Debt label Sep 18, 2024
@tenthirtyam tenthirtyam added this to the v1.5.0 milestone Sep 18, 2024
@tenthirtyam tenthirtyam self-assigned this Sep 18, 2024
@tenthirtyam tenthirtyam requested a review from a team as a code owner September 18, 2024 02:12
@tenthirtyam tenthirtyam marked this pull request as draft September 18, 2024 02:12
@tenthirtyam tenthirtyam force-pushed the chore/test-debt branch 2 times, most recently from 1956617 to 0978490 Compare September 18, 2024 02:16
@tenthirtyam tenthirtyam marked this pull request as ready for review September 19, 2024 00:25
Copy link
Copy Markdown
Contributor

@lbajolet-hashicorp lbajolet-hashicorp left a comment

Choose a reason for hiding this comment

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

Couple nits, but overall LGTM!
I'll let you address my comments and come back later, should be near ready to merge!

Pre-approving in advance

@tenthirtyam tenthirtyam force-pushed the chore/test-debt branch 2 times, most recently from 5269fff to ab6e747 Compare September 24, 2024 21:26
@tenthirtyam tenthirtyam force-pushed the chore/test-debt branch 2 times, most recently from e92207f to 4ca513d Compare September 24, 2024 21:43
Updates the tests and examples from the the original JetBrains content to generalized example content.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam
Copy link
Copy Markdown
Collaborator Author

Ready for review again @lbajolet-hashicorp. 😄

Copy link
Copy Markdown
Contributor

@lbajolet-hashicorp lbajolet-hashicorp left a comment

Choose a reason for hiding this comment

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

All good, thanks for the update @tenthirtyam!

Merging this now

@lbajolet-hashicorp lbajolet-hashicorp merged commit 3b382d2 into main Sep 25, 2024
@lbajolet-hashicorp lbajolet-hashicorp deleted the chore/test-debt branch September 25, 2024 21:08
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

technical-debt Technical Debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants