[njs] Tests: simplified stream_js_import2.t.

Dmitry Volyntsev xeioex at nginx.com
Tue Apr 2 15:52:20 UTC 2024


details:   https://hg.nginx.org/njs/rev/498b2387ef04
branches:  
changeset: 2305:498b2387ef04
user:      Dmitry Volyntsev <xeioex at nginx.com>
date:      Mon Apr 01 23:13:23 2024 -0700
description:
Tests: simplified stream_js_import2.t.

The test should only verify js_import directive in server context
according to its description.

diffstat:

 nginx/t/stream_js_import2.t |  59 +--------------------------------------------
 1 files changed, 1 insertions(+), 58 deletions(-)

diffs (81 lines):

diff -r e16e93f43303 -r 498b2387ef04 nginx/t/stream_js_import2.t
--- a/nginx/t/stream_js_import2.t	Mon Apr 01 23:13:21 2024 -0700
+++ b/nginx/t/stream_js_import2.t	Mon Apr 01 23:13:23 2024 -0700
@@ -42,76 +42,19 @@ stream {
         js_set $test foo.bar.p;
         return  $test;
     }
-
-    server {
-        listen      127.0.0.1:8082;
-
-        js_import lib.js;
-
-        js_access   lib.access;
-        js_preread  lib.preread;
-        js_filter   lib.filter;
-        proxy_pass  127.0.0.1:8083;
-    }
-
-    server {
-        listen  127.0.0.1:8083;
-        return  "x";
-    }
 }
 
 EOF
 
-$t->write_file('lib.js', <<EOF);
-    var res = '';
-
-    function access(s) {
-        res += '1';
-        s.allow();
-    }
-
-    function preread(s) {
-        s.on('upload', function (data) {
-            res += '2';
-            if (res.length >= 3) {
-                s.done();
-            }
-        });
-    }
-
-    function filter(s) {
-        s.on('upload', function(data, flags) {
-            s.send(data);
-            res += '3';
-        });
-
-        s.on('download', function(data, flags) {
-            if (!flags.last) {
-                res += '4';
-                s.send(data);
-
-            } else {
-                res += '5';
-                s.send(res, {last:1});
-                s.off('download');
-            }
-        });
-    }
-
-    export default {access, preread, filter};
-
-EOF
-
 $t->write_file('main.js', <<EOF);
     export default {bar: {p(s) {return "P-TEST"}}};
 
 EOF
 
-$t->try_run('no njs available')->plan(2);
+$t->try_run('no njs available')->plan(1);
 
 ###############################################################################
 
 is(stream('127.0.0.1:' . port(8081))->read(), 'P-TEST', 'foo.bar.p');
-is(stream('127.0.0.1:' . port(8082))->io('0'), 'x122345', 'lib.access');
 
 ###############################################################################


More information about the nginx-devel mailing list