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

Write VIA tests #204

Open
mattgodbolt opened this issue Aug 6, 2019 · 4 comments
Open

Write VIA tests #204

mattgodbolt opened this issue Aug 6, 2019 · 4 comments

Comments

@mattgodbolt
Copy link
Owner

Automate @scarybeasts 's tests attached to #179 so we don't lose the good behaviour.
@mattgodbolt
Copy link
Owner Author

@scarybeasts I'm going to assume you'd be happy if I took these tests into jsbeeb itself.

@mattgodbolt
Copy link
Owner Author

Well, some have atrophied it seems already. bums.

@mattgodbolt
Copy link
Owner Author

Need to fix the AC1 test at least: b971354 broke it (will file separate issue)

@mattgodbolt mattgodbolt mentioned this issue Sep 11, 2024
Closed
mattgodbolt added a commit that referenced this issue Sep 11, 2024
@mattgodbolt
Add a test for the fix for #439, and start on #204
ec43caa
@mattgodbolt mattgodbolt mentioned this issue Sep 11, 2024
Merged
@mattgodbolt
Copy link
Owner Author

Still got two TODOs here, checking the output of C4 and C5. @tom-seddon 's b2 has the same issues (thanks tom!)

mattgodbolt added a commit that referenced this issue Sep 13, 2024
@mattgodbolt
Add VIA tests (#440)
5cd1b95 
Implement VIA tests
- All @scarybeasts' non-NMI tests ported and updated to work in the test framework
- Fixes a trivial issue with joysticks.
- Two tests skipped until I understand what the correct behaviour should be; and the NMI tests ignored for now.

Fixes #439. Part of #204
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
@mattgodbolt