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

Test failures #39

Open
EelcoPeacs opened this issue Jan 13, 2023 · 1 comment
Open

Test failures #39

EelcoPeacs opened this issue Jan 13, 2023 · 1 comment

Comments

@EelcoPeacs
Copy link

Several of the test cases provided with the package indicate differences in outcome. Especially the test cases

  • prn4
  • prn9
  • prn13
  • prn14
  • prn15
  • test2
  • test3

In these test cases it seems that the Fan-in and Fan-out values have switched places in the structural metrics summery. For the other test cases these values are not present, so it seems like a structural bug
@EelcoPeacs
Copy link
Author

Dived a bit deeper into the history and found that this issue is related to #16 where the bug was fixed that cause the tests to fail. Solution will be to adapt the test reference HTML and XML files.

The test HTML reference files also need to be updated for the changed acknowledgement at the bottom of the files

@EelcoPeacs EelcoPeacs mentioned this issue Jan 16, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@EelcoPeacs