-
Notifications
You must be signed in to change notification settings - Fork 1
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
11 Networking.Persistence Module with DAO abstract class, implementations and builder #15
Conversation
Found a tutorial on how to add firebase. So I can probably set it up later today |
core/src/com/softwarearchitecture/networking/persistence/DAO.java
Outdated
Show resolved
Hide resolved
core/src/com/softwarearchitecture/networking/persistence/LocalDAO.java
Outdated
Show resolved
Hide resolved
…dency" This reverts commit 983bd42.
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.
Rename this to LocalDAO or StorageDAO? More descriptive than MockDAO, especially if it's going to be used for tests.
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.
Or i don't see the reason to test something that won't be used, if that makes sense
@Before | ||
public void setUp() { | ||
// dao with all capabilities | ||
mockDAO = new MockDAO<>(true, true, true, true); |
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.
Is this standard, why can't we enforce/expect all the CRUD-operations to be implemented?
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.
Looks good :D
The network.persistence module
This PR adds the following to the project:
There are also tests for this functionality
To run all the tests: