Skip to content
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

fix: test_output_manager_sqlite_db(_encrypted) #4025

Merged
merged 3 commits into from
Apr 13, 2022

Conversation

Cifko
Copy link
Contributor

@Cifko Cifko commented Apr 12, 2022

Description

Fix ignored tests. I had to add one function to the trait to make it work.

How Has This Been Tested?

cargo test --package tari_wallet --test output_manager_service --all-features -- output_manager_service_tests::storage::test_output_manager_sqlite_db --exact
cargo test --package tari_wallet --test output_manager_service --all-features -- output_manager_service_tests::storage::test_output_manager_sqlite_db_encrypted --exact

@aviator-app aviator-app bot removed the mq-failed label Apr 13, 2022
@stringhandler stringhandler merged commit 7a6f980 into tari-project:development Apr 13, 2022
stringhandler added a commit that referenced this pull request May 10, 2022
BREAKING CHANGES
---

* major config rework. Config files should be recreated  (#4006)

Features
---

* allow network to be set by TARI_NETWORK env var ([#4073](#4073)) ([c27be5c](c27be5c))
* **collectibles:** add list assets command ([#3908](#3908)) ([5b726a6](5b726a6))
* **merge mining proxy:** check achieved Monero difficulty before submitting to Tari basenode ([#4019](#4019)) ([b09fa76](b09fa76))
* **p2p:** adds tor.forward_address setting ([#4070](#4070)) ([8c78717](8c78717))
* **tari_explorer:** add total hashrate chart ([#4054](#4054)) ([9e0ec36](9e0ec36))

Bug Fixes
---

* add Environment cfg source and remove --create-id ([#4018](#4018)) ([e59e657](e59e657))
* **base-node:** assign correct base dir to tor identity ([#4081](#4081)) ([1464f8b](1464f8b))
* **dht:** saf storage uses constructs correct msg hash ([#4003](#4003)) ([e1e7669](e1e7669))
* **dht:** sets file default for DHT db ([#4030](#4030)) ([5b125e7](5b125e7))
* **dns-seeds:** use correct cloudflare resolver default ([#4029](#4029)) ([c95e589](c95e589))
* fix github integration tests ([#4008](#4008)) ([aa143c6](aa143c6))
* github actions ([#4014](#4014)) ([a03392e](a03392e))
* ignore test that timeout in github action ([#4010](#4010)) ([6c5471e](6c5471e))
* **key-manager:** remove floating point math from mnemonic code ([#4064](#4064)) ([c2d60b3](c2d60b3))
* launchpad build docker images ([#4042](#4042)) ([50e2812](50e2812))
* **launchpad:** fix config presets ([#4028](#4028)) ([1b8b274](1b8b274))
* long running and non critical github action ([#4009](#4009)) ([3b8cb8b](3b8cb8b))
* makes header consensus encoding infallible ([#4045](#4045)) ([5ebf129](5ebf129))
* only count base nodes in peers count in base node status ([#4039](#4039)) ([190d75a](190d75a))
* prevent seed peer address from being overwritten unless newer ([#4085](#4085)) ([59b76c3](59b76c3))
* support safe non-interactive mode ([#4072](#4072)) ([b34f79d](b34f79d))
* test_output_manager_sqlite_db(_encrypted) ([#4025](#4025)) ([7a6f980](7a6f980))
* update daily test configuration ([#4049](#4049)) ([b15d682](b15d682))
* **wallet:** do not prompt for password if given in config ([#4040](#4040)) ([fc1aa65](fc1aa65))
* **wallet:** ensure block hash exists ([#4083](#4083)) ([a258984](a258984))
* weird behaviour of dates in base node banned peers ([#4037](#4037)) ([7097185](7097185))
@Cifko Cifko deleted the fix-database-test branch June 23, 2022 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants