Skip to content

Conversation

Copy link

Copilot AI commented Feb 2, 2026

Addresses review feedback from #1737 regarding malformed assertion in test/test_database.py:101.

Changes

  • Fixed assertion using comma operator instead of equality check:
    # Before: tests truthiness with misleading error message
    assert first_dqw.message, "Hello == World!"
    
    # After: tests actual equality
    assert first_dqw.message == "Hello, World!"
  • Corrected expected value to match test fixture data ("Hello, World!" vs "Hello == World!")

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: jacalata <2009720+jacalata@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback from PR #1737 for release v0.39 Fix incorrect assertion syntax in test_populate_data_quality_warning Feb 2, 2026
Copilot AI requested a review from jacalata February 2, 2026 06:48
@jacalata jacalata marked this pull request as ready for review February 2, 2026 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants