Fix error in sorting properties that differ by property name

Add unit tests demonstrating the fix.

Author: thk123

src/goto-checker/report_util.cpp

@@ -234,8 +234,8 @@ is_property_less_than(const propertyt &property1, const propertyt &property2)
   const auto left_id_number = left_split.second;
 
   const auto right_split = split_property_id(property2.first);
-  const auto right_id_name = left_split.first;
-  const auto right_id_number = left_split.second;
+  const auto right_id_name = right_split.first;
+  const auto right_id_number = right_split.second;
 
   if(left_id_name != right_id_name)
     return left_id_name < right_id_name;

unit/goto-checker/report_util/is_property_less_than.cpp

@@ -69,4 +69,34 @@ TEST_CASE("Comparing two properties", "[core][goto-checker][report_util]")
       }
     }
   }
+  SECTION("Compare property ids")
+  {
+    goto_programt::instructionst instructions;
+    instructions.push_back(instruction_for_location("fileA", "funcA", 1));
+
+    std::vector<irep_idt> properties;
+    properties.push_back("A.1");
+    properties.push_back("A.2");
+    properties.push_back("B.1");
+    properties.push_back("B.2");
+
+    for(auto first_property = properties.begin();
+        first_property != properties.end();
+        ++first_property)
+    {
+      for(auto second_property = std::next(first_property);
+          second_property != properties.end();
+          ++second_property)
+      {
+        const propertyt p1 =
+          property(*first_property, test_info(instructions.begin()));
+        const propertyt p2 =
+          property(*second_property, test_info(instructions.begin()));
+        INFO(p1);
+        INFO(p2);
+        REQUIRE(is_property_less_than(p1, p2));
+        REQUIRE_FALSE(is_property_less_than(p2, p1));
+      }
+    }
+  }
 }