Skip to content

SDK-3304: Added extra node e2e tests #2468

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
Merged
jholleran merged 24 commits into from
Aug 20, 2024
Merged

SDK-3304: Added extra node e2e tests #2468

jholleran merged 24 commits into from
Aug 20, 2024

Conversation

@jholleran
Copy link
Contributor

@jholleran jholleran commented Aug 20, 2024

Node e2e tests for the Problem Details feature

Checklist

  • All acceptance criteria are met.
  • Relevant documentation, if any, has been written/updated.
  • The changelog has been updated, if applicable.
  • New functions/types have been exported in index.ts, if applicable.
  • New modules (i.e. new .ts files) are listed in the exports field in package.json, if applicable.
  • New modules (i.e. new .ts files) are listed in the typedocOptions.entryPoints field in tsconfig.json, if applicable.
  • Commits in this PR are minimal and have descriptive commit messages.

acoburn and others added 23 commits August 15, 2024 13:38
@acoburn
Integrate problem details into Solid client library
50c04af
@acoburn
Problem details tests
9f8031d
@acoburn
Adjust test URLs
2f67681
@acoburn
Adjust test URLS
2949822
@acoburn
Additional URLs
451e7b6
@acoburn
Code review
aa7cb4b
@acoburn
Merge branch 'main' into feat/problem-details
3c3059d
@NSeydoux
First draft at implementing WithProblemDetails
6dc568e
@jholleran
Fix lint issues
fc25f2a
@acoburn
Linter
e1303be
@acoburn
Adjust tests
f22d522
@acoburn
Linter, clean up tests
4276e8e
@acoburn
Clean up comments
4509588
@jholleran
Throwing fetch error with response body
3a6537b
@jholleran
Added e2e tests to verify problem details responses
8805978
@jholleran
Test error not propagated back to the client
9066724
@jholleran
Added more tests
06cce60
@jholleran
Added e2e tests to verify problem details errors
3296dce
@jholleran
Refactored out util test functions
2c7b20a
@jholleran
Updated e2e tests
b2063f2
@jholleran
Detail and instance are undefined from a head request response
a450255
@jholleran
Merge remote-tracking branch 'origin/main' into feature/SDK-3304-adde…
b385bbf 
…d-extra-node-e2e-tests
@jholleran
Clean up PR
a23d17d
@jholleran jholleran requested a review from a team as a code owner August 20, 2024 10:36
@jholleran jholleran changed the title Feature/sdk 3304 added extra node e2e tests SDK-3304: Added extra node e2e tests Aug 20, 2024
@jholleran jholleran merged commit eb8e86f into main Aug 20, 2024
@jholleran jholleran deleted the feature/SDK-3304-added-extra-node-e2e-tests branch August 20, 2024 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NSeydoux NSeydoux NSeydoux approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jholleran @NSeydoux @acoburn