Add internal/params pacakge test

[hlts2]

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

Description:

I added internal/params pacakge test.
this test' coverage is 100%.

Related Issue:

How Has This Been Tested?:

Environment:

• Go Version: 1.14.3
• Docker Version: 19.03.8
• Kubernetes Version: 1.18.2
• NGT Version: 1.11.5

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.

[pull-assistant]

Best reviewed: commit by commit

---

Optimal code review plan

     feat: add params pacakge test

     feat: add comment for the godoc

     fix: apply suggestion

Powered by Pull Assistant. Last update d6c55a7 ... 0aebb78. Read the comment docs.

[codecov]

Codecov Report

Merging #474 into master will increase coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #474      +/-   ##
=========================================
+ Coverage    7.83%   8.07%   +0.23%     
=========================================
  Files         396     396              
  Lines       20320   20320              
=========================================
+ Hits         1593    1640      +47     
+ Misses      18490   18449      -41     
+ Partials      237     231       -6     

Impacted Files	Coverage Δ
internal/params/option.go	100.00% <ø> (+66.66%)	⬆️
internal/params/params.go	100.00% <ø> (+100.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 674851f...9257230. Read the comment docs.

[hlts2]

/rebase

[vdaas-ci]

[REBASE] Rebase triggered by hlts2 for branch: test/internal/params

[kevindiu]

LGTM. :)

[kevindiu]

You can use the add suggestion to batch and commit my suggestions directly. It will show who is committing the code.

[kevindiu]

LGTM

[vankichi]

@hlts2 document review

[hlts2]

/rebase

[vdaas-ci]

[REBASE] Rebase triggered by hlts2 for branch: test/internal/params

[vankichi]

@hlts2 suggestion :)

[vankichi]

LGTM

[vankichi]

/approve
/format
/approve

[vdaas-ci]

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

[vdaas-ci]

[APPROVED] This PR is approved by vankichi.