Closed rynorris closed 5 years ago
Merging #53 into master will decrease coverage by
1.15%
. The diff coverage is0%
.
@@ Coverage Diff @@
## master #53 +/- ##
==========================================
- Coverage 92.8% 91.64% -1.16%
==========================================
Files 53 54 +1
Lines 3612 3663 +51
==========================================
+ Hits 3352 3357 +5
- Misses 260 306 +46
Impacted Files | Coverage Δ | |
---|---|---|
src/emulator/mappers/cnrom.rs | 70.58% <ø> (ø) |
:arrow_up: |
src/emulator/state.rs | 0% <0%> (ø) |
:arrow_up: |
src/emulator/ines.rs | 89.18% <0%> (-2.24%) |
:arrow_down: |
src/emulator/mappers/color_dreams.rs | 0% <0%> (ø) |
|
src/emulator/mod.rs | 92.22% <0%> (+0.08%) |
:arrow_up: |
src/emulator/ppu/registers.rs | 86.36% <0%> (+0.64%) |
:arrow_up: |
src/emulator/mappers/nrom.rs | 81.48% <0%> (+3.7%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 8a60eb7...c96b982. Read the comment docs.
e.g.![image](https://user-images.githubusercontent.com/3620166/49272539-d8951000-f4ac-11e8-8fd8-c7b159698135.png)
I tested Crystal mines, Menace beach and Metal fighter.
Metal fighter is unplayable, but I don't think it's a mapper issue. The other two work fine.