Skip to content

feat(r8-tests): Add R8 special formats tests#78

Closed
romtsn wants to merge 34 commits intomasterfrom
rz/feat/r8-tests-special-formats
Closed

feat(r8-tests): Add R8 special formats tests#78
romtsn wants to merge 34 commits intomasterfrom
rz/feat/r8-tests-special-formats

Conversation

@romtsn
Copy link
Copy Markdown
Member

@romtsn romtsn commented Jan 19, 2026

Part of #40

R8 Retrace Test Coverage

  • Inlining (13 tests) - inline frames, line numbers, source files, rewriteFrame rules
  • Ambiguous Methods (21 tests) - multiple methods → same obfuscated name, alternatives
  • Outlines (4 tests) - outline callsites, nested outlines, inline-in-outline
  • Synthetic/Lambda (3 tests) - lambda methods, synthetic bridges
  • Source File Edge Cases (7 tests) - colons, unicode, multiple dots, synthesized names
  • Line Number Handling (10 tests) - spans, preambles, invalid ranges, missing lines
  • Exception Handling (6 tests) - obfuscated exceptions, suppressed, circular refs
  • Method Overloading (4 tests) - overloads with/without ranges
  • Special Formats (4 tests) - Java 9+ modules, logcat format, auto-detect

@romtsn romtsn requested a review from loewenheim January 20, 2026 13:07
Comment thread tests/r8-special-formats.NOTES.md Outdated
Comment thread tests/r8-special-formats.NOTES.md Outdated
Comment thread tests/r8-special-formats.NOTES.md Outdated
@romtsn romtsn force-pushed the rz/feat/r8-tests-method-overloading branch from 1f47546 to 94d628e Compare January 30, 2026 12:46
@romtsn romtsn changed the base branch from rz/feat/r8-tests-method-overloading to rz/fix/r8-tests-method-overloading January 30, 2026 14:28
@romtsn romtsn force-pushed the rz/feat/r8-tests-special-formats branch from d1f689b to 5aeaf34 Compare January 30, 2026 14:31
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (rz/fix/r8-tests-method-overloading@24a5ed0). Learn more about missing BASE report.

Additional details and impacted files
@@                          Coverage Diff                          @@
##             rz/fix/r8-tests-method-overloading      #78   +/-   ##
=====================================================================
  Coverage                                      ?   84.92%           
=====================================================================
  Files                                         ?        9           
  Lines                                         ?     3754           
  Branches                                      ?        0           
=====================================================================
  Hits                                          ?     3188           
  Misses                                        ?      566           
  Partials                                      ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@romtsn romtsn changed the base branch from rz/fix/r8-tests-method-overloading to master February 16, 2026 14:36
romtsn and others added 2 commits February 16, 2026 15:40
…ecial-formats

# Conflicts:
#	src/builder.rs
#	src/cache/mod.rs
#	src/mapper.rs
#	src/mapping.rs
#	src/utils.rs
#	tests/r8-line-number-handling.rs
#	tests/r8-method-overloading.rs
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@romtsn
Copy link
Copy Markdown
Member Author

romtsn commented Feb 16, 2026

i'm gonna drop this one since there are no tests here that are valuable or testing something that isn't already tested

@romtsn romtsn closed this Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant