-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Windows pytest fixes #7078
Windows pytest fixes #7078
Conversation
Corrected instance of hard-coded unix path separator.
tools/test/memap/parse_test.py
Outdated
memap.parse(join(dirname(__file__), "arm.map"), "UARM") | ||
assert memap.modules == PARSED_ARM_DATA | ||
|
||
PARSED_DATA_OS_AGNOSTIC = dict() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is a local, let's use lower case.
tools/test/memap/parse_test.py
Outdated
@@ -35,7 +39,12 @@ def test_parse_armcc(): | |||
def test_parse_iar(): | |||
memap = MemapParser() | |||
memap.parse(join(dirname(__file__), "iar.map"), "IAR") | |||
assert memap.modules == PARSED_IAR_DATA | |||
|
|||
PARSED_DATA_OS_AGNOSTIC = dict() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lower case.
tools/test/memap/parse_test.py
Outdated
@@ -49,9 +58,14 @@ def test_parse_iar(): | |||
def test_parse_gcc(): | |||
memap = MemapParser() | |||
memap.parse(join(dirname(__file__), "gcc.map"), "GCC_ARM") | |||
assert memap.modules == PARSED_GCC_DATA | |||
|
|||
PARSED_DATA_OS_AGNOSTIC = dict() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lower case
Output with current PR checked out in a Windows 7 machine:
Beautiful. |
/morph build |
Build : SUCCESSBuild number : 2221 Triggering tests/morph test |
Test : SUCCESSBuild number : 2010 |
Exporter Build : SUCCESSBuild number : 1846 |
/morph uvisor-test |
/morph uvisor-test |
Description
In the process of narrowing down Windows Py3 compatibility issues, discovered that pytest is never actually run on Windows.
Ran pytest and corrected issues found.
Pull request type