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.

Sign up for GitHub

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

Jump to bottom

Forward zone unit test #234

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Forward zone unit test #234

merged 1 commit into from
Jun 20, 2024

Conversation

JchhatbarInfoblox
Copy link
Contributor

Forward zone unit test

Test result for forward zone :

go test -v --ginkgo.focus="Object Manager: forward zone" --ginkgo.trace
=== RUN   TestInfobloxGoClient
Running Suite: InfobloxGoClient Suite - /Users/jchhatbar/GolandProjects/dev/infoblox-go-client
==============================================================================================
Random Seed: 1718801835

Will run 5 of 581 specs
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS•••••SSSSSSSSSSSSSSSSSSSSSSSSSSSSS

Ran 5 of 581 Specs in 0.029 seconds
SUCCESS! -- 5 Passed | 0 Failed | 0 Pending | 576 Skipped
--- PASS: TestInfobloxGoClient (0.04s)
PASS
ok      github.com/infobloxopen/infoblox-go-client/v2   1.196s

Test result for all test cases :

go test -v --ginkgo.trace 
=== RUN   TestInfobloxGoClient
Running Suite: InfobloxGoClient Suite - /Users/jchhatbar/GolandProjects/dev/infoblox-go-client
==============================================================================================
Random Seed: 1718801849

Will run 581 of 581 specs
•••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••

Ran 581 of 581 Specs in 0.033 seconds
SUCCESS! -- 581 Passed | 0 Failed | 0 Pending | 0 Skipped
--- PASS: TestInfobloxGoClient (0.04s)
PASS
ok      github.com/infobloxopen/infoblox-go-client/v2   0.397s

JkhatriInfobox
JkhatriInfobox approved these changes Jun 20, 2024
View reviewed changes
Copy link
Collaborator

@JkhatriInfobox JkhatriInfobox left a comment

Choose a reason for hiding this comment

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

Unit test for forward zone looks good

@JkhatriInfobox JkhatriInfobox merged commit d208226 into infobloxopen:dev Jun 20, 2024
Aish-sp pushed a commit to Aish-sp/infoblox-go-client that referenced this pull request Jun 20, 2024
@JchhatbarInfoblox @Aish-sp
added unit test cases for Forward zone. (infobloxopen#234)
f3baeab
JkhatriInfobox added a commit that referenced this pull request Jun 20, 2024
@JkhatriInfobox @Aish-sp @JchhatbarInfoblox
Upgrade Go version to 1.21+ and add support for new object (#237)
0e7c3b2 
* [ADD] Support zone_forward DNS object

* [IMP] Updated Go Version to support 1.21 and above with updating dependent libs.

* added E2E test cases for Forward zone. (#232)

* added unit test cases for Forward zone. (#234)

* updated go version for travis CI run (#236)

* Add release notes and changelog (#235)

* Add release notes and changelog

* Update CHANGELOG.md

---------

Co-authored-by: Aish-sp <[email protected]>
Co-authored-by: Jaykumar Chhatbar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JkhatriInfobox JkhatriInfobox JkhatriInfobox approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JchhatbarInfoblox @JkhatriInfobox