Skip to content

Commit

Permalink
make exchangerate integration test depend on docker building (airbyte…
Browse files Browse the repository at this point in the history
…hq#1157)

* add exchange rate dep and change behavior

* fix integration test dependencies

* revert singer helpers changes
  • Loading branch information
jrhizor authored Dec 2, 2020
1 parent 75f411d commit 55ea0b6
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,6 @@ def singer_catalog_to_airbyte_catalog(singer_catalog: Dict[str, any]) -> Airbyte
name = stream.get("stream")
schema = stream.get("schema")
airbyte_stream = AirbyteStream(name=name, json_schema=schema)

metadatas = stream.get("metadata")
stream_metadata_container = get_stream_level_metadata(metadatas) if metadatas else None
stream_metadata = stream_metadata_container.get("metadata")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,5 @@ dependencies {
integrationTestImplementation project(':airbyte-protocol:models')
implementation files(project(':airbyte-integrations:bases:base-python').airbyteDocker.outputs)
}

integrationTest.dependsOn airbyteDocker
2 changes: 2 additions & 0 deletions buildSrc/src/main/groovy/airbyte-integration-test-java.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@ class AirbyteIntegrationTestJavaPlugin implements Plugin<Project> {
exceptionFormat "full"
}

outputs.upToDateWhen { false }

dependsOn project.airbyteDocker
mustRunAfter project.test
}
Expand Down

0 comments on commit 55ea0b6

Please sign in to comment.