Skip to content

Add LLVM coverage conversion to LCOV format#55860

Open
andrewdacenko wants to merge 1 commit intofacebook:mainfrom
andrewdacenko:export-D90135000
Open

Add LLVM coverage conversion to LCOV format#55860
andrewdacenko wants to merge 1 commit intofacebook:mainfrom
andrewdacenko:export-D90135000

Conversation

@andrewdacenko
Copy link
Contributor

Summary:
Changelog: [Internal]

This diff adds the ability to convert LLVM coverage JSON output to LCOV format for C++ code coverage in react-native-fantom.

Key additions:

  • convertLLVMCoverage.js: Converts LLVM coverage data (segments, branches, functions) to LCOV format lines
  • types.flow.js: Flow type definitions for LLVM coverage data structures
  • Test utilities and fixtures for validating the conversion with real coverage data from various C++ files (AppSettings.cpp, Class.h, DevSettingsModule.h, NativeFantom.cpp/h, RawPropsKey.cpp)

This enables reporting C++ code coverage from Fantom tests in a standard format that can be consumed by coverage tools.

Differential Revision: D90135000

Summary:
Changelog: [Internal]

This diff adds the ability to convert LLVM coverage JSON output to LCOV format for C++ code coverage in react-native-fantom.

Key additions:
- `convertLLVMCoverage.js`: Converts LLVM coverage data (segments, branches, functions) to LCOV format lines
- `types.flow.js`: Flow type definitions for LLVM coverage data structures
- Test utilities and fixtures for validating the conversion with real coverage data from various C++ files (AppSettings.cpp, Class.h, DevSettingsModule.h, NativeFantom.cpp/h, RawPropsKey.cpp)

This enables reporting C++ code coverage from Fantom tests in a standard format that can be consumed by coverage tools.

Differential Revision: D90135000
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 2, 2026
@meta-codesync
Copy link

meta-codesync bot commented Mar 2, 2026

@andrewdacenko has exported this pull request. If you are a Meta employee, you can view the originating Diff in D90135000.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants