Skip to content

Commit cf35ecf

Browse files
committed
simplify tests
1 parent a46121e commit cf35ecf

File tree

1 file changed

+50
-76
lines changed

1 file changed

+50
-76
lines changed

clang/unittests/Analysis/FlowSensitive/UncheckedOptionalAccessModelTest.cpp

Lines changed: 50 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -3868,20 +3868,16 @@ TEST_P(UncheckedOptionalAccessTest,
38683868
ExpectDiagnosticsFor(R"cc(
38693869
#include "unchecked_optional_access_test.h"
38703870
3871-
class A {
3872-
public:
3873-
const $ns::$optional<int>& get() const { return x; }
3871+
struct A {
3872+
const $ns::$optional<int>& get() const { return x; }
38743873
3875-
private:
3876-
$ns::$optional<int> x;
3874+
$ns::$optional<int> x;
38773875
};
38783876
3879-
class B {
3880-
public:
3881-
const A& getA() const { return a; }
3877+
struct B {
3878+
const A& getA() const { return a; }
38823879
3883-
private:
3884-
A a;
3880+
A a;
38853881
};
38863882
38873883
void target(B& b) {
@@ -3898,20 +3894,16 @@ TEST_P(
38983894
ExpectDiagnosticsFor(R"cc(
38993895
#include "unchecked_optional_access_test.h"
39003896
3901-
class A {
3902-
public:
3903-
const $ns::$optional<int>& get() const { return x; }
3897+
struct A {
3898+
const $ns::$optional<int>& get() const { return x; }
39043899
3905-
private:
3906-
$ns::$optional<int> x;
3900+
$ns::$optional<int> x;
39073901
};
39083902
3909-
class B {
3910-
public:
3911-
const A& getA() const { return a; }
3903+
struct B {
3904+
const A& getA() const { return a; }
39123905
3913-
private:
3914-
A a;
3906+
A a;
39153907
};
39163908
39173909
void target(B& b) {
@@ -3925,20 +3917,16 @@ TEST_P(UncheckedOptionalAccessTest,
39253917
ExpectDiagnosticsFor(R"cc(
39263918
#include "unchecked_optional_access_test.h"
39273919
3928-
class A {
3929-
public:
3930-
const $ns::$optional<int>& get() const { return x; }
3920+
struct A {
3921+
const $ns::$optional<int>& get() const { return x; }
39313922
3932-
private:
3933-
$ns::$optional<int> x;
3923+
$ns::$optional<int> x;
39343924
};
39353925
3936-
class B {
3937-
public:
3938-
const A& getA() const { return a; }
3926+
struct B {
3927+
const A& getA() const { return a; }
39393928
3940-
private:
3941-
A a;
3929+
A a;
39423930
};
39433931
39443932
void target(B& b) {
@@ -3951,24 +3939,20 @@ TEST_P(UncheckedOptionalAccessTest,
39513939
}
39523940

39533941
TEST_P(UncheckedOptionalAccessTest,
3954-
ConstRefAccessorToOptionalViaConstValueAccessorToHoldingObject) {
3942+
ConstRefAccessorToOptionalViaAccessorToHoldingObjectByValue) {
39553943
ExpectDiagnosticsFor(R"cc(
39563944
#include "unchecked_optional_access_test.h"
39573945
3958-
class A {
3959-
public:
3960-
const $ns::$optional<int>& get() const { return x; }
3946+
struct A {
3947+
const $ns::$optional<int>& get() const { return x; }
39613948
3962-
private:
3963-
$ns::$optional<int> x;
3949+
$ns::$optional<int> x;
39643950
};
39653951
3966-
class B {
3967-
public:
3968-
const A copyA() const { return a; }
3952+
struct B {
3953+
const A copyA() const { return a; }
39693954
3970-
private:
3971-
A a;
3955+
A a;
39723956
};
39733957
39743958
void target(B& b) {
@@ -3984,20 +3968,16 @@ TEST_P(UncheckedOptionalAccessTest,
39843968
ExpectDiagnosticsFor(R"cc(
39853969
#include "unchecked_optional_access_test.h"
39863970
3987-
class A {
3988-
public:
3989-
const $ns::$optional<int>& get() const { return x; }
3971+
struct A {
3972+
const $ns::$optional<int>& get() const { return x; }
39903973
3991-
private:
3992-
$ns::$optional<int> x;
3974+
$ns::$optional<int> x;
39933975
};
39943976
3995-
class B {
3996-
public:
3997-
A& getA() { return a; }
3977+
struct B {
3978+
A& getA() { return a; }
39983979
3999-
private:
4000-
A a;
3980+
A a;
40013981
};
40023982
40033983
void target(B& b) {
@@ -4014,23 +3994,20 @@ TEST_P(
40143994
ExpectDiagnosticsFor(R"cc(
40153995
#include "unchecked_optional_access_test.h"
40163996
4017-
class A {
4018-
public:
4019-
const $ns::$optional<int>& get() const { return x; }
4020-
private:
4021-
$ns::$optional<int> x;
3997+
struct A {
3998+
const $ns::$optional<int>& get() const { return x; }
3999+
4000+
$ns::$optional<int> x;
40224001
};
40234002
4024-
class B {
4025-
public:
4026-
const A& getA() const { return a; }
4003+
struct B {
4004+
const A& getA() const { return a; }
40274005
4028-
A& getA() { return a; }
4006+
A& getA() { return a; }
40294007
4030-
void clear() { a = A{}; }
4008+
void clear() { a = A{}; }
40314009
4032-
private:
4033-
A a;
4010+
A a;
40344011
};
40354012
40364013
void target(B& b) {
@@ -4055,23 +4032,20 @@ TEST_P(
40554032
ExpectDiagnosticsFor(R"cc(
40564033
#include "unchecked_optional_access_test.h"
40574034
4058-
class A {
4059-
public:
4060-
const $ns::$optional<int>& get() const { return x; }
4061-
private:
4062-
$ns::$optional<int> x;
4035+
struct A {
4036+
const $ns::$optional<int>& get() const { return x; }
4037+
4038+
$ns::$optional<int> x;
40634039
};
40644040
4065-
class B {
4066-
public:
4067-
const A& getA() const { return a; }
4041+
struct B {
4042+
const A& getA() const { return a; }
40684043
4069-
void callWithoutChanges() const {
4070-
// no-op
4071-
}
4044+
void callWithoutChanges() const {
4045+
// no-op
4046+
}
40724047
4073-
private:
4074-
A a;
4048+
A a;
40754049
};
40764050
40774051
void target(B& b) {

0 commit comments

Comments
 (0)