Skip to content

test only installed packages#10

Merged
samth merged 1 commit intoracket:masterfrom
mflatt:pkg-dirs
Mar 30, 2026
Merged

test only installed packages#10
samth merged 1 commit intoracket:masterfrom
mflatt:pkg-dirs

Conversation

@mflatt
Copy link
Copy Markdown
Member

@mflatt mflatt commented Mar 30, 2026

Instead of trying to test everyting the in "pkgs" directory of a checkout, test only directories that are installed as packages.

This addresses the problem of extra packages in "pkgs" (such as "ffi2", currently) that cannot be tested because they are not installed.

Instead of trying to test everyting the in "pkgs" directory of a
checkout, test only directories that are installed as packages.

This addresses the problem of extra packages in "pkgs" (such as
"ffi2", currently) that cannot be tested because they are not
installed.
@samth samth merged commit 6cd90f3 into racket:master Mar 30, 2026
1 check passed
@rfindler
Copy link
Copy Markdown
Member

Looks like this triggered this failure, maybe? https://drdr.racket-lang.org/72534/cs/racket/share/pkgs/gui-test/tests/gracket/pasteboard.rkt

@rfindler
Copy link
Copy Markdown
Member

Also, is the [current-rev 1234567890987654321] parameter setting on purpose?

@mflatt
Copy link
Copy Markdown
Member Author

mflatt commented Mar 31, 2026

The failing test involves the clipboard, so maybe it's a race with other tests that also use the clipboard — where the race started to happen because the changes here affected the ordering of tests and the ones that run at the same time.

Yes, the parameter setting is on purpose. It's for a mode that runs tests on a local checkout, instead of on the DrDr machine.

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.

3 participants