Compare commits
1
Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
3b86e33436 |
@@ -766,11 +766,13 @@ Three limits are worth knowing, all real differences from the Chrome suite:
|
||||
in `tests/e2e/dapp.js`, which serves the dApp page and a JSON-RPC node and
|
||||
which the extension's `rpcUrl` is pointed at for the dApp steps; a JSON-RPC
|
||||
method that fixture does not model fails the run rather than answering `null`.
|
||||
The suite leaves "Use Blockscout" off, so Blockscout is not asked. A
|
||||
`ReferenceError` in the success path of `renderTransactions`, or of price
|
||||
rendering, passes this suite green. The offline run is also weaker than the
|
||||
Chrome suite's interception for those calls: it proves nothing got out, but it
|
||||
cannot report which requests were attempted.
|
||||
Everything else, Blockscout and the price feed among it, has no fixture and
|
||||
simply fails, and the extension swallows its own fetch failures, so only the
|
||||
_failure_ branches of that code are ever executed. A `ReferenceError` in the
|
||||
success path of `renderTransactions`, or of price rendering, passes this suite
|
||||
green. The offline run is also weaker than the Chrome suite's interception for
|
||||
those calls: it proves nothing got out, but it cannot report which requests
|
||||
were attempted.
|
||||
- **The site-connection prompt always opens in a window of its own.** The
|
||||
profile turns off `extensions.openPopupWithoutUserGesture.enabled`, so
|
||||
`src/background/index.js` falls back from the toolbar popup, which WebDriver
|
||||
@@ -2881,8 +2883,9 @@ Sent the user's data only while its setting is on:
|
||||
- on every balance refresh, in the popup and in the background
|
||||
- when Home, AddressDetail or AddressToken loads its history
|
||||
- when TransactionDetail opens
|
||||
- **While the setting is off**: Each screen that would show Blockscout's
|
||||
data says instead that the setting is off. Only tracked tokens are listed.
|
||||
- **While the setting is off**: each of those screens shows a line saying
|
||||
what the screen leaves out and that "Use Blockscout" is off. Only tracked
|
||||
tokens are listed.
|
||||
- **Saving the URL**: pressing "Save" beside the URL in Settings asks that
|
||||
URL for `/stats`, whether or not the setting is on. This request carries
|
||||
no user data.
|
||||
|
||||
Reference in new issue
Block a user