Add test for pkg/agent/core/ngt/service/vqueue/option

[hlts2]

Signed-off-by: hlts2 hiroto.funakoshi.hiroto@gmail.com

Description:

I added test code for pkg/agent/core/ngt/serice/vqueue/option.go and godoc comment.
And I refactored based on a document of option error handling and fixed ci fails error.

gramer check passed

Related Issue:

How Has This Been Tested?:

Environment:

• Go Version: 1.16
• Docker Version: 19.03.8
• Kubernetes Version: 1.18.2
• NGT Version: 1.12.3

Types of changes:

• Bug fix [type/bug]
• New feature [type/feature]
• Add tests [type/test]
• Security related changes [type/security]
• Add documents [type/documentation]
• Refactoring [type/refactoring]
• Update dependencies [type/dependency]
• Update benchmarks and performances [type/bench]
• Update CI [type/ci]

Changes to Core Features:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your core changes, as applicable?
• Have you successfully ran tests with your changes locally?

Checklist:

• I have read the CONTRIBUTING document.
• I have checked open Pull Requests for the similar feature or fixes?
• I have added tests and benchmarks to cover my changes.
• I have ensured all new and existing tests passed.
• I have commented my code, particularly in hard-to-understand areas
• I have updated the documentation accordingly.

[vdaas-ci]

[CHATOPS:HELP] ChatOps commands.

• 🙆‍♀️ /approve - approve
• 💌 /changelog - replace the PR body by changelog details
• 🍱 /format - format codes and add licenses
• ✅ /gen-test - generate test codes
• 🏷️ /label - add labels
• ⏪ /rebase - rebase master
• 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

[codecov]

Codecov Report

Merging #1233 (59c8071) into master (b18982b) will increase coverage by 0.06%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master    #1233      +/-   ##
==========================================
+ Coverage   17.47%   17.53%   +0.06%     
==========================================
  Files         498      498              
  Lines       30835    30845      +10     
==========================================
+ Hits         5387     5409      +22     
+ Misses      25164    25157       -7     
+ Partials      284      279       -5     

Impacted Files	Coverage Δ
pkg/agent/core/ngt/service/vqueue/queue.go	0.00% <0.00%> (ø)
pkg/agent/core/ngt/service/vqueue/option.go	100.00% <100.00%> (+100.00%)	⬆️
internal/worker/queue.go	95.00% <0.00%> (-3.34%)	⬇️
internal/worker/worker.go	83.33% <0.00%> (-1.05%)	⬇️

---

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 b18982b...59c8071. Read the comment docs.

[kevindiu]

Please also refactor https://github.com/vdaas/vald/blob/master/pkg/agent/core/ngt/service/vqueue/queue.go#L85 to handle option error, otherwise if the option is invalid, the ngt may not work due to your changes.

[kevindiu]

LGTM

[kevindiu]

/rebase
/format

[vdaas-ci]

[REBASE] Rebase triggered by kevindiu for branch: test/pkg/add-agent-core-ngt-service-vqueue-option-test

[vdaas-ci]

[FORMAT] Updating license headers and formatting go codes triggered by kevindiu.

[kevindiu]

/approve

[vdaas-ci]

[APPROVED] This PR is approved by kevindiu.