Skip to content

tests.system.test_sqlalchemy_bigquery: test_dml failed #111

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.

Sign up for GitHub

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

Jump to bottom
Closed
flaky-bot bot opened this issue Apr 2, 2021 · 2 comments · Fixed by #114
Closed

tests.system.test_sqlalchemy_bigquery: test_dml failed #111

flaky-bot bot opened this issue Apr 2, 2021 · 2 comments · Fixed by #114
Assignees
@tswast
Labels
api: bigquery Issues related to the googleapis/python-bigquery-sqlalchemy API. flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@flaky-bot
Copy link

flaky-bot bot commented Apr 2, 2021

This test failed!

To configure my behavior, see the Flaky Bot documentation.

If I'm commenting on this issue too often, add the flakybot: quiet label and
I will stop commenting.

commit: 0cfc5de
buildURL: Build Status, Sponge
status: failed

Test output
engine = Engine(bigquery://)
session = 
table_dml = Table('test_pybigquery.sample_dml', MetaData(bind=Engine(bigquery://)), Column('integer', Integer(), table=), Column('array', ARRAY(Integer()), table=), schema=None)

def test_dml(engine, session, table_dml):
    # test insert
    engine.execute(table_dml.insert(ONE_ROW_CONTENTS_DML))
    result = table_dml.select().execute().fetchall()





  assert len(result) == 1





E       AssertionError: assert 4 == 1

E        +  where 4 = len([(588, datetime.datetime(2013, 10, 10, 11, 27, 16, tzinfo=), 'test', 40.76727216, Decimal('40.76727216'), False, ...ime(2013, 10, 10, 11, 27, 16), datetime.time(11, 27, 16), b'test_bytes', None, None, None, None, None, None, None, [])])


tests/system/test_sqlalchemy_bigquery.py:516: AssertionError
@flaky-bot flaky-bot bot added flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Apr 2, 2021
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery-sqlalchemy API. label Apr 2, 2021
@tswast tswast mentioned this issue Apr 6, 2021
Closed
@tswast tswast self-assigned this Apr 6, 2021
@tswast tswast mentioned this issue Apr 6, 2021
Merged
4 tasks
@flaky-bot
Copy link
Author

flaky-bot bot commented Apr 6, 2021

Test passed for commit 7b14a06 (Build Status, Sponge)! Closing this issue.

@flaky-bot flaky-bot bot closed this as completed Apr 6, 2021
@tswast
Copy link
Collaborator

tswast commented Apr 6, 2021

Test passed only because I manually deleted the problem table. We'll still want #114.

gcf-merge-on-green bot pushed a commit that referenced this issue Apr 7, 2021
@tswast
test: use random suffix in DML test table (#114)
29c7fb0 
This should avoid flakes. Also, adds a table expiration so that
if cleanup fails, BigQuery will still eventually remove the
table.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-bigquery-sqlalchemy/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)

Fixes #111  🦕
@flaky-bot flaky-bot bot mentioned this issue Apr 29, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tswast tswast

Labels
api: bigquery Issues related to the googleapis/python-bigquery-sqlalchemy API. flakybot: issue An issue filed by the Flaky Bot. Should not be added manually. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: use random suffix in DML test table tswast/python-bigquery-sqlalchemy
1 participant
@tswast