Skip to content

test: make more crypto tests work with BoringSSL - #46429

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
codebytere:tests-for-boringssl
Feb 1, 2023
Merged

test: make more crypto tests work with BoringSSL#46429
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
codebytere:tests-for-boringssl

Conversation

@codebytere

Copy link
Copy Markdown
Member

This PR tweaks several crypto tests to work with BoringSSL so that Electron does not need to disable them and can smoke test our embedded Node.js executable with them.

@codebyterecodebytere changed the title test: tweak several tests to work with BoringSSLtest: tweak several crypto tests to work with BoringSSLJan 30, 2023
@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jan 30, 2023
@codebyterecodebytere changed the title test: tweak several crypto tests to work with BoringSSLtest: make more crypto tests work with BoringSSLJan 30, 2023
@lpincalpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 30, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 30, 2023
@nodejs-github-bot

This comment was marked as outdated.

@panvapanva added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 30, 2023
@tony-go

Copy link
Copy Markdown
Member

@dsanders11@RaisinTen

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@codebyterecodebytere added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 31, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 1, 2023
@nodejs-github-bot
nodejs-github-bot merged commit 8e42d8c into nodejs:mainFeb 1, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 8e42d8c

MylesBorins pushed a commit that referenced this pull request Feb 18, 2023
PR-URL: #46429
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Feb 19, 2023
danielleadams pushed a commit that referenced this pull request Apr 11, 2023
PR-URL: #46429
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.needs-ciPRs that need a full CI run.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@codebytere@nodejs-github-bot@tony-go@panva@jasnell@lpinca