Skip to content
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

feat(e2e): smoketest macOS .dmg files COMPASS-8709 #6582

Merged
merged 16 commits into from
Dec 27, 2024
Merged

feat(e2e): smoketest macOS .dmg files COMPASS-8709 #6582

merged 16 commits into from
Dec 27, 2024

Conversation

lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Dec 27, 2024

Replaces #6579

COMPASS-8709

Installs the .app found in the .dmg to /Applications then runs the time to first query test on the app in /Applications.

To run locally:

m -rf /Applications/MongoDB\ Compass\ Dev.app
HADRON_DISTRIBUTION=compass ARCH=arm64 IS_OSX=true ./smoke-test.ts --extension=dmg --skip-download

@github-actions github-actions bot added the feat label Dec 27, 2024
@lerouxb lerouxb added no release notes Fix or feature not for release notes feat and removed feat labels Dec 27, 2024
{
name: 'smoketest-macos-x64',
display_name: 'Smoketest MacOS Intel',
run_on: 'macos-14-gui',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oof. I missed -gui when I first mapped this out and didn't notice for days.

appInfo = await installMacDMG(buildInfo.productName, pkg);
}

// TODO: all the other installers go here
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ie. the other sub-task tickets in this same story.

@lerouxb lerouxb merged commit 6bf3d97 into main Dec 27, 2024
26 of 28 checks passed
@lerouxb lerouxb deleted the test-dmg-2 branch December 27, 2024 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat no release notes Fix or feature not for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants