mirror of
https://github.com/JakeHillion/object-introspection.git
synced 2024-11-14 22:36:54 +00:00
4975b6e9fa
Previously we tested different feature flags by using the `cli_options` field in the test `.toml`. This works for OID but doesn't work for JIT OIL and won't work for AoT OIL when those tests get added. This change adds a new higher level `features` field to the test `.toml` which adds the features to the config file as a prefix. This works with all methods of generation. Change the existing `cli_options` features to `features` except for where they're testing something specific. Enable tests that were previously disabled for OIL but only didn't work because of not being able to enable features. Change pointer tests that are currently broken for OIL from `oil_disable` to `oil_skip` - they can work, but codegen is broken for them at the minute. Test plan: - CI - `make test` is no worse
97 lines
3.1 KiB
TOML
97 lines
3.1 KiB
TOML
includes = ["functional"]
|
||
|
||
definitions = '''
|
||
void myFunction(int x) {
|
||
(void)x;
|
||
}
|
||
|
||
// Put this function pointer inside a struct since the test framework doesn't
|
||
// support the syntax required for passing raw function pointers as arguments
|
||
struct FuncPtrStruct {
|
||
void (*p)(int);
|
||
};
|
||
'''
|
||
|
||
[cases]
|
||
[cases.raw]
|
||
skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
|
||
param_types = ["const FuncPtrStruct&"]
|
||
setup = "return {{myFunction}};"
|
||
expect_json = '''[{
|
||
"staticSize": 8,
|
||
"dynamicSize": 0,
|
||
"members": [{
|
||
"typeName": "void (*)(int)",
|
||
"staticSize": 8,
|
||
"dynamicSize": 0,
|
||
"NOT": {"pointer": 0},
|
||
"NOT": "members"
|
||
}]
|
||
}]'''
|
||
[cases.raw_chase] # We should never chase function pointers
|
||
skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
|
||
param_types = ["const FuncPtrStruct&"]
|
||
setup = "return {{myFunction}};"
|
||
features = ["chase-raw-pointers"]
|
||
expect_json = '''[{
|
||
"staticSize": 8,
|
||
"dynamicSize": 0,
|
||
"members": [{
|
||
"typeName": "void (*)(int)",
|
||
"staticSize": 8,
|
||
"dynamicSize": 0,
|
||
"NOT": {"pointer": 0},
|
||
"NOT": "members"
|
||
}]
|
||
}]'''
|
||
[cases.raw_null]
|
||
skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
|
||
param_types = ["const FuncPtrStruct&"]
|
||
setup = "return {{nullptr}};"
|
||
expect_json = '''[{
|
||
"staticSize": 8,
|
||
"dynamicSize": 0,
|
||
"members": [{
|
||
"typeName": "void (*)(int)",
|
||
"staticSize": 8,
|
||
"dynamicSize": 0,
|
||
"pointer": 0,
|
||
"NOT": "members"
|
||
}]
|
||
}]'''
|
||
|
||
[cases.std_function]
|
||
skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
|
||
param_types = ["std::function<void(int)> &"]
|
||
setup = "return myFunction;"
|
||
expect_json = '''[{
|
||
"typeName": "function<void (int)>",
|
||
"staticSize": 32,
|
||
"dynamicSize": 0,
|
||
"NOT": {"pointer": 0},
|
||
"NOT": "members"
|
||
}]'''
|
||
[cases.std_function_chase] # We should never chase function pointers
|
||
skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
|
||
param_types = ["std::function<void(int)> &"]
|
||
setup = "return myFunction;"
|
||
features = ["chase-raw-pointers"]
|
||
expect_json = '''[{
|
||
"typeName": "function<void (int)>",
|
||
"staticSize": 32,
|
||
"dynamicSize": 0,
|
||
"NOT": {"pointer": 0},
|
||
"NOT": "members"
|
||
}]'''
|
||
[cases.std_function_null]
|
||
skip = "function pointers are not handled correctly" # https://github.com/facebookexperimental/object-introspection/issues/22
|
||
param_types = ["std::function<void(int)> &"]
|
||
setup = "return nullptr;"
|
||
expect_json = '''[{
|
||
"typeName": "function<void (int)>",
|
||
"staticSize": 32,
|
||
"dynamicSize": 0,
|
||
"pointer": 0,
|
||
"NOT": "members"
|
||
}]'''
|