Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add tests #3

Closed
holic opened this issue Jul 19, 2014 · 1 comment
Closed

Add tests #3

holic opened this issue Jul 19, 2014 · 1 comment

Comments

@holic
Copy link
Owner

holic commented Jul 19, 2014

Not really sure how to easily test the Lua code, but rewriting in Go would be a lot easier to test, assuming it's more performant.
@holic
Copy link
Owner Author

holic commented Jul 19, 2014

Maybe move the redirect logic to a function/module with return location, status and test the module with a familiar external language (Node?).

holic added a commit that referenced this issue Jul 19, 2014
@holic
Move redirect config parser into its own module
818d01a 
In preparation for tests (#3)
holic added a commit that referenced this issue Jul 22, 2014
@holic
Convert ngx.re.match to Lua match for easier testing in Lua
b6f6268 
See #3
@holic holic closed this as completed in 537b1ef Jul 22, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@holic