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.

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

feat: test that SDK respects alias #710

Merged
merged 9 commits into from
May 14, 2021
Merged

feat: test that SDK respects alias #710

merged 9 commits into from
May 14, 2021

Conversation

Jwhiles
Copy link
Contributor

@Jwhiles Jwhiles commented May 12, 2021

Adds a test for the entry editor location to make sure the SDK is hitting the correct endpoints in aliased environments.

If we're happy with this approach, we might want to add similar tests for each SDK/location as a follow up

@Jwhiles Jwhiles changed the title Env tests feat: test that SDK respects alias May 14, 2021
@Jwhiles Jwhiles requested a review from andipaetzold May 14, 2021 09:54
@Jwhiles Jwhiles marked this pull request as ready for review May 14, 2021 09:55
test/integration/index.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@andipaetzold andipaetzold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Seems like a straightforward way to check forward way to verify request paths.

@Jwhiles Jwhiles merged commit 5597a75 into master May 14, 2021
@Jwhiles Jwhiles deleted the env-tests branch May 14, 2021 12:40
Jwhiles added a commit that referenced this pull request May 14, 2021
* feat: create an alias before running tests

* feat: alias id test

* feat: try to make it fail

* Revert "feat: try to make it fail"

This reverts commit 8a0539f.

* feat: wait till env is ready to make alias

* Revert "Revert "feat: try to make it fail""

This reverts commit f2e13e0.

* Revert "Revert "Revert "feat: try to make it fail"""

This reverts commit 6de184d.

* feat: add extra assertion

* chore: pass alias id
loweisz added a commit that referenced this pull request May 17, 2021
loweisz added a commit that referenced this pull request May 18, 2021
ghost pushed a commit that referenced this pull request May 18, 2021
## [3.34.3](v3.34.2...v3.34.3) (2021-05-18)

### Reverts

* "chore: test that SDK respects alias ([#710](#710))" ([#717](#717)) ([4ab6e48](4ab6e48))
ghost pushed a commit that referenced this pull request May 18, 2021
# [4.0.0-alpha.4](v4.0.0-alpha.3...v4.0.0-alpha.4) (2021-05-18)

### Bug Fixes

* more explicit space api types ([#711](#711)) ([42d63de](42d63de))

### Reverts

* "chore: test that SDK respects alias ([#710](#710))" ([#717](#717)) ([4ab6e48](4ab6e48))
loweisz pushed a commit that referenced this pull request Jun 30, 2021
# [4.0.0-alpha.4](v4.0.0-alpha.3...v4.0.0-alpha.4) (2021-05-18)

### Bug Fixes

* more explicit space api types ([#711](#711)) ([42d63de](42d63de))

### Reverts

* "chore: test that SDK respects alias ([#710](#710))" ([#717](#717)) ([4ab6e48](4ab6e48))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants