-
Notifications
You must be signed in to change notification settings - Fork 106
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
CUMULUS-175 : Dashboard providers not in sync with AWS providers #234
Changes from 95 commits
0f6b38b
ef9c3a5
68972e2
6283b7d
cb034bd
eef1b7a
112a61d
0651f32
0bded00
dc13c6c
f355950
716236a
5897973
aadd41d
835e425
ec7c968
e8782be
9f1585c
b221d04
b895d5e
5d380a4
aa033e5
60d7971
a67b403
0109204
80e2a3f
097a847
b2dc9e1
1dec5d6
cc67847
7a41c07
8445180
640e072
72be0eb
3d118fb
a016508
ba1e5f8
a250d1f
420920d
6fa6168
a1025e5
c177d73
cde24cc
e821ee9
6e27a3d
985df5b
539af30
8f33f99
d937023
86a8d99
7f2bc4a
a2f59ae
c050529
1b260c9
65641e1
9887f79
8ffe593
36c487c
3506cd3
ef465ba
735b1f1
44049ec
70ff7b3
45c0d3d
f755f66
879b85e
001e00e
ec04089
646ca31
e2f2434
9b7ee0c
017ab82
8da4a47
092544f
d8416f3
63005e8
db4148b
a75daa3
fd229a2
dda5bae
c8cd6fe
c111242
06be378
5ec36c7
76c7c08
05b89d8
3eb8edd
29c7182
81cafc3
f3c1a8d
8d864ef
d35f752
8ffed62
c6371fd
0756f53
adf4b4b
05c31a6
7512183
d8174e6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -29,11 +29,12 @@ See [Cumulus README](https://github.com/cumulus-nasa/cumulus/blob/master/README. | |
Running tests for kinesis-consumer depends on localstack. Once you have installed localstack, you can start it for dynamoDB only: | ||
|
||
``` | ||
SERVICES=dynamodb localstack start | ||
LAMBDA_EXECUTOR=docker localstack start | ||
``` | ||
|
||
Then you can run tests locally via: | ||
|
||
```bash | ||
LOCALSTACK_HOST=localhost npm run test | ||
export DOCKERHOST=$(ifconfig | grep -E "([0-9]{1,3}\.){3}[0-9]{1,3}" | grep -v 127.0.0.1 | awk '{ print $2 }' | cut -f2 -d: | head -n1) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looking at this, I don't think there is a way to run this locally with our main docker image. So I'm not sure anymore if we have to move this to the root level of the repo. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think the db-indexer tests need some work to get working from the root of the repo. I'm going to skip them for now in hopes of returning to get them running as part of the larger test suite. This could help testing other lambdas, especially those in @cumulus/api. |
||
LOCALSTACK_HOST=localhost DOCKERHOST=${DOCKERHOST} IS_LOCAL=true npm run test | ||
``` |
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.
Since we are running tests from the root level of the repo, we should add this to the main readme if necessary. Alternatively, we can add the env variable to the docker-compose.yml, to have it available as part of the
docker-compose up local
command.