fredbi / uri

98%

master: 89%

LAST BUILD ON BRANCH test/regroup-rationalize

branch: test/regroup-rationalize

CHANGE BRANCH

x

Reset

Sync Branches

• test/regroup-rationalize
• build/resolve-go1.19-backwards-compat
• chore/ci-dependabot-update
• ci-govulncheck
• ci/fix-actions
• dependabot/github_actions/ci-actions-167a898cab
• dependabot/github_actions/ci-actions-8e42324b74
• dependabot/github_actions/ci-actions-a331d3ec2d
• dependabot/go_modules/development-dependencies-ddf553c9d0
• dependabot/go_modules/github.com/stretchr/testify-1.9.0
• doc-readme
• doc/readme-todos
• doc/readme-update
• doc/update-doc
• enh-error-handling
• exp/profile-with-bitset
• exp/switch-dns
• feat/default-port
• feat/explicit-errors
• feat/inner-error-state
• feat/more-iana
• feat/strict-ip
• feat/stricter-percent-decode
• feat/stricter-scheme
• fix-3-path-in-host-field
• fix-reduce-regexps
• fix/55
• fix/ci-stuff
• fix/doc
• fix/edgecases
• master
• patch-1
• perf/inner-pointer
• perf/profiling
• perf/slices-to-switch
• perf/string-allocs
• refact-layout
• refact-minor-optims
• refact-tests
• refact-validations
• refs/tags/v0.1.0
• refs/tags/v1.0.0
• refs/tags/v1.1.0
• refs/tags/v1.1.1
• release/prepare-v1.1.0
• test-fuzz
• test/more-coverage
• updated-docs
• v2
• workflow

Build # 6233212335

Build Type

Pull #32

github

Committed by fredbi

Commit Message

refactored tests

Historically, the test suite has been constructed by concatenating
the test suites of many different implementations of this functionality
in other languages.

Now is the time to refactor this and reorganize test cases.

* regrouped test cases in categories relative to the part being
validated
* assert the returned error (check that we indeed captured the invalid
part in the expected method)
* removed redundant test cases

* fixes #21

Signed-off-by: Frédéric BIDON <fredbi@yahoo.com>

Pull Request Pull Request #32: refactored tests

Run Details

13 of 13 new or added lines in 1 file covered. (100.0%)

625 of 636 relevant lines covered (98.27%)

671.01 hits per line