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

Change patrol output #2387

Merged
merged 47 commits into from
Nov 8, 2024
Merged

Change patrol output #2387

merged 47 commits into from
Nov 8, 2024

Conversation

pdenert
Copy link
Collaborator

@pdenert pdenert commented Oct 30, 2024

This PR completely changes the current output when firing patrol tests. It also adds two new flags:

  • showTestSteps,
  • showFlutterLogs.

To reach final effect, we need to handle exceptions during test, but this require #2362.
It's also adding new package: patrol_log, that need setup of the pubspec, changelog etc.

Preview of the new output on the video:

Nagranie.z.ekranu.2024-10-30.o.14.59.55.mov

Copy link

docs-page bot commented Oct 30, 2024

To view this pull requests documentation preview, visit the following URL:

docs.page/leancodepl/patrol~2387

Documentation is deployed and generated using docs.page.

@github-actions github-actions bot added package: patrol Related to the patrol package (native automation, test bundling) package: patrol_cli Related to the patrol_cli package labels Oct 30, 2024
@pdenert pdenert force-pushed the change-patrol-output branch from 7107a4d to 4c67ba8 Compare October 30, 2024 16:32
packages/patrol_log/pubspec.yaml Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/test_entry.dart Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/test_entry.dart Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/step_entry.dart Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/entry.dart Outdated Show resolved Hide resolved
packages/patrol/lib/src/native/native_automator2.dart Outdated Show resolved Hide resolved
packages/patrol/lib/src/native/native_automator.dart Outdated Show resolved Hide resolved
@github-actions github-actions bot added the package: adb Related to the adb package label Nov 6, 2024
@pdenert pdenert requested a review from piotruela November 6, 2024 16:12
Copy link
Collaborator

@piotruela piotruela left a comment

Choose a reason for hiding this comment

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

Looking good. Just a few nitpick

packages/patrol_log/lib/src/patrol_log_reader.dart Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/entry.dart Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/patrol_log_reader.dart Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/patrol_single_test_entry.dart Outdated Show resolved Hide resolved
packages/patrol_log/lib/src/step_entry.dart Outdated Show resolved Hide resolved
@pdenert pdenert force-pushed the change-patrol-output branch from 7191fd0 to 647be21 Compare November 8, 2024 16:35
@pdenert pdenert force-pushed the change-patrol-output branch from 647be21 to 5b644d1 Compare November 8, 2024 17:17
@pdenert pdenert merged commit 3a6854a into master Nov 8, 2024
15 of 18 checks passed
@pdenert pdenert deleted the change-patrol-output branch November 8, 2024 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: adb Related to the adb package package: patrol_cli Related to the patrol_cli package package: patrol Related to the patrol package (native automation, test bundling)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants