Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

Commit 7d3ef40

Browse files
authored
Stabilize discovery test using standard top level breadcrumb filter (singer-io#145)
* Stabilize discovery test using standard top level breadcrumb filter * Move tap-postgres out of tier 1 as tests no longer pass after ubuntu upgrade from 18 to 22 * Pin tap-tester image to ubuntu 18 --------- Co-authored-by: btowles <btowles@stitchdata.com>
1 parent e863a4d commit 7d3ef40

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.circleci/config.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ orbs:
55
jobs:
66
build:
77
docker:
8-
- image: 218546966473.dkr.ecr.us-east-1.amazonaws.com/circle-ci:stitch-tap-tester
8+
- image: 218546966473.dkr.ecr.us-east-1.amazonaws.com/circle-ci:stitch-tap-tester-18.04
99
- image: singerio/postgres:9.6-wal2json-2.2-ssl
1010
environment:
1111
POSTGRES_USER: postgres
@@ -49,7 +49,7 @@ workflows:
4949
- build:
5050
context:
5151
- circleci-user
52-
- tier-1-tap-user
52+
- tap-tester-user
5353
build_daily:
5454
<<: *commit_jobs
5555
triggers:

tests/test_postgres_discovery.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -404,7 +404,7 @@ def discovery_test(self, conn_id):
404404
)
405405
actual_fields_to_datatypes = {
406406
item['breadcrumb'][1]: item['metadata'].get('sql-datatype')
407-
for item in stream_metadata[1:]
407+
for item in stream_metadata if item['breadcrumb'] != []
408408
}
409409

410410
# Verify there is only 1 top level breadcrumb in metadata

0 commit comments

Comments
 (0)