You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think, now that we have automatic CI test runs, a few more test would be helpful. For that i suggest to let the emulator panic or return an error if we hit an unknown command. At the moment unknown commands are simply ignored.
It is maybe a good idea to test against the 6502_functional_test.a65. What say the other people?
The text was updated successfully, but these errors were encountered:
Aye, that's a great idea!
Maybe there's a way to auto-generate some tests from the resource (and others) that you mentioned. Do you want to tackle that one?
We got quite a few more unit tests now, so I'll go ahead and close this.
The functional test is still untouched but I don't think we should keep that issue open any longer. Probably there are better ways to do it by now than my crude attempt in #17.
Hi!
I think, now that we have automatic CI test runs, a few more test would be helpful. For that i suggest to let the emulator panic or return an error if we hit an unknown command. At the moment unknown commands are simply ignored.
It is maybe a good idea to test against the 6502_functional_test.a65. What say the other people?
The text was updated successfully, but these errors were encountered: