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

ensure db is not read-only #218

Merged
merged 1 commit into from
Jun 5, 2023
Merged

ensure db is not read-only #218

merged 1 commit into from
Jun 5, 2023

Conversation

jameskeith
Copy link
Contributor

No description provided.

@jameskeith jameskeith requested a review from tamayok June 2, 2023 08:24
@jameskeith jameskeith merged commit e343206 into release/1.5.3 Jun 5, 2023
@jameskeith jameskeith deleted the mt-1167-update branch June 5, 2023 14:43
tamayok added a commit that referenced this pull request Jun 5, 2023
* [MT-1279] (#214)

* Concurrent module manager

* cleanup

* MT-1167 (#207)

* [MT-1166] Connectivity Collector properties updates (#205)

* Connectivity Collector properties updates

* PR fixes

---------

Co-authored-by: Karen Tamayo <[email protected]>

* MT-1167

* PR fixes

* Update Release version + cleanup

* DispatchStorage delete from dao when dequeuing

* PR fixes - cleanup brace formatting + duplicated db logic

* Update db getter in dao

---------

Co-authored-by: Karen Tamayo <[email protected]>
Co-authored-by: Karen Tamayo <[email protected]>

* Fetch publish settings using etag (#212)

* Fetch publish settings using etag

* PR review fixes

* Add missing etag in loadSettings

* Cleanup

* Remove headers map from ResourceEntity

---------

Co-authored-by: Karen Tamayo <[email protected]>

* Cleanup

* Switch to synchronized map to maintain insertion order

* DB test updates

* Remove DbHelper mock

* DB Test updates

* Cleanup

* Update kvDao in DispatchStorageDao

* kvDao mock

* Update mocked obj

* Add every statement to DB

* Mock dbHelper queue

* Add OpenForTesting to DatabaseHelper

* Cleanup

* ensure db is not read-only (#218)

---------

Co-authored-by: James Keith <[email protected]>
Co-authored-by: Karen Tamayo <[email protected]>
Co-authored-by: Karen Tamayo <[email protected]>
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