search

Kong / ngx_wasm_module

Nginx + WebAssembly
Apache License 2.0
93 stars 8 forks source link

chore(tests) fix test cases for Wasmer in HUP mode #478

Closed thibaultcha closed 10 months ago

thibaultcha commented 10 months ago

Different compilers load the hostcalls filter faster or slower by an order of magnitude (30ms vs 300ms), and cause the old worker "free instance" log to happen sooner or later in req0, especially now that we update the time at module loading.

codecov[bot] commented 10 months ago

Codecov Report

Merging #478 (8b020ff) into main (e7769a0) will not change coverage. Report is 5 commits behind head on main. The diff coverage is n/a.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/Kong/ngx_wasm_module/pull/478/graphs/tree.svg?width=650&height=150&src=pr&token=T0PT2Q9IAN&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Kong)](https://app.codecov.io/gh/Kong/ngx_wasm_module/pull/478?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Kong) ```diff @@ Coverage Diff @@ ## main #478 +/- ## ============================================= Coverage 90.14156% 90.14156% ============================================= Files 46 46 Lines 9890 9890 ============================================= Hits 8915 8915 Misses 975 975 ``` | [Flag](https://app.codecov.io/gh/Kong/ngx_wasm_module/pull/478/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Kong) | Coverage Δ | | |---|---|---| | [unit](https://app.codecov.io/gh/Kong/ngx_wasm_module/pull/478/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Kong) | `90.24390% <ø> (ø)` | | | [valgrind](https://app.codecov.io/gh/Kong/ngx_wasm_module/pull/478/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Kong) | `77.84247% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Kong#carryforward-flags-in-the-pull-request-comment) to find out more.