add issue references to every skipped test

This commit is contained in:
Jake Hillion 2022-12-30 22:41:09 +00:00 committed by Jake Hillion
parent 6fc24b444c
commit bad14de1e2
7 changed files with 25 additions and 25 deletions

View File

@ -133,7 +133,7 @@ definitions = '''
}]''' }]'''
[cases.anon_struct_ptr] [cases.anon_struct_ptr]
skip = "We don't support pointer to anon-structs yet" skip = "We don't support pointer to anon-structs yet" # https://github.com/facebookexperimental/object-introspection/issues/20
param_types = ["const AnonStructPtrContainer&"] param_types = ["const AnonStructPtrContainer&"]
setup = ''' setup = '''
return AnonStructPtrContainer{ return AnonStructPtrContainer{

View File

@ -14,7 +14,7 @@ definitions = '''
[cases] [cases]
[cases.int] [cases.int]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
oil_disable = "oil can't chase raw pointers safely" oil_disable = "oil can't chase raw pointers safely"
param_types = ["int*"] param_types = ["int*"]
setup = "return new int(1);" setup = "return new int(1);"
@ -33,7 +33,7 @@ definitions = '''
] ]
}]''' }]'''
[cases.int_no_follow] [cases.int_no_follow]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
param_types = ["int*"] param_types = ["int*"]
setup = "return new int(1);" setup = "return new int(1);"
expect_json = '''[{ expect_json = '''[{
@ -44,7 +44,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.int_null] [cases.int_null]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
param_types = ["int*"] param_types = ["int*"]
setup = "return nullptr;" setup = "return nullptr;"
expect_json = '''[{ expect_json = '''[{
@ -57,7 +57,7 @@ definitions = '''
[cases.void] [cases.void]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
oil_disable = "oil can't chase raw pointers safely" oil_disable = "oil can't chase raw pointers safely"
param_types = ["void*"] param_types = ["void*"]
setup = "return new int(1);" setup = "return new int(1);"
@ -70,7 +70,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.void_no_follow] [cases.void_no_follow]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
param_types = ["void*"] param_types = ["void*"]
setup = "return new int(1);" setup = "return new int(1);"
expect_json = '''[{ expect_json = '''[{
@ -81,7 +81,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.void_null] [cases.void_null]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
param_types = ["void*"] param_types = ["void*"]
setup = "return nullptr;" setup = "return nullptr;"
expect_json = '''[{ expect_json = '''[{
@ -94,7 +94,7 @@ definitions = '''
[cases.vector] [cases.vector]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
oil_disable = "oil can't chase raw pointers safely" oil_disable = "oil can't chase raw pointers safely"
param_types = ["std::vector<int>*"] param_types = ["std::vector<int>*"]
setup = "return new std::vector<int>{1,2,3};" setup = "return new std::vector<int>{1,2,3};"
@ -113,7 +113,7 @@ definitions = '''
] ]
}]''' }]'''
[cases.vector_no_follow] [cases.vector_no_follow]
oid_skip = "top-level pointers are skipped over" oid_skip = "top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
param_types = ["std::vector<int>*"] param_types = ["std::vector<int>*"]
setup = "return new std::vector<int>{1,2,3};" setup = "return new std::vector<int>{1,2,3};"
expect_json = '''[{ expect_json = '''[{
@ -124,7 +124,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.vector_null] [cases.vector_null]
oid_skip = "BAD DATA SEGMENT!!! top-level pointers are skipped over" oid_skip = "BAD DATA SEGMENT!!! top-level pointers are skipped over" # https://github.com/facebookexperimental/object-introspection/issues/19
param_types = ["std::vector<int>*"] param_types = ["std::vector<int>*"]
setup = "return nullptr;" setup = "return nullptr;"
expect_json = '''[{ expect_json = '''[{
@ -223,7 +223,7 @@ definitions = '''
{"staticSize":8, "dynamicSize":4, "NOT": {"pointer":0}} {"staticSize":8, "dynamicSize":4, "NOT": {"pointer":0}}
]}]''' ]}]'''
[cases.vector_of_pointers_no_follow] [cases.vector_of_pointers_no_follow]
oid_skip = "pointer field is missing from results" oid_skip = "pointer field is missing from results" # https://github.com/facebookexperimental/object-introspection/issues/21
param_types = ["const std::vector<int*>&"] param_types = ["const std::vector<int*>&"]
setup = "return {{new int(1), nullptr, new int(3)}};" setup = "return {{new int(1), nullptr, new int(3)}};"
expect_json = '''[{ expect_json = '''[{

View File

@ -14,7 +14,7 @@ definitions = '''
[cases] [cases]
[cases.raw] [cases.raw]
oid_skip = "function pointers are not handled correctly" oid_skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
param_types = ["const FuncPtrStruct&"] param_types = ["const FuncPtrStruct&"]
setup = "return {{myFunction}};" setup = "return {{myFunction}};"
expect_json = '''[{ expect_json = '''[{
@ -29,7 +29,7 @@ definitions = '''
}] }]
}]''' }]'''
[cases.raw_chase] # We should never chase function pointers [cases.raw_chase] # We should never chase function pointers
oid_skip = "function pointers are not handled correctly" oid_skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
oil_disable = "oil can't chase raw pointers safely" oil_disable = "oil can't chase raw pointers safely"
param_types = ["const FuncPtrStruct&"] param_types = ["const FuncPtrStruct&"]
setup = "return {{myFunction}};" setup = "return {{myFunction}};"
@ -46,7 +46,7 @@ definitions = '''
}] }]
}]''' }]'''
[cases.raw_null] [cases.raw_null]
oid_skip = "function pointers are not handled correctly" oid_skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
param_types = ["const FuncPtrStruct&"] param_types = ["const FuncPtrStruct&"]
setup = "return {{nullptr}};" setup = "return {{nullptr}};"
expect_json = '''[{ expect_json = '''[{
@ -62,7 +62,7 @@ definitions = '''
}]''' }]'''
[cases.std_function] [cases.std_function]
oid_skip = "function pointers are not handled correctly" oid_skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
param_types = ["std::function<void(int)> &"] param_types = ["std::function<void(int)> &"]
setup = "return myFunction;" setup = "return myFunction;"
expect_json = '''[{ expect_json = '''[{
@ -73,7 +73,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.std_function_chase] # We should never chase function pointers [cases.std_function_chase] # We should never chase function pointers
oid_skip = "function pointers are not handled correctly" oid_skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
oil_disable = "oil can't chase raw pointers safely" oil_disable = "oil can't chase raw pointers safely"
param_types = ["std::function<void(int)> &"] param_types = ["std::function<void(int)> &"]
setup = "return myFunction;" setup = "return myFunction;"
@ -86,7 +86,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.std_function_null] [cases.std_function_null]
oid_skip = "function pointers are not handled correctly" oid_skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
param_types = ["std::function<void(int)> &"] param_types = ["std::function<void(int)> &"]
setup = "return nullptr;" setup = "return nullptr;"
expect_json = '''[{ expect_json = '''[{

View File

@ -20,7 +20,7 @@ definitions = '''
[cases] [cases]
[cases.raw] [cases.raw]
oil_disable = "oil can't chase raw pointers safely" oil_disable = "oil can't chase raw pointers safely"
oid_skip = "oid codegen fails on this" oid_skip = "oid codegen fails on this" # https://github.com/facebookexperimental/object-introspection/issues/17
param_types = ["IncompleteType*"] param_types = ["IncompleteType*"]
setup = "return static_cast<IncompleteType*>(::operator new(5));" setup = "return static_cast<IncompleteType*>(::operator new(5));"
cli_options = ["--chase-raw-pointers"] cli_options = ["--chase-raw-pointers"]
@ -32,7 +32,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.raw_no_follow] [cases.raw_no_follow]
oid_skip = "oid codegen fails on this" oid_skip = "oid codegen fails on this" # https://github.com/facebookexperimental/object-introspection/issues/17
param_types = ["IncompleteType*"] param_types = ["IncompleteType*"]
setup = "return static_cast<IncompleteType*>(::operator new(5));" setup = "return static_cast<IncompleteType*>(::operator new(5));"
expect_json = '''[{ expect_json = '''[{
@ -43,7 +43,7 @@ definitions = '''
"NOT": "members" "NOT": "members"
}]''' }]'''
[cases.raw_null] [cases.raw_null]
oid_skip = "oid codegen fails on this" oid_skip = "oid codegen fails on this" # https://github.com/facebookexperimental/object-introspection/issues/17
param_types = ["IncompleteType*"] param_types = ["IncompleteType*"]
setup = "return nullptr;" setup = "return nullptr;"
expect_json = '''[{ expect_json = '''[{

View File

@ -9,7 +9,7 @@ definitions = '''
''' '''
[cases] [cases]
[cases.int_ref] [cases.int_ref]
skip = "references are being treated as raw pointers" skip = "references are being treated as raw pointers" # https://github.com/facebookexperimental/object-introspection/issues/16
param_types = ["const IntRef&"] param_types = ["const IntRef&"]
setup = "return {{*new int(1)}};" setup = "return {{*new int(1)}};"
expect_json = '''[{ expect_json = '''[{
@ -24,7 +24,7 @@ definitions = '''
} }
]}]''' ]}]'''
[cases.vector_ref] [cases.vector_ref]
skip = "references are being treated as raw pointers" skip = "references are being treated as raw pointers" # https://github.com/facebookexperimental/object-introspection/issues/16
param_types = ["const VectorRef&"] param_types = ["const VectorRef&"]
setup = "return {{*new std::vector<int>{1,2,3}}};" setup = "return {{*new std::vector<int>{1,2,3}}};"
expect_json = '''[{ expect_json = '''[{

View File

@ -1,7 +1,7 @@
includes = ["unordered_map"] includes = ["unordered_map"]
[cases] [cases]
[cases.int_int] [cases.int_int]
skip = true skip = true # https://github.com/facebookexperimental/object-introspection/issues/15
param_types = ["const std::unordered_map<int, int>&"] param_types = ["const std::unordered_map<int, int>&"]
setup = "return {{{1,2},{3,4}}};" setup = "return {{{1,2},{3,4}}};"
# TODO confirm this JSON is correct # TODO confirm this JSON is correct

View File

@ -9,12 +9,12 @@ includes = ["vector"]
setup = "return {{1,2,3}};" setup = "return {{1,2,3}};"
expect_json = '[{"staticSize":24, "dynamicSize":12, "length":3, "capacity":3, "elementStaticSize":4}]' expect_json = '[{"staticSize":24, "dynamicSize":12, "length":3, "capacity":3, "elementStaticSize":4}]'
[cases.bool_empty] [cases.bool_empty]
skip = true skip = true # https://github.com/facebookexperimental/object-introspection/issues/14
param_types = ["const std::vector<bool>&"] param_types = ["const std::vector<bool>&"]
setup = "return {};" setup = "return {};"
expect_json = '[{"staticSize":40, "dynamicSize":0, "length":0, "capacity":0, "elementStaticSize":0.125}]' expect_json = '[{"staticSize":40, "dynamicSize":0, "length":0, "capacity":0, "elementStaticSize":0.125}]'
[cases.bool_some] [cases.bool_some]
skip = true skip = true # https://github.com/facebookexperimental/object-introspection/issues/14
param_types = ["const std::vector<bool>&"] param_types = ["const std::vector<bool>&"]
setup = "return {{true, false, true}};" setup = "return {{true, false, true}};"
expect_json = '[{"staticSize":40,"dynamicSize":8, "length":3, "capacity":64, "elementStaticSize":0.125}]' expect_json = '[{"staticSize":40,"dynamicSize":8, "length":3, "capacity":64, "elementStaticSize":0.125}]'