test: check the plugin name for each plugin (#2607)

Close #2579

Makefile

@@ -162,6 +162,10 @@ install: default
 	$(INSTALL) -d $(INST_LUADIR)/apisix/plugins/prometheus
 	$(INSTALL) apisix/plugins/prometheus/*.lua $(INST_LUADIR)/apisix/plugins/prometheus/
 
+	$(INSTALL) -d $(INST_LUADIR)/apisix/plugins/serverless
+	$(INSTALL) apisix/plugins/serverless/*.lua $(INST_LUADIR)/apisix/plugins/serverless/
+
+	$(INSTALL) -d $(INST_LUADIR)/apisix/plugins/limit-count
 	$(INSTALL) -d $(INST_LUADIR)/apisix/plugins/zipkin
 	$(INSTALL) apisix/plugins/zipkin/*.lua $(INST_LUADIR)/apisix/plugins/zipkin/
 

apisix/plugins/serverless-post-function.lua

@@ -14,4 +14,4 @@
 -- See the License for the specific language governing permissions and
 -- limitations under the License.
 --
-return require("apisix.plugins.serverless")("serverless-post-function", -2000)
+return require("apisix.plugins.serverless.init")("serverless-post-function", -2000)

apisix/plugins/serverless-pre-function.lua

@@ -14,4 +14,4 @@
 -- See the License for the specific language governing permissions and
 -- limitations under the License.
 --
-return require("apisix.plugins.serverless")("serverless-pre-function", 10000)
+return require("apisix.plugins.serverless.init")("serverless-pre-function", 10000)

t/plugin/plugin.t

@@ -0,0 +1,54 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+use t::APISIX 'no_plan';
+
+add_block_preprocessor(sub {
+    my ($block) = @_;
+
+    $block->set_value("no_error_log", "[error]");
+    $block->set_value("request", "GET /t");
+
+    $block;
+});
+
+no_long_string();
+no_root_location();
+log_level("info");
+run_tests;
+
+__DATA__
+
+=== TEST 1: ensure all plugins have exposed their name
+--- config
+    location /t {
+        content_by_lua_block {
+            local lfs = require("lfs")
+            for file_name in lfs.dir(ngx.config.prefix() .. "/../../apisix/plugins/") do
+                if string.match(file_name, ".lua$") then
+                    local expected = file_name:sub(1, #file_name - 4)
+                    local plugin = require("apisix.plugins." .. expected)
+                    if plugin.name ~= expected then
+                        ngx.say("expected ", expected, " got ", plugin.name)
+                        return
+                    end
+                end
+            end
+            ngx.say('ok')
+        }
+    }
+--- response_body
+ok

t/stream-plugin/plugin.t

@@ -0,0 +1,54 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+use t::APISIX 'no_plan';
+
+add_block_preprocessor(sub {
+    my ($block) = @_;
+
+    $block->set_value("no_error_log", "[error]");
+
+    $block;
+});
+
+no_long_string();
+no_shuffle();
+no_root_location();
+
+run_tests;
+
+__DATA__
+
+=== TEST 1: ensure all plugins have exposed their name
+--- stream_enable
+--- stream_server_config
+    content_by_lua_block {
+        local lfs = require("lfs")
+        for file_name in lfs.dir(ngx.config.prefix() .. "/../../apisix/stream/plugins/") do
+            if string.match(file_name, ".lua$") then
+                local expected = file_name:sub(1, #file_name - 4)
+                local plugin = require("apisix.stream.plugins." .. expected)
+                if plugin.name ~= expected then
+                    ngx.say("expected ", expected, " got ", plugin.name)
+                    return
+                end
+            end
+        end
+        ngx.say('ok')
+    }
+--- stream_response
+ok