workflows/tests: use ruby/setup-ruby

formulae_dependents: install formula dependencies first

formulae_dependents: run postinstall before testing dependents

This fixes an issue where important etc/var files, which may have been nuked during the build st...

Gemfile: restrict activesupport version range

build(deps): bump minitest from 5.14.4 to 5.15.0

build(deps-dev): bump activesupport from 6.1.4.1 to 6.1.4.3

Bumps [minitest](https://github.com/seattlerb/minitest) from 5.14.4 to 5.15.0.
- [Release notes]...

Bumps [activesupport](https://github.com/rails/rails) from 6.1.4.1 to 6.1.4.3.
- [Release notes]...

Apply self-hosted logic to Linux

Rename env to `GITHUB_ACTIONS_HOMEBREW_SELF_HOSTED`.

formulae_dependents: skip strict linkage check when building from source

Building dependents from source is an opportunistic linkage party. We
don't really need to see t...

formulae_dependents: install subversion, etc when needed

Fixes CI failures in Homebrew/homebrew-core#89642.

formulae: don't skip formulae with a dependent in the same test-bot run

While doing some testing I discovered that adding two new formulae,
where one dependends on the ...

formulae_dependents: build dependents from source only on request

formulae: check deps against `testing_formulae` instead

I trialled this behaviour in Homebrew/homebrew-core#89857, and it adds a
significant amount of t...

This is a better fix for the issue I tried to fix in #733.

Checking dependents against `@new_fo...

formulae: call `new`

This just produces an error otherwise.

formulae: check for added formulae in `build_bottle?`

Fixes skipping bottle builds for PRs that add a formula and switch other
formulae to depend on t...

formulae_dependents: fix bottle notification

The `file` needs to be relative to the repository. I thought
`GitHub::Actions::Annotation` was s...

formulae_dependents: build more dependents from source

cmd/test-bot.rb: remove `--skip-unbottled-{arm,linux}`

Improve whitespace placement

This should make whitespace placement in the CI log a bit more logical.

Add tests for opportunistic linkage

This comes immediately after doing `brew linkage` on the same formula,
so we can just use `--cac...

This will just duplicate the error message we saw from the first linkage
test otherwise.

See discussion at Homebrew/brew#12454.

lib/step: enable annotations on Linux

lib/step: fix `os_string`

This is currently an empty string on Intel macOS.

formulae_dependents: traverse dep tree when linking dependencies

lib/step: improve GitHub Actions logging and annotations

We probably still want annotations even if `--verbose` is passed to
`test-bot`. Also, make sure ...

We duplicate some code in exchange, but a little duplication is probably
worth the reduced nesting.

1. Fix output grouping.
2. Don't join with `\n`. `@output.lines` doesn't strip each line, so
...

Co-authored-by: fn ⌃ ⌥ <[email protected]>

Co-authored-by: Bo Anderson <[email protected]>

Co-authored-by: fn ⌃ ⌥ <[email protected]>

Co-authored-by: Bo Anderson <[email protected]>

formulae: fix missing `=`

Oops

See discussion at #719.

lib/step: don't use .present on output.

See
https://github.com/Homebrew/homebrew-test-bot/pull/710#discussion_r752419047
and https://git...

On macOS, all formulae are possible `source_dependents`. On Linux, only
Linux-only formulae are ...

If the `--build-dependents-from-source` flag is passed, then all
dependents are built from sourc...

Display unbottled dependencies when skipping

Revert "formulae_dependents: prune build deps when linking test deps"

lib/step: improve GitHub Actions annotations.

- clarify that these are GitHub Actions annotations in method name
- don't duplicate the output ...

This still doesn't fix the overzealous linking of formulae,
unfortunately. See Homebrew/homebrew...

formulae_dependents: prune build deps when linking test deps

lib/step: skip annotations on Linux (for now)

The logic for `:all` bottles is now in the `#bottled?` method.

We also remove `no_older_version...

We also no longer need to handle the `:all` bottle case separately after
e41d68eb6ad35b01595597e...

We currently have a few places elsewhere that check for `:all` bottle
dependencies separately. W...

Using `GitHub::Actions::Annotation` causes errors in the Linux
container. See #712.

formulae_dependents: display annotations for missing bottles

This change checks for formulae that should be bottled but aren't. When
it finds one, it display...

lib/tests/formulae_dependents: link only installed formulae

We started trying to link recursive dependencies in #711. However, this
causes `test-bot` to att...

formula_dependents: ensure recursive dependencies are linked

lib/step: improve Actions annotations.

- make the method definition stricter
- allow passing through a title
- don't use `<details`> (i...

lib/step: more annotation format tweaks

This doesn't seem to render correctly at the moment.

lib/step: tweak annotation code/output.

- Use early `return`s to reduce indentation
- Use `<details>` to optionally display the full out...

Implement annotations for relevant errors

Skip online checks when there are more than five testing formulae

This makes things easier to understand for now, but requires
`named_args` and `command` to not c...

Closes Homebrew/homebrew-core#89265.

These have been replaced by the new annotations and ignore API.

This implements a way to produce annotations for all errors in a
`test-bot` run. It requires bak...

lib/test_formulae: fix undefined method error

This is missing a `#to_s` call.

lib/tests/formulae: demote more unbottled failures to warnings