aboutsummaryrefslogtreecommitdiffstats
path: root/test/unit
Commit message (Collapse)AuthorAgeFilesLines
* Merge remote-tracking branch 'origin/kumavis-readme-gource' into RevertTxManagerDan Finlay2017-01-036-131/+202
|\
| * Merge branch 'dev' into TxManagerFrankie2016-12-241-1/+1
| |\
| * | fix some minor spelling mistakes and clean up codeFrankie2016-12-241-6/+6
| | |
| * | Merge branch 'dev' into TxManagerFrankie2016-12-224-44/+123
| |\ \
| * | | First pass at revision requestsFrances Pangilinan2016-12-223-77/+48
| | | |
| * | | add Test for txManager. As well as fix tests to account for txManager.Frances Pangilinan2016-12-154-80/+180
| | | |
* | | | Correct getState test to be asyncDan Finlay2016-12-241-4/+7
| |_|/ |/| |
* | | Refine a lostAccount testDan Finlay2016-12-211-1/+1
| |/ |/|
* | Merge pull request #916 from MetaMask/i894-DetectBadAccountsDan Finlay2016-12-212-41/+5
|\ \ | | | | | | I894 detect bad accounts
| * | Made changes according to feedback.Dan Finlay2016-12-201-1/+0
| | |
| * | Mostly added bad account detectionDan Finlay2016-12-092-41/+6
| |/ | | | | | | Currently riddled with logs, because the migrator is inexplicably returning before generating the new style accounts for comparison.
* | Merged master into devDan Finlay2016-12-202-3/+118
|\ \ | |/ |/|
| * move notice code from metamask-controller + config-manager, in to ↵kumavis2016-12-172-94/+115
| | | | | | | | notice-controller
| * clean - code stylekumavis2016-12-171-4/+3
| |
| * Add ability to show notices to user & get confirmation.Kevin Serrano2016-12-171-0/+95
| | | | | | | | | | | | | | | | | | | | | | | | Implement generation of markdown for notice files. Create npm command. Enhance notice generation. Add test files to test multiple notices. Add basic markdown support to notices. Interval checks for updates. Add extensionizer and linker Add terms and conditions state file Add link support to disclaimer. Changelog addition.
* | Remove .only from unit test suiteDan Finlay2016-12-081-1/+1
| |
* | Fix nodeifyDan Finlay2016-11-291-0/+22
| |
* | Denodeify most of KeyringControllerDan Finlay2016-11-292-10/+20
| | | | | | | | | | | | | | | | | | | | Mostly Fixes #893 A couple methods cache callbacks, and will require a larger refactor to fully denodeify. Specifically, our methods involving web3 requests to sign a tx, sign a message, and approve or cancel either of those. I think we should postpone those until the TxManager refactor, since it will likely handle this response caching itself.
* | Merge branch 'master' into devDan Finlay2016-11-241-7/+2
|\|
| * Merge branch 'master' into i868-estimateGasTooHighkumavis2016-11-231-2/+8
| |\
| | * Unify test wordingDan Finlay2016-11-221-2/+2
| | |
| | * Add temporary morden explorer link supportDan Finlay2016-11-221-0/+6
| | |
| | * Fix ropsten link testDan Finlay2016-11-221-2/+2
| | |
| * | idStore - fix estimateGas calculationkumavis2016-11-231-7/+2
| |/
* | Fix last async keyring testDan Finlay2016-11-241-8/+10
| |
* | Fix more keyring asyncifying testsDan Finlay2016-11-241-1/+8
| |
* | Mostly got async keyringController tests passingDan Finlay2016-11-233-55/+91
| |
* | Fix password referenceDan Finlay2016-11-231-0/+1
| |
* | Fix vault migration unit testDan Finlay2016-11-231-1/+1
| |
* | Improve vault migration unit testDan Finlay2016-11-231-1/+2
| |
* | Simplify Encryptor API SurfaceDan Finlay2016-11-231-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | At least, the portion of it that we use. Moved salting within the encryptor, so it does not need to be managed externally. KeyringController now caches the password instead of a passwordDerivedKey, since it is ignorant of the salt. Encryptor payload is now in a JSON format, so its portions are both base64 encoded *and* labeled appropriately. The format is `{ "data": "0x0", "iv": "0x0", "salt": "string" }`. Fixes #843 Fixes #859
* | Merge pull request #858 from MetaMask/i842-WaitForSeedWordkumavis2016-11-231-2/+8
|\ \ | | | | | | I842 wait for seed word
| * | Fix etherscan link generation testDan Finlay2016-11-231-2/+8
| | |
* | | Merge branch 'dev' into dev-cleanupKevin Serrano2016-11-232-7/+7
|\| |
| * | Fix keyring testDan Finlay2016-11-221-1/+1
| | |
| * | Improve Keyring organizationDan Finlay2016-11-222-6/+6
| | | | | | | | | | | | | | | Separated public & private methods. (Fixes #845) Made class method `type()` into a simple property. (Fixes #846)
| * | Revert premature link test fixDan Finlay2016-11-221-2/+2
| | |
| * | Fixed account link test for ropsten networkDan Finlay2016-11-221-2/+2
| | |
* | | change all instances of selectedAddress to selectedAccount.Kevin Serrano2016-11-221-1/+0
| | |
* | | Remove entropy from encryption and project.Kevin Serrano2016-11-223-6/+4
|/ /
* | Minimize repeated code.Kevin Serrano2016-11-161-1/+1
| |
* | Re-enable disclaimer screen. Rename variables to reflect role more clearly.Kevin Serrano2016-11-151-14/+14
| |
* | Add idStore tests again.Kevin Serrano2016-11-151-0/+198
| |
* | Rename n variables to numberOfAccounts inside of test suite. Remove idStore ↵Kevin Serrano2016-11-123-203/+4
| | | | | | | | tests.
* | Merge branch 'master' into i328-MultiVaultFrankie2016-11-111-6/+4
|\|
| * Adjust gas buffer to be 20% over estimateDan Finlay2016-11-091-5/+4
| |
| * Increase gas bufferDan Finlay2016-11-091-2/+1
| | | | | | | | Fixes #795
* | Merge branch 'master' into i328-MultiVaultDan Finlay2016-11-081-11/+39
|\|
| * Fix gas price bufferingDan Finlay2016-11-081-8/+6
| | | | | | | | | | | | | | Our gas price buffering logic had a bug, because bn.js has inconsistent behavior when using hex-prefixed output. The issue has been opened with them here: https://github.com/indutny/bn.js/issues/151 We've corrected our usage in the mean time.
| * Added failing test for #787Dan Finlay2016-11-081-11/+41
| |
* | Fix nickname testDan Finlay2016-11-081-0/+1
| |
* | Fix BN referenceDan Finlay2016-11-081-0/+1
| |
* | Fix 787 gas buffer bugDan Finlay2016-11-081-4/+20
| |
* | Finish fixing nicknaming bugDan Finlay2016-11-041-0/+31
| |
* | Fix account nicknaming bugDan Finlay2016-11-041-0/+11
| | | | | | | | When nicknaming, we weren't normalizing the input, and so we were retrieving with differently formatted addresses than we were persisting.
* | Fix vault creation bugDan Finlay2016-11-041-8/+28
| |
* | Fix dual vault creation bugDan Finlay2016-11-031-1/+12
| |
* | Fix handling of migrating old vault styleDan Finlay2016-11-032-10/+36
| | | | | | | | | | | | Now old vaults are recognized as an "Initialized" MetaMask instance. Upon logging in, when fetching the initial password-derived key, if there is no new-style vault, but there is an old style vault, it is migrated to the new format before proceeding through the usual unlocking steps.
* | Now migrating old vaults to new DEN formatDan Finlay2016-10-291-9/+62
| |
* | Add vault migration testDan Finlay2016-10-291-0/+91
| |
* | Added hd wallet testsDan Finlay2016-10-281-0/+97
| |
* | Merge branch 'master' into i328-MultiVaultDan Finlay2016-10-261-4/+17
|\|
| * test - fix delegate-call testkumavis2016-10-251-10/+6
| |
| * Merge branch 'master' into i743-FixDelegateCallFlagkumavis2016-10-251-0/+22
| |\
| * | Reproduced issue 743 in test caseDan Finlay2016-10-181-3/+20
| | | | | | | | | | | | | | | | | | This contract hex does include the value `f4`, but it was compiled from a contract with no instance of `.delegatecall`. I believe `f4` in this case is part of some other value or contract address, and `ethBinToOps` has some error in how it skips pushed data. @kumavis
* | | Merge branch 'master' into i328-MultiVaultDan Finlay2016-10-221-0/+22
|\ \ \ | | |/ | |/|
| * | Clean up testsFrankie2016-10-211-4/+4
| | |
| * | Add test and ability for isHex to handle hex strings with hex-prefixFrankie2016-10-201-0/+5
| | |
| * | Add unit test for isHex and add to CHANGELOG.mdFrankie2016-10-201-0/+17
| |/
* | Fix bug in new KeyringController vault restoring logic.Dan Finlay2016-10-221-0/+31
| |
* | Create new vault before each keyringController unit testDan Finlay2016-10-221-3/+5
| |
* | Create basic keyring-controller unit test fileDan Finlay2016-10-221-20/+35
| |
* | Added SimpleKeyring testsDan Finlay2016-10-221-0/+83
| |
* | Fix testsDan Finlay2016-10-213-170/+4
| |
* | Git merge resolution.Kevin Serrano2016-10-201-0/+18
|\|
| * Restore hex prefix to gas priceDan Finlay2016-10-181-0/+1
| |
| * Increase gas estimate by 100000 weiDan Finlay2016-10-181-0/+17
| | | | | | | | | | | | To prevent minor gas estimation errors, probably usually related to operating on dynamic state. Fixes #738.
* | Intermediary commit.Kevin Serrano2016-10-202-5/+5
| |
* | Add initial KeyringController filesDan Finlay2016-10-121-0/+141
|/
* Remove logsDan Finlay2016-09-151-2/+0
|
* Merge pull request #669 from MetaMask/Version-2.12.1Dan Finlay2016-09-151-8/+10
| | | | Version 2.12.1
* Fix clearing of account cache on vault restoreDan Finlay2016-09-131-8/+6
|
* Fix some referencesDan Finlay2016-09-131-2/+2
|
* Fix account unlockingDan Finlay2016-09-131-19/+12
| | | | | | | | SubmitPassword was not creating a new id-management This is because I broke up the old "createIdmgmt" method to not perform as much conditional logic. Now the pieces are reusable and do what they should do.
* Select first address when restoring vaultDan Finlay2016-09-121-0/+18
| | | | Fixes #642
* Fixed caching bugDan Finlay2016-09-111-6/+5
| | | | | | Fixed bug where the second new vault created in an IdStore would initially return the accounts from the original store. Also fixed some tests that were incorrect.
* Correct assertion for BIP32 complianceDan Finlay2016-09-111-1/+1
| | | | According to [axic's work here](https://github.com/MetaMask/metamask-plugin/issues/640#issuecomment-246133672), MetaMask is generating the correct address, so I've corrected that assertion accordingly.
* Remove logDan Finlay2016-09-111-1/+0
|
* Add more bip44 assertionsDan Finlay2016-09-111-1/+9
|
* Made bip44 assertions easier to add toDan Finlay2016-09-111-27/+36
|
* Clean up testsDan Finlay2016-09-111-18/+17
|
* Correctly clear ethStore cache on new vault restoreDan Finlay2016-09-111-6/+22
|
* Captured #640 in failing testDan Finlay2016-09-111-1/+12
|
* Add new eth-lightwallet salting to vault.Dan Finlay2016-09-101-1/+3
| | | | | | | | | | eth-lightwallet was previously not salting vault passwords, potentially making it easier to crack them once obtained. This branch incorporates the API changes to allow us to take advantage of the new salting logic. This is still throwing deprecation warnings, but that's actually a bug in eth-lightwallet I wrote, [I've submitted a PR for that here](https://github.com/ConsenSys/eth-lightwallet/pull/116). Fixes #555
* Add negative value validation to tx approvalDan Finlay2016-09-071-0/+97
|
* Add test for Edge compatibilityDan Finlay2016-08-301-0/+29
|
* Naming issue.Kevin Serrano2016-08-251-1/+1
|
* Implement usage of nock.Kevin Serrano2016-08-251-2/+11
|
* Now calls API functions with mock options.Kevin Serrano2016-08-251-3/+5
|
* Merge branch 'master' into i390-TransactionLimitDan Finlay2016-08-231-1/+6
|\
| * Add testFrankie2016-08-201-1/+6
| |
* | Added `view more` button to transaction listDan Finlay2016-08-191-0/+12
| | | | | | | | Visible at the end of the transaction list, or if no transactions are listed, displayed right after the `No Transactions` message.
* | Enforce tx history limitDan Finlay2016-08-191-0/+11
|/
* Added compliance for tests and properly accounts for N/A conversions.Kevin Serrano2016-08-171-7/+6
|
* Remove extra test.Kevin Serrano2016-08-171-4/+0
|
* Merge branch 'master' into usd-conversionKevin Serrano2016-08-051-3/+12
|\
| * Fix extension testsDan Finlay2016-07-261-3/+12
| |
* | Change function names. Add interval polling for api. Refactor functions.Kevin Serrano2016-07-221-3/+3
| |
* | Merge branch 'master' into usd-conversionKevin Serrano2016-07-221-0/+39
|\|
| * Implement some cross-browser practices (#473)Dan Finlay2016-07-221-0/+39
| | | | | | | | | | | | | | | | | | | | | | | | * Add mozilla plugin key to manifest * Move all chrome references into platform-checking module Addresses #453 * Add chrome global back to linter blacklist * Add tests
* | Progress on config manager. Plus tests.Kevin Serrano2016-07-211-1/+81
|/
* Remove unused functions. Readd if necessary.Kevin Serrano2016-07-121-19/+0
|
* Refactor util tests.Kevin Serrano2016-07-121-11/+8
|
* Add nickname rendering for recipient addressDan Finlay2016-07-081-0/+44
|
* Merge branch 'master' into ConfirmationStyleDan Finlay2016-07-081-4/+17
|\
| * Revert back to original formatBalanceFrankie2016-07-081-5/+17
| |
* | Merged latest balance formatting codeDan Finlay2016-07-071-17/+4
|\|
| * Add decimal hendeling to ETH balanceFrankie2016-07-071-17/+5
| |
* | Began reworking tx conf viewDan Finlay2016-07-071-0/+6
| |
* | Simplify empty account balance renderingDan Finlay2016-07-061-1/+1
|/
* Remove useless and buggy clearWallet functionDan Finlay2016-07-011-11/+0
|
* Replicated really strange bug with testDan Finlay2016-07-011-1/+16
|
* Fixed testsDan Finlay2016-06-252-2/+9
|
* stop pedantic lintingkumavis2016-06-241-1/+1
|
* icon-factory - overhaul to provide svg or imagified elkumavis2016-06-241-31/+0
|
* Fix eth sign with dennis' test inputDan Finlay2016-06-221-11/+13
|
* Add linting to our mocha suiteDan Finlay2016-06-221-0/+9
|
* added a failing case and fixed it by refactoring everything to stringsZac Mitton2016-06-211-1/+15
|
* Merge branch 'master' into SignFullDataNotHashDan Finlay2016-06-171-1/+31
|\
| * Add confirmation persisting to localStorageDan Finlay2016-06-171-1/+31
| |
* | Fixed signing of hashesDan Finlay2016-06-171-4/+6
| | | | | | | | | | | | Signing now always takes a 64 digit hex string, and returns a message signature which appropriately pads r, s, and v with zeroes. Need to verify with Denis that this is the behavior he requires.
* | Began adding signMsg unit testDan Finlay2016-06-161-0/+31
|/
* Cache identiconsDan Finlay2016-06-071-0/+31
| | | | | | | | Fixes #197 Also as a side effect, by creating this `iconFactory.cache` object, we have a convenient place for specifying stock icons for known contracts! We can just hard-code image addresses in the `ui/lib/icon-factory.js` cache instantiation, and those values will be injected into the identicon image tag `src` attributes.
* Fix hashed address validationDan Finlay2016-06-031-3/+3
|
* Add test case for sample hashed addressDan Finlay2016-06-021-0/+7
|
* Merge pull request #222 from MetaMask/balanceskumavis2016-05-261-5/+30
|\ | | | | Fixes #206
| * correctly displaying small balances with testsZac Mitton2016-05-261-5/+30
| |
* | Fix persistent warning bugDan Finlay2016-05-261-0/+51
|/
* Added ability to nickname wallets locallyDan Finlay2016-05-212-0/+57
| | | | The changes are persisted to localstorage, so they cannot be restored on a new computer, but for right now it's a nice organizational feature.
* Add full precision to send tx value field.Dan Finlay2016-05-201-0/+14
|
* Increase send value precisionDan Finlay2016-05-201-0/+8
|
* Add address capitalization checksummingDan Finlay2016-05-201-0/+46
|
* Streamlined some transition logicDan Finlay2016-05-132-1/+28
| | | | | | | | Fixes #122 Had used multiple actions for some transitions, which would lead to brief intermediary states. Now making a few actions much more explicit about what they route to, so there is less intermediary logic, and we can transition confidently to the correct view.
* Move all users to new scalable backendDan Finlay2016-05-121-0/+5
|
* Fix testDan Finlay2016-05-111-1/+2
|
* Remove test only statementDan Finlay2016-05-111-1/+1
|
* Fix outdated transitionsDan Finlay2016-04-301-23/+23
| | | | | | | | Fixes #151 - Cancelling or completing a tx now goes back to account detail view. - Restoring a vault now does not select an unloaded account, shows account list. - Account list now never selects an item only uses the cells as buttons.
* Fix testDan Finlay2016-04-281-4/+8
|
* Fix explorer link generationDan Finlay2016-04-281-0/+11
|
* Make account detail view the primary viewDan Finlay2016-04-261-1/+1
| | | | | | | | - When unlocking, the first account is now selected by default and displayed as the main view. - There is now a "CHANGE ACCT" button on the detail view to show the accounts list. - Clicking an account from the accounts list now navigates to the detail view and selects that account. - Config/Info screen "back" buttons now fire a new action, `GO_HOME`, which is configured to navigate to the accountDetail view, putting that logic in one place. - When locking and unlocking again, the first account is always displayed, eventually we should persist the selection.
* Add migration to actual migration array, fix testDan Finlay2016-04-231-3/+8
|
* Deprecate rawtestrpc.metamask.ioDan Finlay2016-04-231-3/+11
| | | | This migration will move users who have their clients configured to point at `rawtestrpc.metamask.io` to point at our new test-net RPC, `testrpc.metamask.io`.
* Store metamaskId on metaTx instead of getTxWithParams method.Dan Finlay2016-04-211-26/+0
|
* Fix floating point input bugDan Finlay2016-04-201-27/+41
| | | | | | When sending a transaction, we were converting to BN before handling decimals, which meant we were losing any precision past a decimal point, since BN does not handle decimals! Put this numeric normalization into a utility function with a test around it and got it working.
* Add extra tx methods to configManagerDan Finlay2016-04-201-0/+36
|
* Persist transactions to config-managerDan Finlay2016-04-191-0/+79
| | | | | | | | Transactions are now stored, and are never deleted, they only have their status updated. We can add deleting later if we'd like. I've hacked on emitting the new unconfirmedTx key to the UI in the format it received before, I want Aaron's opinion on where I should actually do that.
* Unify test suitesDan Finlay2016-04-198-1/+454
|
* Fix plugin testsDan Finlay2016-04-191-5/+8
|
* Migrate etherscan providers to our RPCDan Finlay2016-04-131-0/+14
|
* Made configuration migrateableDan Finlay2016-03-311-0/+71
| | | | | | | | | | | | | | Abstract all configuration data into a singleton called `configManager`, who is responsible for reading and writing to the persisted storage (localStorage, in our case). Uses my new module [pojo-migrator](https://www.npmjs.com/package/pojo-migrator), and wraps it with the `ConfigManager` class, which we can hang any state setting or getting methods we need. By keeping all the persisted state in one place, we can stabilize its outward-facing API, making the interactions increasingly atomic, which will allow us to add features that require restructuring the persisted data in the long term without having to rewrite UI or even `background.js` code. All the restructuring and data-type management is kept in one neat little place. This should make it very easy to add new configuration options like user-configured providers, per-domain vaults, and more! I know this doesn't seem like a big user-facing feature, but we have a big laundry list of features that I think this will really help streamline.
* Correct the bip44 test accountDan Finlay2016-03-261-5/+5
|
* Implemented BIP44 compliance test.Dan Finlay2016-03-261-0/+81
Also added the hdPath that Christian had told me to our calls to the LightWallet, but this does not seem to have made us generate the same accounts as `testrpc` yet.