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.

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

Input Output file bugs #59

Closed
Ank1taS opened this issue Jul 5, 2022 · 1 comment · Fixed by #60, #62 or #64
Closed

Input Output file bugs #59

Ank1taS opened this issue Jul 5, 2022 · 1 comment · Fixed by #60, #62 or #64
Assignees
@H1manshu21 @chandrakant100 @Ank1taS
Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed Important
Milestone
Testing and Deplo...

Comments

@Ank1taS
Copy link
Member

Ank1taS commented Jul 5, 2022

Found the Following 4 possible bugs (till now) while passing I/O file(s) in the command line:

  • When the Input file is a .BMP file but the output file is not a .BMP file, the application executes without resulting in any error message.

notbmpout

  • When Input file is an exiting non .BMP file, the application terminates but not with an appropriate error message however it should happen before promoting the option to the user.

Screenshot_2022-07-05_22-15-12

  • When Both the input and output files are not .BMP files, the application executes without resulting in any error message.

bothnotBMP

  • When Input and Output file both are the same file, the application terminates but not with an appropriate error massage however it should happen before promoting the option to the user.

sameBMP
@H1manshu21 H1manshu21 added bug Something isn't working enhancement New feature or request help wanted Extra attention is needed Important labels Jul 6, 2022
@H1manshu21 H1manshu21 added this to the Testing and Deployment milestone Jul 6, 2022
@H1manshu21 H1manshu21 pinned this issue Jul 6, 2022
@H1manshu21
Copy link
Member

@chandrakant100 These bugs need to be fixed since it can drive the application in another way instead of the right direction.

chandrakant100 added a commit that referenced this issue Jul 7, 2022
@chandrakant100
FIxed issue #59: Input output file bugs
b4bbe0a
@chandrakant100 chandrakant100 linked a pull request Jul 7, 2022 that will close this issue
Fixed issue #59: Input output file bugs #60
Merged
chandrakant100 added a commit that referenced this issue Jul 8, 2022
@chandrakant100
Merge pull request #60 from Blaze-Stars/bugFix
f753151 
Fixed issue #59: Input output file bugs
This was linked to pull requests Jul 9, 2022
Usage rectification #62
Merged
recorrect exceptions and add exception for same input output file #64
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@H1manshu21 H1manshu21

@chandrakant100 chandrakant100

@Ank1taS Ank1taS

Labels
bug Something isn't working enhancement New feature or request help wanted Extra attention is needed Important
Projects
None yet
Milestone
Testing and Deployment
3 participants
@H1manshu21 @chandrakant100 @Ank1taS