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

Feat/add delete method #982

Open
wants to merge 92 commits into
base: master
Choose a base branch
from
Open

Feat/add delete method #982

wants to merge 92 commits into from

Conversation

Ethan-t-n
Copy link

forgot to add delete method and test

bensjsilverstein and others added 30 commits June 1, 2022 11:52
@bensjsilverstein
Create class files and test files
cda41e9
@Ethan-t-n
Merge pull request #1 from Ethan-t-n/main
bf2cfdd 
Create class files and test files
@sdecesari
Inital commit
33c3324
@sdecesari
Merge branch 'master' of github.com:Ethan-t-n/black_thursday
029b0c8
@Ethan-t-n
initial commit
e7a402d
@Ethan-t-n
test 2
74c7434
@bensjsilverstein
Feat - Add 'exists' test
2e49e40
@bensjsilverstein
Feat - Create Merchant class
60132a6
@bensjsilverstein
Feat - Add id and name test
29adaf8
@bensjsilverstein
Feat - Add id and name methods
071c6a1
@bensjsilverstein
Feat - Add 'exists' test
6a18dd7
@bensjsilverstein
Feat - Add MerchantRepository class
3add087
@bensjsilverstein
Feat - Add test for 'all' method
e7b7710
@bensjsilverstein
Feat - Add 'all' method
78f3104
@bensjsilverstein
Fix -
b4e493e 
Adjust 'all' test to include count.
Adjust initialize to include more CSV path
@bensjsilverstein
Feat - Add 'find_by_id' test
05198a5
@bensjsilverstein
Fix -
cc806c1 
Adjust 'find_by_id' method
Adjust 'find_by_id' test
@bensjsilverstein
Feat - Add 'find_by_name' test
8bd70d5
@bensjsilverstein
Feat - Add 'find_by_name' method
92684cc
@bensjsilverstein
Feat - Add 'find_all_by_name' test
2a8c97f
@bensjsilverstein
Feat - Add 'find_all_by_name' method
b9677a6
@bensjsilverstein
Feat - Add 'create' test
6250075
@bensjsilverstein
Fix - remove 'create' from expect statements
4850cad
@bensjsilverstein
Fix - adjust 'create' test spacing
82ee722
@bensjsilverstein
Feat - Add MerchantRepository 'update' test
ebf0b70
@bensjsilverstein
Feat - Add MerchantRepository 'update' method
d0df47c
@bensjsilverstein
Fix - Adjust expect statement (remove .all)
848f221
@Ethan-t-n
Merge pull request #2 from Ethan-t-n/merchant_test_bjs
f88d7b7 
Merchant test bjs
@Ethan-t-n
Merge branch 'master' of github.com:Ethan-t-n/black_thursday
ebb0cb4
@bensjsilverstein
Feat - Add MerchantRepository 'delete' test
79a0df2
sdecesari and others added 30 commits June 2, 2022 17:42
@sdecesari
Merge pull request #5 from Ethan-t-n/feat/add_sales_engine_and_tests
f0adf87 
Feat/add sales engine and tests
@sdecesari
Merge branch 'master' of github.com:Ethan-t-n/black_thursday
6fb7fbb
@sdecesari
Simplecov update
ce2eaa5
@bensjsilverstein
Merge pull request #6 from Ethan-t-n/feat/simplecov-sd
e5d0742 
Feat/simplecov sd

BenS Review:
PR #6
Ran SimpleCov - observed 100% coverage (NICE) - change approved
Coverage files added - change approved
@sdecesari
Add exists test
3ba7a90
@sdecesari
Add analyst method
a0cbb4c
@sdecesari
Add non-working initialize
2a7f70d
@Ethan-t-n
Merge pull request #7 from Ethan-t-n/feat/iteration1-sd
3065773 
Feat/iteration1 sd
@bensjsilverstein
Add 'entry.rb' file to track require statements for class files
2c1c2c1
@bensjsilverstein
Fix - spacing adjusted
5728d09
@bensjsilverstein
Test - add 'exists' and 'average_items' test for SalesAnalyst
4428c3b
@bensjsilverstein
Feat - add 'average_items_per_merchant' method
e60ad0b
@bensjsilverstein
Fix - Adjust require statement, adjust SalesAnalys argument
7931d9b
@Ethan-t-n
add initial test and code for invoice repo
9813775
@Ethan-t-n
add initial test for invoice
933436e
@Ethan-t-n
add invoice class
dd0d0f6
@Ethan-t-n
fix require
1e022e3
@Ethan-t-n
add test for returning array and find by id
a212f10
@Ethan-t-n
add find_by_id method
a9adbb0
@Ethan-t-n
add test for find all by customer id
c542793
@Ethan-t-n
add test for find all by merchant id
5632c92
@Ethan-t-n
add method for find all by merchant id an refactor test
ac6ec20
@Ethan-t-n
add test for find all by status
f13eb14
@Ethan-t-n
add method for find all by status
a07ea4e
@Ethan-t-n
add initial test for create
00f57a8
@Ethan-t-n
add method for create
914cfae
@Ethan-t-n
add test for update
f2c31c0
@Ethan-t-n
add method for update and refactor tests
ce21731
@Ethan-t-n
add delete test for invoice repo
16432fa
@Ethan-t-n
add delete method
9bea140
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.
4 participants
@Ethan-t-n @bensjsilverstein @sdecesari @brycesimonds