Skip to content

ANDROID-15807 Add @IgnoreLoggerazzi annotation to ignore a test#7

Merged
jeslat merged 2 commits intomainfrom
ANDROID-15807-ignore-loggerazzi
Feb 27, 2025
Merged

ANDROID-15807 Add @IgnoreLoggerazzi annotation to ignore a test#7
jeslat merged 2 commits intomainfrom
ANDROID-15807-ignore-loggerazzi

Conversation

@jeslat
Copy link
Contributor

@jeslat jeslat commented Feb 27, 2025

🎟️ Jira ticket

ANDROID-15807

🥅 What's the goal?

Add @IgnoreLoggerazzi annotation to be able to ignore a test.

🚧 How do we do it?

  • Add @IgnoreLoggerazzi annotation.
  • Check the annotation in GenericLoggerazziRule and don't verify the test if it exists.

📘 Documentation changes?

  • REAME updated

🧪 How can I test this?

  • A new example test has been added to check that the test is ignored and it doesn't fail.

@jeslat jeslat requested review from a team, dpastor and jeprubio and removed request for a team February 27, 2025 14:07
Copy link
Contributor

@jeprubio jeprubio left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link

@dpastor dpastor left a comment

Choose a reason for hiding this comment

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

Nice!

@jeslat jeslat merged commit 12b4e03 into main Feb 27, 2025
1 check passed
@jeslat jeslat deleted the ANDROID-15807-ignore-loggerazzi branch February 27, 2025 15:08
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.

3 participants