add test for internal/errors/vald

[vankichi]

Signed-off-by: vankichi kyukawa315@gmail.com

I created the test for internal/errors/vald.go and also added comments for internl/errors/vald.go.
grammar check has passed.

Description:

Related Issue:

How Has This Been Tested?:

Environment:

• Go Version: 1.15.7
• Docker Version: 19.03.8
• Kubernetes Version: 1.18.2
• NGT Version: 1.13.1

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 - add changelog comment
• 🍱 /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

[github-actions]

🚫 [golangci] _{reported by reviewdog 🐶}
File is not gofmt-ed with -s (gofmt)

[codecov]

Codecov Report

Merging #958 (8864ece) into master (f8abd67) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #958   +/-   ##
=======================================
  Coverage   15.18%   15.18%           
=======================================
  Files         493      492    -1     
  Lines       25683    25622   -61     
=======================================
- Hits         3900     3891    -9     
+ Misses      21533    21490   -43     
+ Partials      250      241    -9     

Impacted Files	Coverage Δ
internal/errors/vald.go	100.00% <ø> (+100.00%)	⬆️
internal/net/net.go
internal/worker/worker.go	83.33% <0.00%> (+1.04%)	⬆️
internal/errgroup/group.go	93.93% <0.00%> (+1.51%)	⬆️
internal/errors/errors.go	37.50% <0.00%> (+1.56%)	⬆️
internal/db/storage/blob/s3/reader/reader.go	94.56% <0.00%> (+31.52%)	⬆️

---

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 488dd4a...d1a0154. Read the comment docs.

[hlts2]

thank you
LGTM

[kevindiu]

/format
/rebase
/approve

[kevindiu]

LGTM

[vdaas-ci]

[REBASE] Rebase triggered by kevindiu for branch: test/internal/add-newtest-for-errors-vald

[vdaas-ci]

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

[vankichi]

/format

[vdaas-ci]

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