Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Update test_server.py #1218

Merged
merged 1 commit into from
Apr 4, 2024
Merged

fix: Update test_server.py #1218

merged 1 commit into from
Apr 4, 2024

Conversation

cpacker
Copy link
Collaborator

@cpacker cpacker commented Apr 3, 2024

Loosen test to fix non-deterministic unit test, eg see failure in: https://github.com/cpacker/MemGPT/actions/runs/8544849064/job/23411836736

        assert passages_1[0]["text"] == "alpha"
>       assert len(passages_2) == 3
E       AssertionError: assert 4 == 3
E        +  where 4 = len([{'agent_id': UUID('48249189-c3d0-490f-a3ea-b0f4a6336518'), 'created_at': datetime.datetime(2024, 4, 3, 20, 8, 21, 422464, tzinfo=datetime.timezone.utc), 'data_source': 'test_source', 'doc_id': UUID('c4f43552-5cee-cf6b-cd4f-667287a2a23f'), ...}, {'agent_id': UUID('48249189-c3d0-490f-a3ea-b0f4a6336518'), 'created_at': datetime.datetime(2024, 4, 3, 20, 8, 55, 972034, tzinfo=datetime.timezone.utc), 'data_source': None, 'doc_id': None, ...}, {'agent_id': UUID('48249189-c3d0-490f-a3ea-b0f4a6336518'), 'created_at': datetime.datetime(2024, 4, 3, 20, 8, 21, 689716, tzinfo=datetime.timezone.utc), 'data_source': 'test_source', 'doc_id': UUID('aa188965-821d-3c57-ff38-d911514ca632'), ...}, {'agent_id': UUID('48249189-c3d0-490f-a3ea-b0f4a6336518'), 'created_at': datetime.datetime(2024, 4, 3, 20, 8, 21, 572660, tzinfo=datetime.timezone.utc), 'data_source': 'test_source', 'doc_id': UUID('e1a06b72-6d11-3a7c-2623-fc48d72da8e9'), ...}])

@cpacker cpacker requested a review from sarahwooders April 3, 2024 20:34
@cpacker cpacker merged commit 9efc892 into main Apr 4, 2024
4 checks passed
@cpacker cpacker deleted the cpacker-patch-1 branch April 4, 2024 00:59
mattzh72 pushed a commit that referenced this pull request Oct 9, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant