Refactor: Delete outdated dummy data and unused mock assets#616
Refactor: Delete outdated dummy data and unused mock assets#616uju09 wants to merge 4 commits intoAOSSIE-Org:devfrom
Conversation
Signed-off-by: uju09 <24bcs012@iiitdwd.ac.in>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🎉 Welcome @uju09!
We appreciate your contribution! 🚀 |
|
@M4dhav I used script to check the dummy data, but the tests are failing as they are searching for mocks. |
|
@coderabbitai Review |
✅ Actions performedReview triggered.
|
|
@uju09: I'll review the changes in this PR. I'll pay special attention to the failing tests you mentioned that are looking for the removed mocks - this will need to be addressed to ensure the tests pass after the cleanup. ✅ Actions performedReview triggered.
|
Could you please elaborate on this |
|
@M4dhav I used a dart script to find unused assets and deleted them, but the run test is failing. |
|
@M4dhav This are the unused assets that have been deleted.
|
|
I would recommend going through the files manually, as there is dummy data except assets |
|
@M4dhav Ok, I will go through files manually, but it will take sometime as my End Sems are going on. |
|
is this issue still open?? |
this is a PR, you can check #555 for the issue |
M4dhav
left a comment
There was a problem hiding this comment.
Automated tests are still failing
|
@M4dhav I will work on this PR in two working days. |
Signed-off-by: uju09 <24bcs012@iiitdwd.ac.in>
Signed-off-by: uju09 <24bcs012@iiitdwd.ac.in>
|
@M4dhav I have delete the mock data, if u want any changes, let me know. |
|
@M4dhav I request you to review it. |
M4dhav
left a comment
There was a problem hiding this comment.
Some non-mock data has been removed that needs to be added back, and other mock data is still not removed
|
@M4dhav Hey, Can I know which non-mock data is removed. I used script to identify the mock data into the codebase. |
|

Description
This PR removes unused dummy/mock data and outdated asset files that were previously used for testing UI functionality but are no longer referenced anywhere in the codebase.
Cleaning these unnecessary files improves maintainability, reduces project size, and aligns the app with current production assets.
Fixes #555
Type of change
How Has This Been Tested?
lib/directory.No functional behavior was changed.
Checklist:
Maintainer Checklist
refactor