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

V2 Unit Testing - Implementation and Client Testing #246

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

V2 Unit Testing - Implementation and Client Testing #246

wants to merge 82 commits into from

Conversation

Zakaria-Kofiro
Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro commented Jun 29, 2023 •
edited

title: - V2 Unit Testing - Implementation and Client Testing

Unit Testing covering V2 implementation code using JUnit and Mockito

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

zkofiro added 30 commits March 23, 2023 10:29
feat: Add V2 clients
cc89064
fix: clean up + switching to webclient exception handling
0f5989d
fix: better exception handling
efe65c6
feat: [SRE-26521] upload/download for scripts/datafiles
66d2ca4 
fix: supports both normal and gzip file uploads

fix: updating get specific script to script JSON description, download handles content

fix: updated max file size and added exception handling for file exceptions

fix: set debug to false

fix: group all exception handlers

fix: updating prefix

fix: unneeded file

fix: reverting file change

Revert "fix: reverting file change"

This reverts commit 0ba90f6.

fix: revert file

fix: reverting file as well

fix: detailed error msg for incorrect request
Revert "feat: [SRE-26521] upload/download for scripts/datafiles"
2fe9b7e 
This reverts commit 66d2ca4.
Merge branch 'master' into zkofiro/v2-clients
630a5cb
feat: Updated clients, new routes, status response from agent/job ops…
96c58f8 
…, and testing agent client with APIHarness
fix: listing only user-facing endpoints
c550a9e
fix: small updates to deps + docs for client changes
ae9923d
fix: temp fix for tools while testing agent
2e70ebb
fix: removing unneeded files
5c96250
Merge branch 'master' into zkofiro/v2-clients
567757d
fix: temp fix for client tools
bb9e9eb
fix: missing file
24c8ba5
moved last backend V1 deps over with models + tested locally
b84720f
updating file documentation
b2a0e7a
reverting debug flag
5167739
moving other misc backend changes to this branch
d98d611
moving over one more backend change
6259919
updated in UI ref branch
7ac6a61
Merge branch 'zkofiro/move-remaining-V1' into zkofiro/SRE-26707
a096470
Merge branch 'master' into zkofiro/v2-clients
42d5651
Merge branch 'master' into zkofiro/SRE-26707
99c5754
Merge branch 'zkofiro/SRE-26707' into zkofiro/v2-clients
e29c166
cleaning up, removing old files
997ebde
testing base agent client
cf8a544
small clean up
875e8f6
dep injection fix
9287885
need to shorten V2 url due to dep inj fix
59ba285
missing dependency
f587553
zkofiro added 19 commits June 12, 2023 22:50
revert after testing logging fix
7f031e2
legacy spring startup, removing from web/WEB-INF
d2ae166
re-adding bridge deps
a57438c
fixed versions
37ff905
logging config issue fix: including maven-shade-plugin fixes issue lo…
3778440 
…cally
updating gitignore
094ef10
logs showing locally, just needed to exclude extra servlet fragment
30d448b
switching scope out for absolute-ordering, logging to file locally
95f4295
moving from tank-api to agent module
99c4239
exclude Log4j2plugins.dat w/ maven shaded plugin instead, avoiding de…
e358551 
…p issues
updating fix for other modules, tested locally
72bb839
uncommenting tests now that builds passing
5b9ec82
updated not needed, pending deletion
932b170
filter endpoint fix - moving script models out to api module
6db2e74
Revert "filter endpoint fix - moving script models out to api module"
f627fec 
This reverts commit 6db2e74.
new filter fix: reverting all script models to v1 until external scri…
0a2cc49 
…pts updated
reduced war file size by 80M, still needs more dep exclusions from sp…
5779312 
…ring
updated unit testing
4a2ca07
reverting
e8eb83e
@Zakaria-Kofiro Zakaria-Kofiro marked this pull request as ready for review June 29, 2023 21:03
Base automatically changed from zkofiro/v2-clients to master June 29, 2023 21:49
@Zakaria-Kofiro Zakaria-Kofiro removed the request for review from shawn-h-park December 1, 2023 05:30
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
@Zakaria-Kofiro