Skip to content

Commit 7ae640c

Browse files
committed
Fix OS specific tests
1 parent f2e6671 commit 7ae640c

4 files changed

Lines changed: 12 additions & 7 deletions

File tree

csharp/ql/test/library-tests/cil/attributes/attribute.expected

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,6 @@ attrNoArg
99
| Interop.Sys.<>c__DisplayClass37_0 | [CompilerGeneratedAttribute(...)] |
1010
| Interop.Sys.FileStatusFlags | [FlagsAttribute(...)] |
1111
| Interop.Sys.OpenFlags | [FlagsAttribute(...)] |
12-
| Interop.libobjc.NSOperatingSystemVersion.majorVersion | [NativeIntegerAttribute(...)] |
13-
| Interop.libobjc.NSOperatingSystemVersion.minorVersion | [NativeIntegerAttribute(...)] |
14-
| Interop.libobjc.NSOperatingSystemVersion.patchVersion | [NativeIntegerAttribute(...)] |
1512
| InteropErrorExtensions | [ExtensionAttribute(...)] |
1613
| Microsoft.CodeAnalysis.EmbeddedAttribute | [CompilerGeneratedAttribute(...)] |
1714
| Microsoft.CodeAnalysis.EmbeddedAttribute | [EmbeddedAttribute(...)] |

csharp/ql/test/library-tests/cil/attributes/attribute.ql

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,11 @@
11
import semmle.code.cil.Attribute
22
import semmle.code.cil.Declaration
33

4+
private predicate isOsSpecific(Declaration d) { d.getQualifiedName().matches("%libobjc%") }
5+
46
query predicate attrNoArg(string dec, string attr) {
57
exists(Declaration d, Attribute a |
8+
not isOsSpecific(d) and
69
a.getDeclaration() = d and
710
not exists(a.getAnArgument())
811
|
@@ -14,6 +17,7 @@ query predicate attrNoArg(string dec, string attr) {
1417
query predicate attrArgNamed(string dec, string attr, string name, string value) {
1518
exists(Declaration d, Attribute a |
1619
a.getDeclaration() = d and
20+
not isOsSpecific(d) and
1721
a.getNamedArgument(name) = value
1822
|
1923
dec = d.toStringWithTypes() and
@@ -24,6 +28,7 @@ query predicate attrArgNamed(string dec, string attr, string name, string value)
2428
query predicate attrArgPositional(string dec, string attr, int index, string value) {
2529
exists(Declaration d, Attribute a |
2630
a.getDeclaration() = d and
31+
not isOsSpecific(d) and
2732
a.getArgument(index) = value
2833
|
2934
dec = d.toStringWithTypes() and

csharp/ql/test/library-tests/cil/typeAnnotations/typeAnnotations.expected

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -923,7 +923,6 @@
923923
| Parameter 1 of SetException | parameter | 32 |
924924
| Parameter 1 of SetNotificationForWaitCompletion | parameter | 32 |
925925
| Parameter 1 of SetValue | parameter | 32 |
926-
| Parameter 1 of SkipWhitespace | parameter | 32 |
927926
| Parameter 1 of SnapForObservation | parameter | 32 |
928927
| Parameter 1 of SplitName | parameter | 32 |
929928
| Parameter 1 of Start | parameter | 32 |
@@ -1038,7 +1037,6 @@
10381037
| Parameter 1 of UInt32ToNumber | parameter | 32 |
10391038
| Parameter 1 of UInt64ToNumber | parameter | 32 |
10401039
| Parameter 1 of Unscale | parameter | 32 |
1041-
| Parameter 1 of ValidateVariableAndValue | parameter | 32 |
10421040
| Parameter 1 of VarDecCmp | parameter | 32 |
10431041
| Parameter 1 of VarDecCmpSub | parameter | 32 |
10441042
| Parameter 1 of VarDecDiv | parameter | 32 |
@@ -1318,7 +1316,6 @@
13181316
| Parameter 2 of TryGetStringValue | parameter | 32 |
13191317
| Parameter 2 of TryGetTimeZone | parameter | 32 |
13201318
| Parameter 2 of TryGetTimeZoneFromLocalMachine | parameter | 32 |
1321-
| Parameter 2 of TryGetUserNameFromPasswd | parameter | 32 |
13221319
| Parameter 2 of TryGetValue | parameter | 32 |
13231320
| Parameter 2 of TryGetValueWorker | parameter | 32 |
13241321
| Parameter 2 of TryLoadTzFile | parameter | 32 |

csharp/ql/test/library-tests/cil/typeAnnotations/typeAnnotations.ql

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,5 +21,11 @@ private string elementType(Element e) {
2121
}
2222

2323
from Element e, int i
24-
where cil_type_annotation(e, i)
24+
where
25+
cil_type_annotation(e, i) and
26+
(
27+
not e instanceof Parameter or
28+
e.(Parameter).getDeclaringElement().(Method).getDeclaringType().getQualifiedName() !=
29+
"System.Environment" // There are OS specific methods in this class
30+
)
2531
select e.toString(), elementType(e), i

0 commit comments

Comments
 (0)