FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add system test for 'Sink.update'. by tseaver · Pull Request #1681 · googleapis/google-cloud-python · GitHub

Add system test for 'Sink.update'. - #1681

Merged
tseaver merged 1 commit into
googleapis:logging-apifrom
tseaver:logging-system_tests-sink_update
Mar 30, 2016
Merged

Add system test for 'Sink.update'.#1681
tseaver merged 1 commit into
googleapis:logging-apifrom
tseaver:logging-system_tests-sink_update

Conversation

tseaver commented Mar 30, 2016

Copy link
Copy Markdown
Contributor

No description provided.

tseaver added testing api: logging Issues related to the Cloud Logging API. labels Mar 30, 2016
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 30, 2016
Comment thread system_tests/logging_.py
def test_update_sink(self):
bucket_uri = self._init_storage_bucket()
dataset_uri = self._init_bigquery_dataset()
UPDATED_FILTER = 'logName:syslog'

This comment was marked as spam.

This comment was marked as spam.

dhermes commented Mar 30, 2016

Copy link
Copy Markdown
Contributor

LGTM

tseaver commented Mar 30, 2016

Copy link
Copy Markdown
Contributor Author

Travis is dawwwwg-slow today.

tseaver commented Mar 30, 2016

Copy link
Copy Markdown
Contributor Author

I pushed this change 'Two hours ago' -- still no Travis love?

tseaver merged commit 7ae4fd5 into googleapis:logging-api Mar 30, 2016
tseaver deleted the logging-system_tests-sink_update branch March 30, 2016 19:25
parthea pushed a commit that referenced this pull request Apr 1, 2026
* test: streaming test check bigtable and pubsub results

* fix mypy

* remove unrelated
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: logging Issues related to the Cloud Logging API. cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL