Skip to content
This repository has been archived by the owner on May 19, 2023. It is now read-only.

Add unit-tests for rewrite middleware #78

Merged
merged 1 commit into from
Jan 18, 2023
Merged

Add unit-tests for rewrite middleware #78

merged 1 commit into from
Jan 18, 2023

Conversation

gaby
Copy link
Member

@gaby gaby commented Jan 18, 2023

Completed:

  • Fixed panic caused by passing an empty config to rewrite.New()
  • Added unit-tests for Filter function.
  • Added unit-tests for Regex Rules.
  • Added unit-tests for default config
  • Coverage report is now at 100% of statements
  • Added go 1.19.x to test suite

Results:

ubuntu@ubuntu:~/Desktop/git/fix/rewrite$ go test ./...
ok  	github.com/gofiber/rewrite/v2	0.004s
ubuntu@ubuntu:~/Desktop/git/fix/rewrite$ go test -cover ./...
ok  	github.com/gofiber/rewrite/v2	0.004s	coverage: 100.0% of statements

@ReneWerner87 ReneWerner87 merged commit 9b86150 into master Jan 18, 2023
@gaby gaby deleted the add-tests branch January 18, 2023 12:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants