Test::Nginx &amp; env settings

rwag · 2015-06-20T05:14:30+00:00

On Sat, Jun 20, 2015 at 12:03 PM, Yichun Zhang (agentzh) <age...@gmail.com> wrote: > ensure you're armed with enough knowledge to save both your and...

Test::Nginx & env settings

rwag

I have an application that uses an environment variable.

In the application's nginx.conf it looks something like this.

====

env DATA_DIRECTORY;

====

For testing I am using Test::Nginx::Socket::Lua.

In a t-file for testing how do I get that same sort of value set?

Right now I have a hack where I pass the string in to a module with a setting.

I would like to have the module code function as before, where it read the value as-follows.

====

os.getenv("DATA_DIRECTORY")

====

Any info is much appreciated.

agentzh

Hello!

On Sat, Jun 20, 2015 at 5:14 AM, Rollin Crittendon wrote:
> I have an application that uses an environment variable.
> In the application's nginx.conf it looks something like this.
> ====
> env DATA_DIRECTORY;
> ====
> For testing I am using Test::Nginx::Socket::Lua.
> In a t-file for testing how do I get that same sort of value set?
>

Ah, there's no way to set that in the test file at the moment due to
the lack of interface. Maybe we can add a Perl function for that? For
example,

    env_to_nginx("foo", "bar=32");

will add the following lines to the nginx.conf generated:

    env foo;
    env bar=32;

Will you submit a pull request for this feature? Thanks a lot!

Best regards,
-agentzh

agentzh

Hello!

On Sat, Jun 20, 2015 at 11:27 AM, Yichun Zhang (agentzh) wrote:
> Ah, there's no way to set that in the test file at the moment due to
> the lack of interface. Maybe we can add a Perl function for that? For
> example,
>
>     env_to_nginx("foo", "bar=32");
>
> will add the following lines to the nginx.conf generated:
>
>     env foo;
>     env bar=32;
>
> Will you submit a pull request for this feature? Thanks a lot!
>

Okay, I've just implemented this env_to_nginx() function in the git
master branch of test-nginx:

https://github.com/openresty/test-nginx/commit/b02a309

Please see the detailed documentation in the commit.

Please let me know if it works for you :)

Thanks!
-agentzh