-
Notifications
You must be signed in to change notification settings - Fork 264
test: First draft for settings wrappers and test file added #6984
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
AYAHASSAN287
wants to merge
47
commits into
develop
Choose a base branch
from
test/settings-tests
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+699
−9
Draft
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
611323b
test: First draft for settings wrappers and test file added
AYAHASSAN287 eed34eb
test: Making changes in test env & add tests
AYAHASSAN287 9583dcd
test: Add new test
AYAHASSAN287 5ef6ef6
feat: improve blockchainstate
dlipicar a032e5e
feat: replace old transfer detection algorithm with multistandardbala…
dlipicar d6f006a
build: add Android x86_64 support to Makefile (#6990)
glitchminer 52f1fbf
ci: use official base docker image for builds
markoburcul e7e188f
test: community chats (#6987)
fbarbu15 5fc58ab
test: Adding node tests
AYAHASSAN287 a1943ab
test: contact verification (#6997)
fbarbu15 2e34d37
chore: point go.mod to original go-ethereum repo
dlipicar d58773d
chore: drop usage of non-multichain ethclient methods in rpc chain cl…
dlipicar 9837fca
chore: drop usage of non-multichain ethclient methods in community to…
dlipicar d626137
chore: drop usage of non-multichain ethclient methods in fees manager
dlipicar 7879025
chore: adapt to new geth logger
osmaczko d66bf11
chore: remove unused log_parser code
dlipicar 5eb8f52
chore: replace custom NoSign parameter from bind.TransactOpts
dlipicar 20a5993
chore: update SingleRequestCodec to go-ethereum upstream
igor-sirotin 75439f3
chore: use `go-waku` with upstream `go-ethereum` (#6978)
igor-sirotin 1d7080c
chore: fix tests
dlipicar bc85526
chore: bump go-waku and go-discover (#6983)
igor-sirotin bf4c893
refactor: use common Migrate function
osmaczko a25c5e5
refactor: move pubkeys utils from protocol to crypto package
osmaczko 37b67fd
fix: Crash on uninitialized filterManager
alexjba f7a427c
test: Add new tests
AYAHASSAN287 6f39c0a
test: Adding wrappers &tests
AYAHASSAN287 5560bca
test: Add positive & negative tests
AYAHASSAN287 c3bc720
test: Added backup tests
AYAHASSAN287 5e77631
feat: clean up balance fetching from token manager (#6991)
dlipicar 3b0af69
fix(scripts): branch_version_generated use current base branch (#7007)
igor-sirotin 4fe3de8
fix: goimports configuration (#7004)
igor-sirotin cbe7387
feat: local timesource (#7003)
igor-sirotin 4ad6877
feat(benchmark): threads and goroutines count (#7006)
igor-sirotin 4d480c9
chore: cleanup chain client and transactor (#7008)
dlipicar fcd6bc2
test: add all APIs tests
AYAHASSAN287 8a708ea
test: Apply linters
AYAHASSAN287 c94a52e
Merge branch 'develop' into test/settings-tests
AYAHASSAN287 a6fe6f0
fix: reduce test variants to shorten test durations
romanzac 90f3347
Merge branch 'develop' into test/settings-tests
romanzac c72d8d4
fix: lints
romanzac bf9fa39
fix: remove print statements for debug-only purpose
romanzac b707a8f
fix: use logger when necessary
romanzac f112f98
chore: use `go-waku` with upstream `go-ethereum` (#6978)
igor-sirotin bd167b4
chore: bump go-waku and go-discover (#6983)
igor-sirotin d9ce56e
feat: clean up balance fetching from token manager (#6991)
dlipicar dbae7fc
fix: rebase on develop
romanzac 92cd47e
fix: wait longer for backend in Jenkins
romanzac File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -32,6 +32,7 @@ | |
| from utils import fake | ||
| from utils import keys | ||
| from utils.config import Config | ||
| import copy | ||
|
|
||
| NANOSECONDS_PER_SECOND = 1_000_000_000 | ||
|
|
||
|
|
@@ -80,7 +81,7 @@ def __init__(self, privileged=False, ipv6=USE_IPV6, **kwargs): | |
| self.events = Events() | ||
| self.version = "unknown" | ||
| self.network_id = 1 | ||
|
|
||
| self._boot_api_config = None | ||
| RpcClient.__init__(self) | ||
| ApiClient.__init__(self, self.api_url) | ||
| SignalClient.__init__(self, self.ws_url) | ||
|
|
@@ -114,7 +115,7 @@ def shutdown(self, log_sufix=""): | |
| self.temp_dir.cleanup() | ||
|
|
||
| @retry( | ||
| stop=stop_after_delay(10), | ||
| stop=stop_after_delay(30), | ||
| wait=wait_exponential(multiplier=1, min=0.1, max=5), | ||
| retry=retry_if_exception_type((ConnectionError, requests.RequestException)), | ||
| reraise=True, | ||
|
|
@@ -302,6 +303,7 @@ def restore_account_and_login(self, user=user_1, **kwargs): | |
| method = "RestoreAccountAndLogin" | ||
| data = self._create_account_request(password=user.password, **kwargs) | ||
| data["mnemonic"] = user.passphrase | ||
| self._boot_api_config = copy.deepcopy(data.get("apiConfig", {})) | ||
| return self.api_request_json(method, data) | ||
|
|
||
| def login(self, key_uid, password: str, kdf_iterations=256000): | ||
|
|
@@ -472,6 +474,9 @@ def image_server_tls_cert(self): | |
| response = self.api_request(method, {}) | ||
| return response.content.decode("utf-8") | ||
|
|
||
| def get_boot_api_config(self): | ||
| return copy.deepcopy(self._boot_api_config) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you already did a deepcopy at line 295 / 303 |
||
|
|
||
| def serialize_legacy_key(self, key): | ||
| method = "SerializeLegacyKey" | ||
| # Use client.post directly, because this method is old and has json-incompatible arguments | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -15,6 +15,7 @@ def test_full_migrate_flow(self): | |
| add_resp = self.account.accounts_service.add_keypair_via_seed_phrase( | ||
| user_1.passphrase, self.account.password, keypair_name, wallet_account_details_derivation | ||
| ) | ||
| self.account.accounts_service.migrate_non_profile_keycard_keypair_to_app | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. what's with this change? |
||
| assert "error" not in add_resp | ||
| add_result = add_resp | ||
| assert add_result is not None | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's with the changes in the .go files?