Skip to content

Commit

Permalink
Merge pull request #1070 from dandi/1065-failing-tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jjnesbitt authored Apr 29, 2022
2 parents 4a6e91d + 7d20685 commit 3794e59
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 1 deletion.
1 change: 1 addition & 0 deletions .github/workflows/backend-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,3 +54,4 @@ jobs:
DJANGO_DANDI_WEB_APP_URL: http://localhost:8085
DJANGO_DANDI_API_URL: http://localhost:8000
DJANGO_DANDI_JUPYTERHUB_URL: https://hub.dandiarchive.org/
DANDI_ALLOW_LOCALHOST_URLS: 1
2 changes: 2 additions & 0 deletions .github/workflows/frontend-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,7 @@ jobs:
DJANGO_DANDI_WEB_APP_URL: http://localhost:8085
DJANGO_DANDI_API_URL: http://localhost:8000
DJANGO_DANDI_JUPYTERHUB_URL: https://hub.dandiarchive.org/
DANDI_ALLOW_LOCALHOST_URLS: 1

# TODO: Remove this job when the rest of the puppeteer-based tests are ported to pyppeteer
test-e2e-jest-puppeteer:
Expand Down Expand Up @@ -110,6 +111,7 @@ jobs:
DJANGO_DANDI_WEB_APP_URL: http://localhost:8085
DJANGO_DANDI_API_URL: http://localhost:8000
DJANGO_DANDI_JUPYTERHUB_URL: https://hub.dandiarchive.org/
DANDI_ALLOW_LOCALHOST_URLS: 1

# Web client env vars
VUE_APP_DANDI_API_ROOT: http://localhost:8000/api/
Expand Down
2 changes: 1 addition & 1 deletion dandiapi/api/tests/factories.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ class Meta:
model = User

username = factory.SelfAttribute('email')
email = factory.Faker('safe_email')
email = factory.Faker('free_email')
first_name = factory.Faker('first_name')
last_name = factory.Faker('last_name')

Expand Down
1 change: 1 addition & 0 deletions dev/.env.docker-compose
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,4 @@ DJANGO_DANDI_DANDISETS_EMBARGO_BUCKET_NAME=dandi-embargo-dandisets
DJANGO_DANDI_WEB_APP_URL=http://localhost:8085
DJANGO_DANDI_API_URL=http://localhost:8000
DJANGO_DANDI_JUPYTERHUB_URL=https://hub.dandiarchive.org/
DANDI_ALLOW_LOCALHOST_URLS=1
1 change: 1 addition & 0 deletions dev/.env.docker-compose-native
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,4 @@ DJANGO_DANDI_DANDISETS_EMBARGO_BUCKET_NAME=dandi-embargo-dandisets
DJANGO_DANDI_WEB_APP_URL=http://localhost:8085
DJANGO_DANDI_API_URL=http://localhost:8000
DJANGO_DANDI_JUPYTERHUB_URL=https://hub.dandiarchive.org/
DANDI_ALLOW_LOCALHOST_URLS=1
2 changes: 2 additions & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ passenv =
DJANGO_DANDI_WEB_APP_URL
DJANGO_DANDI_API_URL
DJANGO_DANDI_JUPYTERHUB_URL
DANDI_ALLOW_LOCALHOST_URLS
extras =
dev
test
Expand Down Expand Up @@ -84,6 +85,7 @@ passenv =
DJANGO_DANDI_WEB_APP_URL
DJANGO_DANDI_API_URL
DJANGO_DANDI_JUPYTERHUB_URL
DANDI_ALLOW_LOCALHOST_URLS
PYPPETEER_BROWSER_HEADLESS
deps =
factory-boy
Expand Down

0 comments on commit 3794e59

Please sign in to comment.