#5977: Add unit test checking the precedence of parsed decls with the…

… same name (test is failing right now)

test/DeclManager.cpp

@@ -515,4 +515,16 @@ TEST_F(DeclManagerTest, ReloadDeclarationsIncreasesParseStamp)
     EXPECT_NE(decl->getParseStamp(), firstParseStamp) << "Parse stamp should have changed on reload";
 }
 
+TEST_F(DeclManagerTest, DeclarationPrecedence)
+{
+    GlobalDeclarationManager().registerDeclType("testdecl", std::make_shared<TestDeclarationCreator>());
+    GlobalDeclarationManager().registerDeclFolder(decl::Type::Material, "testdecls", ".decl");
+
+    expectMaterialIsPresent(decl::Type::Material, "decl/precedence_test/1");
+
+    // The first decl in the file precedence_test1.decl takes precedence over any decls
+    // declared in the same file or other files sorted after precedence_test1.decl
+    expectMaterialContains(decl::Type::Material, "decl/precedence_test/1", "diffusemap textures/numbers/1");
+}
+
 }

test/resources/tdm/testdecls/precedence_test1.decl

@@ -0,0 +1,14 @@
+// Test declarations used for precedence tests
+
+// This first decl makes the cut
+// all following decls with the same type and name are ignored and warned about
+decl/precedence_test/1
+{
+    diffusemap textures/numbers/1
+}
+
+decl/precedence_test/1
+{
+    diffusemap textures/numbers/2
+}
+

test/resources/tdm/testdecls/precedence_test2.decl

@@ -0,0 +1,8 @@
+// Test declarations used for precedence tests
+
+// This one is ignored since a decl in the precedence_test1.decl has already been declared with this name
+// The file precedence_test1.decl is sorted before precedence_test2.decl, so it takes precedence
+decl/precedence_test/1
+{
+    diffusemap textures/numbers/3
+}