Skip to content
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

[functions] fix: fix broken tests #3774

Merged
merged 4 commits into from
May 18, 2020

Conversation

tmatsuo
Copy link
Contributor

@tmatsuo tmatsuo commented May 16, 2020

fixes #3773

This PR disables some broken tests.

fixes GoogleCloudPlatform#3773

This PR disables some broken tests.
@tmatsuo tmatsuo requested review from grant and a team as code owners May 16, 2020 20:21
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 16, 2020
@tmatsuo tmatsuo requested a review from billyjacobson May 16, 2020 20:22
@tmatsuo tmatsuo requested a review from kurtisvg May 18, 2020 14:50
@kurtisvg
Copy link
Contributor

Why do we want to disable the tests? I would prefer to leave them enabled so that they block any changes to the samples until fixed

functions/pubsub/main_test.py Show resolved Hide resolved
functions/slack/noxfile_config.py Show resolved Hide resolved
@tmatsuo
Copy link
Contributor Author

tmatsuo commented May 18, 2020

I'll merge this since I know the other changes are in different directory

@tmatsuo tmatsuo merged commit b63fc2c into GoogleCloudPlatform:master May 18, 2020
@tmatsuo tmatsuo deleted the fix-functions branch May 18, 2020 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

functions.bigtable.main_test: test_main failed
4 participants