verilog-language-server_test: Added register variable and gate symbol…

…s test Signed-off-by: Jan Bylicki <[email protected]>
chipsalliance · Jun 16, 2023 · b4e9fbf · b4e9fbf
1 parent 5ff1475
commit b4e9fbf
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/verilog/tools/ls/document-symbol-filler.cc b/verilog/tools/ls/document-symbol-filler.cc
@@ -107,7 +107,7 @@ void DocumentSymbolFiller::Visit(const verible::SyntaxTreeNode &node) {
       break;
     }
     case verilog::NodeEnum::kRegisterVariable: {
-      const auto* variable_name =
+      const auto *variable_name =
           GetSubtreeAsLeaf(node, NodeEnum::kRegisterVariable, 0);
       if (variable_name) {
         is_visible_node = true;
@@ -118,7 +118,7 @@ void DocumentSymbolFiller::Visit(const verible::SyntaxTreeNode &node) {
       break;
     }
     case verilog::NodeEnum::kGateInstance: {
-      const auto* variable_name =
+      const auto *variable_name =
           GetSubtreeAsLeaf(node, NodeEnum::kGateInstance, 0);
       if (variable_name) {
         is_visible_node = true;

diff --git a/verilog/tools/ls/verilog-language-server_test.cc b/verilog/tools/ls/verilog-language-server_test.cc
@@ -284,6 +284,11 @@ endpackage
 module mini(input clk);
   always@(posedge clk) begin : labelled_block
   end
+
+  reg foo;
+  net bar;
+  some_class baz();
+
 endmodule
 )");
 
@@ -331,9 +336,18 @@ endmodule
 
   // Descent tree into module and find labelled block.
   std::vector<verible::lsp::DocumentSymbol> module = toplevel[1].children;
-  EXPECT_EQ(module.size(), 1);
+  EXPECT_EQ(module.size(), 4);
   EXPECT_EQ(module[0].kind, verible::lsp::SymbolKind::kNamespace);
   EXPECT_EQ(module[0].name, "labelled_block");
+
+  EXPECT_EQ(module[1].kind, verible::lsp::SymbolKind::kVariable);
+  EXPECT_EQ(module[1].name, "foo");
+
+  EXPECT_EQ(module[2].kind, verible::lsp::SymbolKind::kVariable);
+  EXPECT_EQ(module[2].name, "bar");
+
+  EXPECT_EQ(module[3].kind, verible::lsp::SymbolKind::kVariable);
+  EXPECT_EQ(module[3].name, "baz");
 }
 
 // Tests closing of the file in the LS context and checks if the LS