Skip to content

Commit

Permalink
[integration] Fix log lines affected by headers
Browse files Browse the repository at this point in the history
Not the ideal fix - just a quick out: the addition of the apache headers changes the line numbers that are logged out when warnings are emitted. Our tests shouldn’t be this brittle, but this is a quick work-around for now.
  • Loading branch information
econchick committed Sep 16, 2020
1 parent 2d3a17f commit 29be0a5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion integration/batch-modular-default/expected_job_output.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
INFO:root:Found worker image: integration-batch-modular-default:batch-modular-default
/usr/src/app/transforms.py:9: KlioFutureWarning: 'handle_klio' is experimental and is subject to incompatible changes, or removal in a future release of Klio.
/usr/src/app/transforms.py:22: KlioFutureWarning: 'handle_klio' is experimental and is subject to incompatible changes, or removal in a future release of Klio.
@decorators.handle_klio
INFO:klio:Output not found at gs://klio-integration/batch-modular-default/outputs/battleclip_daq.wav
INFO:klio:Output not found at gs://klio-integration/batch-modular-default/outputs/callout_Turn-Table.wav
Expand Down
2 changes: 1 addition & 1 deletion integration/read-file/expected_job_output.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
INFO:root:Found worker image: integration-test-klio-read-file-write-file:read-file
/usr/src/app/transforms.py:12: KlioFutureWarning: 'handle_klio' is experimental and is subject to incompatible changes, or removal in a future release of Klio.
/usr/src/app/transforms.py:25: KlioFutureWarning: 'handle_klio' is experimental and is subject to incompatible changes, or removal in a future release of Klio.
@decorators.handle_klio
INFO:klio:Hello, Klio!
INFO:klio:Received element b'0nQLxiNGbbUoSGrxksETLm'
Expand Down

0 comments on commit 29be0a5

Please sign in to comment.