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

Structure and test refactor #24

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

Eathox
Copy link
Owner

@Eathox Eathox commented Jun 1, 2022

Split most structs into their own files

@github-actions
Copy link

github-actions bot commented Jun 1, 2022

Test Results

  1 files  ±0    4 suites  ±0   0s ⏱️ ±0s
73 tests +5  73 ✔️ +5  0 💤 ±0  0 ±0 

Results for commit 92b6e57. ± Comparison against base commit b01fdac.

This pull request removes 59 and adds 64 tests. Note that renamed tests count towards both.
input::tests_input ‑ duplicate_facts
input::tests_input ‑ duplicate_queries
input::tests_input ‑ empty_facts
input::tests_input ‑ empty_queries
input::tests_input ‑ error_double_facts
input::tests_input ‑ error_double_queries
input::tests_input ‑ error_empty
input::tests_input ‑ error_from_file_non_exist
input::tests_input ‑ error_no_facts
input::tests_input ‑ error_no_queries
…
input::sanitize_lines::tests::remove_comment ‑ mixed
input::sanitize_lines::tests::remove_comment ‑ no_comment
input::sanitize_lines::tests::remove_comment ‑ only_comment
input::sanitize_lines::tests::remove_spaces ‑ mixed
input::sanitize_lines::tests::remove_spaces ‑ no_whitespace
input::sanitize_lines::tests::remove_spaces ‑ only_whitespace
input::sanitize_lines::tests::sanitize_lines ‑ empty_lines
input::sanitize_lines::tests::sanitize_lines ‑ empty_lines_at_end
input::sanitize_lines::tests::sanitize_lines ‑ every_other_empty
input::sanitize_lines::tests::sanitize_lines ‑ example_input
…

♻️ This comment has been updated with latest results.

@Eathox Eathox requested a review from nanderstabel June 1, 2022 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant