Enable windup's tests on Windows and MacOS #1671

mrizzi · 2023-05-24T13:43:47Z

The goal is to check issue https://issues.redhat.com/browse/WINDUP-3881 will happen when running ZipExportingTest on Windows.

Changes:

ReferenceResolvingVisitor.extractDefinitionLine method made 'new line marker' indipendent
JavaASTReferenceResolverTest: same test Java classes in java-ast/tests/src/test/resources have different sizes on different OSes
ExecutionStatisticsTest: replace Thread.sleep with busy wait because Linux and Windows have different interrupt perios (ref. https://stackoverflow.com/a/11498647)
Tests: refactored FileUtils.deleteDirectory in finally blocks to FileUtils.deleteQuietly due to issues with deletions on Windows (which would be worth being properly investigated rather than being workarounded)
Switch to shell: pwsh to consistently retrieve the CHROMEWEBDRIVER env variable

…izes on diffent OS

…'FileUtils.deleteQuietly'

…variable

…Windows characters error

mrizzi and others added 13 commits May 24, 2023 15:42

Enable windup's tests on Windows and MacOS

0163eb7

actions/cache enableCrossOsArchive

46c2f44

Absolute path to local maven repo

d65c4e8

extractDefinitionLine made 'new line marker' indipendent

d1733d1

JavaASTReferenceResolverTest: manage same test files with different s…

32b8fc6

…izes on diffent OS

extractDefinitionLine made 'new line marker' indipendent

442a3d2

ExecutionStatisticsTest: replace Thread.sleep with busy wait

bbd5c78

Windows: fix the wrong 'webdriver.chrome.driver=/chromedriver' value

f2d7c31

Tests: refactored 'FileUtils.deleteDirectory' in 'finally' blocks to …

f8b824b

…'FileUtils.deleteQuietly'

NoInputOrOutputPathTest: fixed the never thrown IOException

53feab8

Switch to 'shell: pwsh' to consistently retrieve CHROMEWEBDRIVER env …

3f65aff

…variable

chromedriver: conditional path to executable

b8cdda5

Output URI created from a java.io.File rather than a String to avoid …

117afd9

…Windows characters error

Provide feedback