Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ref(profiling): move test helpers to separate code unit; prefix with _sentry #3873

ref: move test helpers to separate code unit; prefix with _sentry

9d0cb71
Select commit
Loading
Failed to load commit list.
Merged

ref(profiling): move test helpers to separate code unit; prefix with _sentry #3873

ref: move test helpers to separate code unit; prefix with _sentry
9d0cb71
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Apr 23, 2024 in 0s

71.794% of diff hit (target 90.639%)

View this Pull Request on Codecov

71.794% of diff hit (target 90.639%)

Annotations

Check warning on line 35 in Sources/Sentry/Profiling/SentryProfilerTestHelpers.m

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerTestHelpers.m#L35

Added line #L35 was not covered by tests

Check warning on line 39 in Sources/Sentry/Profiling/SentryProfilerTestHelpers.m

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerTestHelpers.m#L37-L39

Added lines #L37 - L39 were not covered by tests

Check warning on line 41 in Sources/Sentry/Profiling/SentryProfilerTestHelpers.m

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerTestHelpers.m#L41

Added line #L41 was not covered by tests

Check warning on line 50 in Sources/Sentry/Profiling/SentryProfilerTestHelpers.m

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/Profiling/SentryProfilerTestHelpers.m#L50

Added line #L50 was not covered by tests

Check warning on line 115 in Sources/Sentry/SentryProfileTimeseries.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/SentryProfileTimeseries.mm#L112-L115

Added lines #L112 - L115 were not covered by tests

Check warning on line 120 in Sources/Sentry/SentryProfileTimeseries.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/SentryProfileTimeseries.mm#L120

Added line #L120 was not covered by tests

Check warning on line 133 in Sources/Sentry/SentryProfileTimeseries.mm

See this annotation in the file changed.

@codecov codecov / codecov/patch

Sources/Sentry/SentryProfileTimeseries.mm#L130-L133

Added lines #L130 - L133 were not covered by tests