Skip to content

Commit 88872a4

Browse files
Update test expectations -- codeql run test expects noncompliance.
1 parent 89464ce commit 88872a4

10 files changed

+26
-2
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
| Wcast-function-type.cpp:0:0:0:0 | Wcast-function-type.cpp | No warning-level options were used in the compilation of 'Wcast-function-type.cpp'. |

cpp/autosar/test/rules/A1-1-2.2/CompilerWarningLevelNotInCompliance.expected.clang

Whitespace-only changes.

cpp/autosar/test/rules/A1-1-2.2/CompilerWarningLevelNotInCompliance.expected.gcc

Whitespace-only changes.

cpp/autosar/test/rules/A1-1-2.2/CompilerWarningLevelNotInCompliance.expected.qcc

Whitespace-only changes.
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,14 @@
11
// semmle-extractor-options: --clang -std=c++14 -Wcast-function-type
2-
// COMPLIANT
2+
// COMPLIAN
3+
4+
// NOTE: When tested with `codeql test run`, the test extractor provides `-w`
5+
// which overrides `-Wcast-function-type` and causes this test case to be
6+
// non-compliant.
7+
//
8+
// However, when tested with our compiler matrix tests, this test db is built
9+
// via `codeql database create --command="..."`, and the `-w` flag will NOT be
10+
// used. This means the `-Wcast-function-type` flag is active and the test case
11+
// is compliant.
12+
//
13+
// Therefore, the .expected file for this test expects non-compliance, and the
14+
// .expected.gcc and .expected.clang files expect this test to be compliant.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
| Wall.cpp:0:0:0:0 | Wall.cpp | No warning-level options were used in the compilation of 'Wall.cpp'. |

cpp/autosar/test/rules/A1-1-2/CompilerWarningLevelNotInCompliance.expected.clang

Whitespace-only changes.

cpp/autosar/test/rules/A1-1-2/CompilerWarningLevelNotInCompliance.expected.gcc

Whitespace-only changes.

cpp/autosar/test/rules/A1-1-2/CompilerWarningLevelNotInCompliance.expected.qcc

Whitespace-only changes.
+11-1
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,12 @@
11
// semmle-extractor-options: --clang -std=c++14 -Wall
2-
// COMPLIANT
2+
// COMPLIANT
3+
4+
// NOTE: When tested with `codeql test run`, the test extractor provides `-w`
5+
// which overrides `-Wall` and causes this test case to be non-compliant.
6+
//
7+
// However, when tested with our compiler matrix tests, this test db is built
8+
// via `codeql database create --command="..."`, and the `-w` flag will NOT be
9+
// used. This means the `-Wall` flag is active and the test case is compliant.
10+
//
11+
// Therefore, the .expected file for this test expects non-compliance, and the
12+
// .expected.gcc and .expected.clang files expect this test to be compliant.

0 commit comments

Comments
 (0)