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

Add methods for exporting and importing backups #26

Merged
merged 5 commits into from
Jul 28, 2023

Conversation

modos189
Copy link
Contributor

In the 'Manager' class, the 'getBackupData' and 'setBackupData' methods have been added to handle the export and import of backups, respectively. These methods allow the user to retrieve backup data based on specified parameters and set backup data using the provided backup object.

The storage state is now reset before each test run to ensure test isolation and avoid interference between tests. This guarantees that each test starts with a clean state and avoids potential issues with shared data.
…and add code comments"

The 'get' method in the 'storage.js' mock now accepts 'null' as an argument, allowing the retrieval of all data stored in the object. Additionally, comprehensive code comments have been added to explain each function's purpose and behavior in the 'storage.js' file. This improves code readability and maintainability.
Instead of returning a simple copy of the entire storage, it now correctly iterates over all keys in the store and applies the '_get_one' method to each key, returning an object with all the data.
In the 'Manager' class, the 'getBackupData' and 'setBackupData' methods have been added to handle the export and import of backups, respectively. These methods allow the user to retrieve backup data based on specified parameters and set backup data using the provided backup object.
@modos189 modos189 merged commit a17e588 into IITC-CE:master Jul 28, 2023
5 checks passed
@modos189 modos189 deleted the backup branch July 28, 2023 21:13
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.

1 participant