-
Notifications
You must be signed in to change notification settings - Fork 258
Fix JavaLocalLambdaRunConfigurationIntegrationTest integ tests #4549
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.
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
Merged
vchikoti1998
merged 62 commits into
main
from
vchikoti/javaLocalLambdaRunConfigInteg_Test
Jul 10, 2024
Merged
Changes from all commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
93f7afb
testing integ tests
vchikoti1998 e461ac4
add gradle user home
vchikoti1998 897d9f0
increased memory to 4gb
vchikoti1998 823b54c
deleting lock file if exists
vchikoti1998 82f471c
fix trimIndent
vchikoti1998 45e90e8
fix detekt
vchikoti1998 c03058f
setup gradle.properties to tst-project dir
vchikoti1998 6af9479
test
vchikoti1998 b720ba1
tst
vchikoti1998 2ea0f42
tst with module main path
vchikoti1998 6b7379e
set property Gradle_user_home
vchikoti1998 c8e352f
set env var for specific lambda runConfig
vchikoti1998 dcf4c88
created a tem folder
vchikoti1998 0da051f
tst
vchikoti1998 6c6137c
temp
vchikoti1998 f7af73e
envVar in SamBuildLambda
vchikoti1998 8ddbc85
testing
vchikoti1998 9fa1948
Memory size bumped
vchikoti1998 e934195
cleanUp folder
vchikoti1998 294b61b
add sleep
vchikoti1998 f15e33e
removed sleep
vchikoti1998 d7f60a0
inclue stacktrace
vchikoti1998 cdc67b0
revert stacktrace
vchikoti1998 f9e97d5
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 570c6ba
stacktrace for integTests
vchikoti1998 7f14384
set maxHeapSize
vchikoti1998 2a06d8f
tst debug
vchikoti1998 be59d68
revert heapSize
vchikoti1998 4601fca
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 80167ef
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 b8d1547
test maxheapsize
vchikoti1998 69eec7d
revert maxheap
vchikoti1998 bd3ce52
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 094f55c
tst memeory leaks
vchikoti1998 8c4a7d1
tst rider memory leaks
vchikoti1998 9ef586a
tst
vchikoti1998 51751a6
revert deploy and upload integs ignore
vchikoti1998 9ccfe1a
revert python localLAmbda integ ignore
vchikoti1998 a334f2d
removed ignore for riderInteg
vchikoti1998 e666ce1
delete sln file after test
vchikoti1998 2436377
test
vchikoti1998 3da6809
test
vchikoti1998 c1b5951
set env var
vchikoti1998 341784e
stacktrace for linuxIntegs
vchikoti1998 5eb7e29
cleanUp tempdir
vchikoti1998 968ef56
cleanUp code
vchikoti1998 78dbd55
addressed feedback
vchikoti1998 9bd6e08
remove @afterclass in src
vchikoti1998 0fef2f6
test
vchikoti1998 8d3ca01
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 d2c098a
add sam build option flag
vchikoti1998 6afb57e
revert test case samExecuableTest
vchikoti1998 e28dca9
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 7966412
comment out for test
vchikoti1998 caacfef
changes reverted
vchikoti1998 7f82b2d
set env var in runconfig
vchikoti1998 568b258
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 371b856
added buildInContainer
vchikoti1998 f1c5342
removed env map
vchikoti1998 93c06ce
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 cd041d4
add comment
vchikoti1998 72495b2
Merge branch 'main' into vchikoti/javaLocalLambdaRunConfigInteg_Test
vchikoti1998 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you explain why we need to build in container? why would we want to do one over the other?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As expected, making it run in a containerized environment doesn't require env vars to be set The test case passed since container has its own grade user home dir.