-
Notifications
You must be signed in to change notification settings - Fork 13
Feature/e2e integration tests #31
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
Merged
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
09f4cff
Merge pull request #17 from jpmorganchase/main
knighto82 5582e23
Incremented version to 0.03-SNAPSHOT
knighto82 2f944fe
feature/update usage examples (#30)
knighto82 205848b
Initial play with pact for contract testing
knighto82 e46eef3
Added PACT consumer tests
knighto82 c95755d
Update build-on-push.yml
knighto82 c43fffa
Update build-on-push.yml
knighto82 24037b1
Changed home of PACT tests
knighto82 43a8b5b
Merge branch 'feature/e2e-integration-tests' of github.com:jpmorganch…
knighto82 5c067d8
Additional Logging
knighto82 65b2cb2
Updated logging
knighto82 9d3d27c
Updated Logging
knighto82 3755079
spotless
knighto82 549f652
Log level to DEBUG
knighto82 cf3de6e
Remove verification of User-Agent header in PACT
knighto82 bee3e1c
spotless
knighto82 f1224b3
Tidy up following github failing build
knighto82 9213ffc
Update Consumer Tests for FusionApi
knighto82 a50fe63
Updating PACT tests to align to provider
knighto82 3d1e11c
Updating PACT tests to align to provider
knighto82 8f55772
spotless corrections
knighto82 e43543d
Updated provider and consumer ids
knighto82 2e6f00f
Updated PACT Samples
knighto82 ee3e2f6
Added some negative test cases for PACT
knighto82 61d0d83
spotless adjustments
knighto82 0f0a0f5
Update 110274-fusionsdk-consumer-110274-fusionapi-provider.json
knighto82 4173923
Further PACT tweaks
knighto82 b79ea5b
Tweaks as per provider integration of PACT
knighto82 f67e36e
Corrected failing test
knighto82 27f7c56
Update 110274-fusionsdk-consumer-110274-fusionapi-provider.json
knighto82 028e099
Removing redundant e2e tests
knighto82 2b62dad
simplified the provision of dataset tokens at construction
knighto82 2e32d12
changes as per pr review
knighto82 4fb2319
spotless ....
knighto82 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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,3 +4,4 @@ | |
**/localAuthCredential.json | ||
**/client_credentials.json | ||
.vscode | ||
.DS_Store |
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
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
62 changes: 0 additions & 62 deletions
62
src/it/java/io/github/jpmorganchase/fusion/e2e/MetadataRetrievalTest.java
This file was deleted.
Oops, something went wrong.
30 changes: 0 additions & 30 deletions
30
src/it/resources/e2e-responses/catalog-success-response.json
This file was deleted.
Oops, something went wrong.
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
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
27 changes: 27 additions & 0 deletions
27
src/main/java/io/github/jpmorganchase/fusion/oauth/provider/SimpleDatasetTokenProvider.java
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
package io.github.jpmorganchase.fusion.oauth.provider; | ||
|
||
import io.github.jpmorganchase.fusion.oauth.exception.OAuthException; | ||
import io.github.jpmorganchase.fusion.oauth.model.BearerToken; | ||
import java.util.Map; | ||
|
||
public class SimpleDatasetTokenProvider implements DatasetTokenProvider { | ||
|
||
private static final String BEARER_TOKEN_NOT_FOUND_ERROR = "Bearer token not found for catalog %s and dataset %s"; | ||
|
||
private final Map<String, BearerToken> datasetTokens; | ||
|
||
public SimpleDatasetTokenProvider(Map<String, BearerToken> datasetTokens) { | ||
this.datasetTokens = datasetTokens; | ||
} | ||
|
||
@Override | ||
public String getDatasetBearerToken(String catalog, String dataset) { | ||
|
||
String tokenKey = String.format("%s_%s", catalog, dataset); | ||
if (datasetTokens.containsKey(tokenKey)) { | ||
return datasetTokens.get(tokenKey).getToken(); | ||
} | ||
|
||
throw new OAuthException(String.format(BEARER_TOKEN_NOT_FOUND_ERROR, catalog, dataset)); | ||
} | ||
} |
43 changes: 0 additions & 43 deletions
43
src/test/java/io/github/jpmorganchase/fusion/example/FusionInstanceCreationExamples.java
This file was deleted.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.