Skip to content

Comments

Re-enable tests, and make (most of) them pass#10

Open
eric-wieser wants to merge 3 commits into6004x:masterfrom
eric-wieser:fix-bsim-tests
Open

Re-enable tests, and make (most of) them pass#10
eric-wieser wants to merge 3 commits into6004x:masterfrom
eric-wieser:fix-bsim-tests

Conversation

@eric-wieser
Copy link

Starts fixing #9

@eric-wieser eric-wieser mentioned this pull request Nov 2, 2015
@eric-wieser
Copy link
Author

Remaining failures for which I don't know what the correct fix is:

  • Emulation: Mouse interrupts: Mouse coordinates stored in mouse coord buffer
    Test returns 0x00c80064, expects 0x006400c8 (ie, x and y swapped). Is the test wrong, or the emulator
- Verifier checksums seem not to agree. Maybe the algorithm changed since the test was written? Is it choking on windows line endings in source files due to a windows checkout? - discussed [here](https://github.com/6004x/6.004_tools/commit/9b24a8e94a3f91898ebfa378c90a4c5d0f26657f#commitcomment-14140970)

@eric-wieser eric-wieser changed the title Make metadata optional. Most tests now pass. Re-enable test, and make (most of) them pass Nov 2, 2015
@eric-wieser eric-wieser changed the title Re-enable test, and make (most of) them pass Re-enable tests, and make (most of) them pass Nov 2, 2015
eric-wieser referenced this pull request Nov 3, 2015
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