No test for malformed graph

[GREENRAT-K405]

The current tests for cli only capture two scenarios:

• file is missing or
• file is valid
  it lacks to test the cases where the file does exist but the graphml file has corrupted XML.

I propose adding a corrupted XML test

[GREENRAT-K405]

@pradeeban , what are your thoughts on this? is this required?

I'd like to work on this issue!

[avinxshKD]

@pradeeban , what are your thoughts on this? is this required?

I'd like to work on this issue!

Makes sense ,, I'd suggest we test at least:

1. Broken XML syntax
2. Missing required GraphML attributes
3. Invalid data types in node properties

@pradeeban want me to draft a PR with a few test cases?

[GREENRAT-K405]

@avinxshKD I am already working on this issue. Thanks for offering your help but please work on different issue.

[avinxshKD]

@avinxshKD I am already working on this issue. Thanks for offering your help but please work on different issue.

Just to clarify, I was adding suggestions to the discussion, not claiming the issue. All good, I’ll put up a PR for the test cases I mentioned. If thats okay @pradeeban, thanks