mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

Code coverage: Collect test coverage for the protocol layer

Browse source 
Because we were only doing a partial instrumentation run prior to this
change, un-instrumented code had references to un-instrumented code
(because Blanket's instrumentation re-runs the code once it is
instrumented, updating the window.NAME reference).

A big part of this was that libtextsecure was not instrumented. And for
good reason; it takes a long time to load and run, especially when
instrumented. But without covering it, we'll have an incomplete view
of our code coverage. So, all coverage collection remains off by
default.

FREEBIE
This commit is contained in:
Scott Nonnenberg 2017-07-03 15:08:10 -07:00
parent 10f4f3fea5
commit 5c990d2a35
1 changed files with 18 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

36
test/index.html
View file

@ -546,30 +546,33 @@
</div>
</script>
<script type="text/javascript" src="../js/components.js"></script>
<script type="text/javascript" src="test.js"></script>
<script type='text/javascript' src='../js/registration.js'></script>
<script type="text/javascript" src="../js/expire.js"></script>
<script type="text/javascript" src="../js/chromium.js"></script>
<script type="text/javascript" src="../js/database.js"></script>
<script type="text/javascript" src="../js/storage.js"></script>
<script type="text/javascript" src="../js/signal_protocol_store.js"></script>
<script type="text/javascript" src="../js/libtextsecure.js"></script>
<!-- Comment out to turn off code coverage. Useful for getting real callstacks.
Don't forget about turning on the mantual test start below! -->
<!-- <script type="text/javascript" src="blanket_mocha.js"></script> -->
<script type='text/javascript' src='../js/registration.js' data-cover></script>
<script type="text/javascript" src="../js/expire.js" data-cover></script>
<script type="text/javascript" src="../js/chromium.js" data-cover></script>
<script type="text/javascript" src="../js/database.js" data-cover></script>
<script type="text/javascript" src="../js/storage.js" data-cover></script>
<script type="text/javascript" src="../js/signal_protocol_store.js" data-cover></script>
<script type="text/javascript" src="../js/libtextsecure.js" data-cover></script>
<script type="text/javascript" src="../js/libphonenumber-util.js"></script>
<script type="text/javascript" src="../js/models/messages.js" data-cover></script>
<script type="text/javascript" src="../js/models/conversations.js" data-cover></script>
<script type="text/javascript" src="../js/models/blockedNumbers.js" data-cover></script>
<script type="text/javascript" src="../js/conversation_controller.js" data-cover></script>
<script type="text/javascript" src="../js/panel_controller.js"></script>
<script type='text/javascript' src='../js/emoji_util.js'></script>
<script type="text/javascript" src="../js/keychange_listener.js"></script>
<script type='text/javascript' src='../js/expiring_messages.js'></script>
<script type='text/javascript' src='../js/notifications.js'></script>
<script type="text/javascript" src="../js/panel_controller.js" data-cover></script>
<script type='text/javascript' src='../js/emoji_util.js' data-cover></script>
<script type="text/javascript" src="../js/keychange_listener.js" data-cover></script>
<script type='text/javascript' src='../js/expiring_messages.js' data-cover></script>
<script type='text/javascript' src='../js/notifications.js' data-cover></script>
<script type="text/javascript" src="../js/chromium.js"></script>
<script type="text/javascript" src="../js/chromium.js" data-cover></script>
<script type='text/javascript' src='../js/views/whisper_view.js' data-cover></script>
<script type='text/javascript' src='../js/views/debug_log_view.js' data-cover></script>
@ -626,10 +629,7 @@
<script type="text/javascript" src="fixtures.js"></script>
<script type="text/javascript" src="fixtures_test.js"></script>
<!-- Comment out to turn off code coverage. Useful for getting real callstacks. -->
<!-- <script type="text/javascript" src="blanket_mocha.js"></script> -->
<!-- Uncomment to start tests without code coverage enabled -->
<!-- Uncomment to start tests without code coverage enabled (don't forget about blanket stuff above) -->
<script type="text/javascript">
mocha.run();
</script>