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

added test_fileMetadata.py #106

Merged
merged 4 commits into from
Oct 9, 2024
Merged

added test_fileMetadata.py #106

merged 4 commits into from
Oct 9, 2024

Conversation

Sai-ganesh-0004
Copy link
Contributor

@Sai-ganesh-0004 Sai-ganesh-0004 commented Oct 9, 2024

Description
This PR introduces the file test_fileMetadata.py, which contains unit tests for the insert_file method in the FileMetadata class. These tests ensure that:

  • A new file path can be successfully inserted into the metadata.
  • Duplicate file paths are not added to the list

Fixes #7: Added unit tests for the insert_file method in the FileMetadata class to ensure that new files can be added correctly and that duplicates are not inserted.

Type of change

  • Tests update

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes
  • I have maintained a clean commit history by using the necessary Git commands
  • I have checked that my code does not cause any merge conflicts

So what I have done is I have created 4 dummy files to test the behavior of the insert_file method and tested based on that.

Screenshot 2024-10-09 085959

@Kota-Karthik
Copy link
Owner

@Sai-ganesh-0004
delete this file twinTrim/dataStructures/test_fileMetadata.py

@Sai-ganesh-0004
Copy link
Contributor Author

Can you check it once

@techy4shri
Copy link
Collaborator

@Sai-ganesh-0004 We will check this as soon as we can!

@Kota-Karthik
Copy link
Owner

@Sai-ganesh-0004 LGTM!

@Kota-Karthik Kota-Karthik merged commit 92c3996 into Kota-Karthik:main Oct 9, 2024
2 checks passed
@Sai-ganesh-0004 Sai-ganesh-0004 deleted the branch1 branch October 10, 2024 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Write Unit Tests for insert_file Method
3 participants