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

BankAccount(1) #3

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

BankAccount(1) #3

wants to merge 106 commits into from

Conversation

paxmeny
Copy link

@paxmeny paxmeny commented Feb 5, 2020

No description provided.

jmoczynski and others added 30 commits January 22, 2020 13:41
@jmoczynski
updated tests
2e1bc98
@paxmeny
Update .gitignore
d77676f
@paxmeny
Update BankAccountTest.java
636e7f1
@damionlance
Fixed isEmailValid
db7e412
@damionlance
fixed push error
94ae5a1
@damionlance
added comments
ef4c075
@jmoczynski
incorporates InsufficientFundsException
dbfa1c7
@cmartano
is email valid code
493681d
@cmartano
is email valid code added
39075ce
@jmoczynski
updated tests to include border cases
c3eb9e2
@cmartano
withdraw function updated
eda63cb
@jmoczynski
fixed withdraw arguments, fixed withdraw tests
f1c2537
@jmoczynski
added structure and tests for BankAccount.isAmountValid
b84760a
@jmoczynski
implements BankAccount.isAmountValid, fixed tests for BankAccount.wit…
4d2e7d7 
…hdraw
@jmoczynski
updated tests for constructor
128b8be
@jmoczynski
updated implementations for constructor and BankAccount.withdraw
6350588
@jmoczynski
added structure for BankAccount.deposit and BankAccount.transfer, add…
9ff3ed9 
…ed tests for BankAccount.deposit
@jmoczynski
added tests for BankAccount.transfer
c4e8a4c
@jmoczynski
implemented BankAccount.deposit and BankAccount.transfer
220123e
@jmoczynski
updated test and implementation for values of 0
2eece30
@jmoczynski
added provided interfaces for project 2
deb9a5a
@Rmelikyan
added User class
1776b71
@paxmeny
Add files via upload
8d2ebbc
@paxmeny
AddedCheckingAccount
318ebad
@Rmelikyan
started working on Deposit! much todo
c7365cc
@Rmelikyan
Merge branch 'master' into RobertsDeposit
2b65e38 
Merging master to my deposit work so far
@Rmelikyan
sdk setup
4305b8f
@Rmelikyan
Projects UML Class diagram
c9969c6
@Rmelikyan
updated central bank to acknowledge implementation of BasicAPI
e56a604
@Rmelikyan
Merge branch 'master' of https://github.com/jmoczynski/SoftwareEngine…
9cbb4bb 
…eringPractice
jmoczynski and others added 30 commits February 14, 2020 01:18
@jmoczynski
Merge pull request #1 from paxmeny/master
00deb45 
resolved potential conflicts
@jmoczynski
Merge branches 'SavingsAccount' and 'master' of https://github.com/jm…
00dccf0 
…oczynski/SoftwareEngineeringPractice into SavingsAccount
@jmoczynski
wrote some tests in SavingsAccountTest.transferTest, modified some te…
d49af2e 
…sts in SavingsAccountTest.withdrawTest, modified methods and structure of SavingsAccount
@jmoczynski
fixed BankAccount.isAmountValid and BankAccountTest.isAmountValidTest
bc1c1c5
@jmoczynski
renamed SavingsAccount.applyCompoundInterest and SavingsAccount.apply…
fcaf3da 
…CompoundInterestTest
@Rmelikyan
fixed merge conflict with is amount valid
7a8704b
@jmoczynski
fixed implementation BankAccount.isAmountValid, completed implementat…
1798a59 
…ion of SavingsAccount.java, fixed tests in SavingsAccountTest.java
@jmoczynski
Merge branch 'master' into SavingsAccount
37e534d
@jmoczynski
updated BankAccount.isAmountValid
7ff029b
@jmoczynski
Merge pull request #3 from jmoczynski/SavingsAccount
61fac69 
Savings account pull request
@Rmelikyan
Updated BankUML.pdf
c49a199
@Rmelikyan
readme update
4664f39
@Rmelikyan
readMe
21e6c97
@Rmelikyan
withdraw unit tests added along with some necessary functionallity
5dc27ec
@Rmelikyan
withdraw is working
507137b
@jmoczynski
adding tests for CheckingAccount in CheckingAccountTest
81df945
@jmoczynski
updated CheckingAccountTest.constructorTest, CheckingAccountTest.crea…
30551db 
…teCheckingAccountTest
@jmoczynski
updated CheckingAccountTest.withdrawTest
e834c7f
@jmoczynski
updated CheckingAccountTest.depositTest, CheckingAccountTest.transfer…
6442b3b 
…Test, SavingsAccount, SavingsAccountTest
@Rmelikyan
Added ATM_StateDiagram.pdf
83ef6ee
@Rmelikyan
added state diagram to github
71777cd
@Rmelikyan
Merge branch 'master' of https://github.com/jmoczynski/SoftwareEngine…
70683d3 
…eringPractice
@Rmelikyan
Added unit tests for confirm credentials and some implementation infa…
4d0bc46 
…structure
@Rmelikyan
tests pass'
1f696d1
@jmoczynski
Merge branches 'CheckingAccount' and 'master' of https://github.com/j…
5fbd1a8 
…moczynski/SoftwareEngineeringPractice
@jmoczynski
Merge pull request #4 from jmoczynski/CheckingAccount
9204610 
Checking account
@paxmeny
Add files via upload
796d0be
@Rmelikyan
Implemented a UI for ATM. Extensive manual testing has found and solv…
2e35f2c 
…ed many bugs
@Rmelikyan
Merge branch 'master' of https://github.com/jmoczynski/SoftwareEngine…
2e9cbfd 
…eringPractice
@Rmelikyan
restored ATMUI
10ae75e 
"
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.
5 participants
@paxmeny @jmoczynski @damionlance @cmartano @Rmelikyan