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

Sus activity message #28

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

Sus activity message #28

wants to merge 125 commits into from

Conversation

jmendezdietsch
Copy link

No description provided.

jmendezdietsch and others added 30 commits January 22, 2020 14:41
False tests added
5e43a8c
@MunkyCode
isEmailValid built to tests
b06d5c7
@MunkyCode
sdding comments on equivalence cases and border cases
2babee9
One letter and true tests added
21a3248
@MunkyCode
now include single letter extensions
320e723
New test added for equvilence test and boundaries, need single extens…
2016659 
…ion code and withdraw tests
Some more tests added for assing 1-3
e6a2965
Comments to new tests
f39df93
getBalanceTest test added (need to change code for negative numbers)
bc4e081
Withdraw tests added
91a2d08
Tests update
e15f2d5
@MunkyCode
Implemntation for isEmailValid and withdraw methods
6ed4c13
More comments added to main to describe actions of is email as well a…
a8bbd87 
…s separation of isEmailValid
isAmountValid tests added and some code; 4/5 tests pass
17cd360
decimalPoints fixed to find index
ce0a1f0
isAmountValid works but check with toby about exception, deposit work…
5f948d1 
…s/needs comments, and transfer tests/need comments
All test do work but need to do illegalException throw
3a962b8
Exception throw messed transfer, deposit, withdraw, and get balance t…
461d1c6 
…ests
All tests pass with all working code
fe21ecd
Toby code added, CODE LEVELS UP
bf6a900
@klpolley
create account setup/tests
56f42aa
@klpolley
updated test for correct email rules, added quick account existence test
2835e22
@klpolley
create account functioning
83f59dd
@klpolley
made central bank match API (total assets method name error)
702e465
@klpolley
merge master
75c793a
@vcrabtree
Added withdraw branch
618b805
@vcrabtree
Added DepositTest class with tests for deposit
2dcd72b
@klpolley
tests
714a7c2
@vcrabtree
Updates to test
671c711
@klpolley
deposit
e91d7e3
jmendezdietsch and others added 30 commits February 10, 2020 17:05
Error updates after merge
0570fce
Fixed/added to suspicious acctounts
462225e
Test updated to add freeze/unfreeze and isFrozen
c7719aa
Merge branch 'transaction'
ddcd8c1
All tests work except for HistoryTest, numbers shift to different acc…
80a6c22 
…ounts
@vcrabtree
Integration test
629fe2d
@klpolley
fixing withdraw real quick
ab32bfb
@vcrabtree
System test
5cc4979
@vcrabtree
Fixed merge conflicts
ae47deb
@jmendezdietsch
Merge pull request #11 from jmendezdietsch/veraTests
d8c932c 
Vera tests
ReadMe.md and some transactionTest work
88b6c78
@klpolley
fixing all of the things!!!!
7d41182
@klpolley
Merge branch 'advancedAPITest' and ALL THE TESTS PASS WOOOOOOO
1299f90
@vcrabtree
Updated Vera system Test
2aad42d
@klpolley
Update README.md
2119026
equals change
53494b0
What to do (Josue) in read me
277b472
@klpolley
Update README.md
b20cc42
@klpolley
some tests to determine what is suspicious and what is not
26574fb
@klpolley
updated tests for actual suspicion criteria
2ce62de
@klpolley
find suspicious accounts code
4d0d400
@klpolley
testing below s.d.
cdb8212
@vcrabtree
Merge pull request #12 from jmendezdietsch/suspiciousUpdate
220a09f 
Suspicious Accounts
@vcrabtree
Added account frozen exception tests to withdraw, deposit, and transfer
b87f7f6
@vcrabtree
Added additional test to transfer
c1e7e83
@vcrabtree
Updated transfer method
9e5c36b
@klpolley
Merge pull request #13 from jmendezdietsch/accoutFrozenTests
c3d09a9 
Added account frozen exception tests to withdraw, deposit, and transfer
The print of the question but scanner doesnt accept any input
74477c2
confirmActivity method created to print values/clear suspicious activ…
f6b9421 
…ity but Scanner not allowing to input
@klpolley
just a thought
550acee
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
@jmendezdietsch @MunkyCode @klpolley @vcrabtree