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.

Sign up for GitHub

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

Jump to bottom

Integrate DatabasePresenter into DatabaseCLI, Command + DatabaseCLITests #7

Merged
merged 23 commits into from
May 11, 2022
Merged

Integrate DatabasePresenter into DatabaseCLI, Command + DatabaseCLITests #7

merged 23 commits into from
May 11, 2022

Conversation

withaspirit
Copy link
Owner

@withaspirit withaspirit commented May 11, 2022 •
edited
Loading

Changes

  • DatabaseCLI:
    • added DatabasePresenter, added to methods
    • expanded CRUD read() method
  • CommandTest
    • added inputs.json for testing valid/invalid commands with Command.regex
  • Database
    • added getSizeOfTable() method
  • DatabaseCLITest:
    • added tests for CRUD methods
  • DatabasePresenterTest:
    • added more CRUD tests
  • Item:
    • added ID to getValuesAsArray(), getAttributeNamesAsArray()
  • pom.xml
    • added junit-jupiter-params dependency

@withaspirit
DatabaseCLI: Add DatabasePresenter
8720c8f
@withaspirit
Merge branch 'master' of https://github.com/cyberphoria/Sample-CRUD-B…
ce90bcb 
…ackend into Integrate_Presenter_View

� Conflicts:
�	src/main/java/view/DatabaseCLI.java
@withaspirit
DatabaseCLI: import DatabasePresenter
4b66dd5
@withaspirit
DatabaseCLITest: add Database, DatabasePresenter
9bb0007
@withaspirit
CommandTest: add ParameterizedTest
ee14a70
@withaspirit
Added inputs.json for input tests
ec9ed20
@withaspirit
CommandTest: added methods to test Command.regex
3c2ca43
@withaspirit
Database: add getSizeOfTable() method
9ccb7a0
@withaspirit
DatabaseCLITest: added test for deleting item
b38c2b4
@withaspirit
Tests: Optimized imports
7c923de
@withaspirit
DatabaseCLITest: add test method for updating database
b73edb2
@withaspirit
DatabasePresenterTest: fixed testUpdateOneItemOneAttribute method
97474bd
@withaspirit
Removed unnecessary output statements
00bbf51
@withaspirit
Use database.getSizeOfTable() in tests
5de0f28
@withaspirit
Reformat CommandTest
de845de
@withaspirit
ItemTest comment
d3ab7bf
@withaspirit
DatabasePresenterTest: add tests for read methods
17eaf11
@withaspirit
DatabaseCLI: added output for read method
e2cacdc
@withaspirit
Item: add ID to getValuesAsArray, getAttributeNamesAsArray
20a793d
@withaspirit
DatabaseCLITest: add testRead methods
dfffbbf
@withaspirit
DatabaseCLI: add javadocs to read, delete methods
40f4aac
@withaspirit withaspirit changed the title Integrate DatabasePresenter into DatabaseCLI Integrate DatabasePresenter into DatabaseCLI, Command + DatabaseCLITests May 11, 2022
@withaspirit withaspirit merged commit bab6ab0 into master May 11, 2022
@withaspirit withaspirit deleted the Integrate_Presenter_View branch May 11, 2022 14:28
@withaspirit withaspirit mentioned this pull request May 18, 2022
Closed
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@withaspirit