Un-share UnionKeywordUsed

Author: jeongsoolee09

cpp/autosar/src/rules/A9-5-1/UnionKeywordUsedAutosarCpp.ql

@@ -0,0 +1,42 @@
+/**
+ * @id cpp/autosar/unions-used
+ * @name A9-5-1: Unions shall not be used
+ * @description Unions shall not be used. Tagged unions can be used if 'std::variant' is not
+ *              available.
+ * @kind problem
+ * @precision very-high
+ * @problem.severity error
+ * @tags external/autosar/id/a9-5-1
+ *       correctness
+ *       scope/single-translation-unit
+ *       external/autosar/allocated-target/implementation
+ *       external/autosar/enforcement/automated
+ *       external/autosar/obligation/required
+ */
+
+import cpp
+import codingstandards.cpp.autosar
+
+// A tagged union is a class or a struct
+// that has exactly one union and exactly one enum with
+// corresponding member variable that represents
+// the data type in the union
+class TaggedUnion extends UserType {
+  TaggedUnion() {
+    (this instanceof Class or this instanceof Struct) and
+    count(Enum e | e.getParentScope() = this) = 1 and
+    count(Union u | u.getParentScope() = this) = 1 and
+    count(MemberVariable m, Enum e |
+      m.getDeclaringType() = this and
+      e.getDeclaringType() = this and
+      m.getType().getName() = e.getName()
+    ) = 1
+  }
+}
+
+from Union u, string message
+where
+  not isExcluded(u, BannedSyntaxPackage::unionsUsedQuery()) and
+  not u.getParentScope() instanceof TaggedUnion and
+  message = "'" + u.getName() + "' is not a tagged union."
+select u, message

cpp/autosar/src/rules/A9-5-1/UnionsUsed.ql

@@ -0,0 +1,2 @@
+| test.cpp:14:9:14:9 | u | 'u' is not a tagged union. |
+| test.cpp:21:7:21:7 | u | 'u' is not a tagged union. |

cpp/autosar/test/rules/A9-5-1/UnionKeywordUsedAutosarCpp.testref

@@ -0,0 +1 @@
+rules/A9-5-1/UnionsUsed.ql

cpp/autosar/test/rules/A9-5-1/UnionsUsed.expected

@@ -3,24 +3,24 @@ import cpp
 import RuleMetadata
 import codingstandards.cpp.exclusions.RuleMetadata
 
-newtype Banned6Query = TUnionKeywordUsedMisraCppQuery()
+newtype Banned6Query = TUnionKeywordUsedQuery()
 
 predicate isBanned6QueryMetadata(Query query, string queryId, string ruleId, string category) {
   query =
-    // `Query` instance for the `unionKeywordUsedMisraCpp` query
-    Banned6Package::unionKeywordUsedMisraCppQuery() and
+    // `Query` instance for the `unionKeywordUsed` query
+    Banned6Package::unionKeywordUsedQuery() and
   queryId =
-    // `@id` for the `unionKeywordUsedMisraCpp` query
-    "cpp/misra/union-keyword-used-misra-cpp" and
+    // `@id` for the `unionKeywordUsed` query
+    "cpp/misra/union-keyword-used" and
   ruleId = "RULE-12-3-1" and
   category = "required"
 }
 
 module Banned6Package {
-  Query unionKeywordUsedMisraCppQuery() {
+  Query unionKeywordUsedQuery() {
     //autogenerate `Query` type
     result =
-      // `Query` type for `unionKeywordUsedMisraCpp` query
-      TQueryCPP(TBanned6PackageQuery(TUnionKeywordUsedMisraCppQuery()))
+      // `Query` type for `unionKeywordUsed` query
+      TQueryCPP(TBanned6PackageQuery(TUnionKeywordUsedQuery()))
   }
 }

cpp/autosar/test/rules/A9-5-1/UnionsUsed.qlref

@@ -16,7 +16,7 @@ newtype BannedSyntaxQuery =
   TTypedefSpecifierUsedQuery() or
   TAsmDeclarationUsedQuery() or
   TFunctionsDefinedUsingTheEllipsisNotationQuery() or
-  TUnionKeywordUsedAutosarCppQuery() or
+  TUnionsUsedQuery() or
   TCommaOperatorUsedQuery() or
   TUsingDirectivesUsedQuery() or
   TUsingDeclarationsUsedInHeaderFilesQuery() or
@@ -132,11 +132,11 @@ predicate isBannedSyntaxQueryMetadata(Query query, string queryId, string ruleId
   category = "required"
   or
   query =
-    // `Query` instance for the `unionKeywordUsedAutosarCpp` query
-    BannedSyntaxPackage::unionKeywordUsedAutosarCppQuery() and
+    // `Query` instance for the `unionsUsed` query
+    BannedSyntaxPackage::unionsUsedQuery() and
   queryId =
-    // `@id` for the `unionKeywordUsedAutosarCpp` query
-    "cpp/autosar/union-keyword-used-autosar-cpp" and
+    // `@id` for the `unionsUsed` query
+    "cpp/autosar/unions-used" and
   ruleId = "A9-5-1" and
   category = "required"
   or
@@ -262,11 +262,11 @@ module BannedSyntaxPackage {
       TQueryCPP(TBannedSyntaxPackageQuery(TFunctionsDefinedUsingTheEllipsisNotationQuery()))
   }
 
-  Query unionKeywordUsedAutosarCppQuery() {
+  Query unionsUsedQuery() {
     //autogenerate `Query` type
     result =
-      // `Query` type for `unionKeywordUsedAutosarCpp` query
-      TQueryCPP(TBannedSyntaxPackageQuery(TUnionKeywordUsedAutosarCppQuery()))
+      // `Query` type for `unionsUsed` query
+      TQueryCPP(TBannedSyntaxPackageQuery(TUnionsUsedQuery()))
   }
 
   Query commaOperatorUsedQuery() {