Skip to content

test: add android test lanes#943

Open
ovitrif wants to merge 8 commits into
feat/calculator-widget-v61from
test/ai-dev-tests-suite-setup
Open

test: add android test lanes#943
ovitrif wants to merge 8 commits into
feat/calculator-widget-v61from
test/ai-dev-tests-suite-setup

Conversation

@ovitrif
Copy link
Copy Markdown
Collaborator

@ovitrif ovitrif commented May 16, 2026

Related: #950

Stacked on #942.

The Claude auto-review workflow change was split into #949 so this PR only carries Android test lane setup.

Description

  • Adds marker annotations for Compose UI, device storage, core service, and broad device integration instrumented tests to group existing tests.
  • Dynamically registers devDebug Android test lane tasks from annotation files in to.bitkit.test.annotations, so adding a new annotation class automatically adds the matching lane task.
  • Classifies existing instrumented tests into the new lanes without changing test behavior.
  • Extends the manual UI Tests workflow with a suites input. Use all for the full Android test suite, or comma-separated simple annotation names such as ComposeUi,CoreServiceIntegration.

Preview

N/A - test workflow enhancements.

QA Notes

Manual Tests

  • 1. GitHub Actions → manually run UI Tests with suites=ComposeUi: workflow accepts the targeted suite input.

Automated Checks

  • Local: ./gradlew compileDevDebugKotlin testDevDebugUnitTest compileDevDebugAndroidTestKotlin detekt
  • Local: ./gradlew tasks --all | rg "connectedDevDebug.*AndroidTest"
  • Local dry run: ./gradlew connectedDevDebugAndroidTest -PbitkitAndroidTestAnnotation=ComposeUi -m
  • Local dry run: ./gradlew connectedDevDebugComposeUiAndroidTest -m
  • (after merge) Verify the manual UI Tests workflow exposes the new suites input by running one targeted suite.

@ovitrif ovitrif added this to the 2.3.0 milestone May 16, 2026
@ovitrif ovitrif self-assigned this May 16, 2026
@ovitrif ovitrif changed the title test: add android test lanes test: add android test lanes + disable claude auto review May 16, 2026
@ovitrif ovitrif mentioned this pull request May 16, 2026
27 tasks
@ovitrif ovitrif changed the title test: add android test lanes + disable claude auto review test: add android test lanes + disable claude reviews May 16, 2026
@ovitrif ovitrif marked this pull request as ready for review May 18, 2026 00:04
@ovitrif ovitrif requested review from ben-kaufman and jvsena42 May 18, 2026 00:04
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0b1e6d0c59

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/src/androidTest/java/to/bitkit/test/annotations/DeviceUiIntegration.kt Outdated
@ovitrif ovitrif force-pushed the feat/calculator-widget-v61 branch from 2183e5d to 5178b8d Compare May 18, 2026 01:13
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from 0b1e6d0 to c167ac7 Compare May 18, 2026 01:17
@ovitrif ovitrif force-pushed the feat/calculator-widget-v61 branch from 5178b8d to 635084f Compare May 18, 2026 01:24
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from c167ac7 to e8f05e7 Compare May 18, 2026 01:25
@ovitrif ovitrif force-pushed the feat/calculator-widget-v61 branch from 635084f to 286c5d5 Compare May 18, 2026 01:36
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from e8f05e7 to d4e542d Compare May 18, 2026 01:37
@ovitrif ovitrif force-pushed the feat/calculator-widget-v61 branch from 286c5d5 to 4d16994 Compare May 18, 2026 08:02
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from d4e542d to 3fa8959 Compare May 18, 2026 08:04
@ovitrif ovitrif force-pushed the feat/calculator-widget-v61 branch from 4d16994 to 85fcf98 Compare May 18, 2026 08:13
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch 2 times, most recently from 3c440c9 to 966c034 Compare May 18, 2026 10:29
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from 966c034 to 8f0dd84 Compare May 18, 2026 13:34
@ovitrif ovitrif changed the title test: add android test lanes + disable claude reviews test: add android test lanes May 18, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8f0dd84e8e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/build.gradle.kts Outdated
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from 8f0dd84 to 74056bd Compare May 18, 2026 15:24
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 74056bd489

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ui-tests.yml Outdated
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from 74056bd to 3b82d78 Compare May 18, 2026 15:29
@ovitrif ovitrif force-pushed the test/ai-dev-tests-suite-setup branch from 3b82d78 to 618af1b Compare May 18, 2026 15:36
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 618af1b1de

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/build.gradle.kts Outdated
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.

1 participant