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

Create some tests for porting stuff working #38

Merged
merged 15 commits into from
Dec 13, 2024
Merged

Create some tests for porting stuff working #38

merged 15 commits into from
Dec 13, 2024

Conversation

Toddelismyname
Copy link
Contributor

Calls filegenerator and then loops the file and trys to find old v2 stuff.

@Toddelismyname
first try tests
f8f5d8d
@Toddelismyname
fix false positive
d3694f6
@Toddelismyname
add solver interface closing check and change m2: to be recognized
e827cdd
@Toddelismyname
loop over lines so it works correctly
204dbd3
@Toddelismyname
add helpful "prints" in test that tells if something was ported right
e6fe2c8
@Toddelismyname
add timing check
7b37450
@Toddelismyname
fix typo
aeb78b0
@Toddelismyname
fix typo and remove wrongly commited file
1a32f91
@Toddelismyname
github workflow first attempt (untested)
59d5685
@Toddelismyname
remove develop branch and allow manually triggering the workflow
7ea2bc8
@Toddelismyname
add port example because test cant use generated folder(gitignore)
727ad5b
@Toddelismyname
use new port example in workflow and remove verbose arg for pytest(as…
8bd30aa 
… well as use newer checkout action)
@Toddelismyname
add flag to pytest so print statements can be seen
8904c2f
@Toddelismyname
instead of checking file call filegenerator and then test the output …
c898d89 
…of it
@Toddelismyname Toddelismyname self-assigned this Dec 13, 2024
@VanLaareN VanLaareN merged commit 7b27977 into main Dec 13, 2024
1 check passed
@VanLaareN VanLaareN deleted the work/toddel/some-tests branch December 13, 2024 13:15
@Toddelismyname Toddelismyname mentioned this pull request Dec 13, 2024
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VanLaareN VanLaareN VanLaareN approved these changes

Assignees

@Toddelismyname Toddelismyname

Labels
None yet
Projects
None yet
Milestone
First ~10 Porting Points done
Development

Successfully merging this pull request may close these issues.
2 participants
@Toddelismyname @VanLaareN