Skip to content

Commit

Permalink
test: skip doxygen test on macos 11 + use brew
Browse files Browse the repository at this point in the history
  • Loading branch information
aminya committed Feb 19, 2024
1 parent dbfc8a8 commit ea2732f
Show file tree
Hide file tree
Showing 8 changed files with 19 additions and 12 deletions.
2 changes: 1 addition & 1 deletion dist/actions/setup-cpp.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/actions/setup-cpp.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/legacy/setup-cpp.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/legacy/setup-cpp.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/modern/setup-cpp.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/modern/setup-cpp.js.map

Large diffs are not rendered by default.

6 changes: 6 additions & 0 deletions src/doxygen/__tests__/doxygen.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,15 @@ import { InstallationInfo } from "../../utils/setup/setupBin"
import { getVersion } from "../../versions/versions"
import which from "which"
import { ubuntuVersion } from "../../utils/env/ubuntu_version"
import { macosVersion } from "../../utils/env/macos_version"

jest.setTimeout(300000)
describe("setup-doxygen", () => {
if (process.platform === "darwin" && macosVersion()[0] <= 11) {
test.skip("Skipping doxygen test on macOS 11 or earlier", () => {})
return
}

let directory: string
beforeAll(async () => {
directory = await setupTmpDir("doxygen")
Expand Down
13 changes: 7 additions & 6 deletions src/doxygen/doxygen.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,12 +75,13 @@ export async function setupDoxygen(version: string, setupDir: string, arch: stri
return installationInfo
}
case "darwin": {
let installationInfo: InstallationInfo
try {
installationInfo = await setupBin("doxygen", version, getDoxygenPackageInfo, setupDir, arch)
} catch {
installationInfo = await setupBrewPack("doxygen", undefined)
}
// let installationInfo: InstallationInfo
// try {
// installationInfo = await setupBin("doxygen", version, getDoxygenPackageInfo, setupDir, arch)
// } catch {
const installationInfo = await setupBrewPack("doxygen", undefined)
// }

// only install graphviz if the macOS version is greater than 11
if (macosVersion()[0] > 11) {
await setupGraphviz(getVersion("graphviz", undefined), "", arch)
Expand Down

0 comments on commit ea2732f

Please sign in to comment.