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

Refactor MTE-1776 [v120] CopiedLinksTests and SettingsTests with TR #16930

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

@dragosb01 dragosb01 requested a review from isabelrios October 19, 2023 12:08
@dragosb01 dragosb01 requested review from a team as code owners October 19, 2023 12:08
@mobiletest-ci-bot
Copy link

mobiletest-ci-bot commented Oct 19, 2023

Messages
📖 Edited 9 files
📖 Created 1 files

Generated by 🚫 Danger Swift against 9461548

@@ -2,37 +2,12 @@
// License, v. 2.0. If a copy of the MPL was not distributed with this
// file, You can obtain one at http://mozilla.org/MPL/2.0/

import XCTest
import Foundation
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: is this on purpose? I think we could remove this test suite... the test is disabled and with the new way it works (opening the app 5 times before this message is shown) I'm not sure it will be enabled back soon

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is an area that is not that easy to be automated and i agree i dont think it will be enabled soon. I will remove it.

@isabelrios isabelrios merged commit 6904307 into main Oct 23, 2023
9 of 11 checks passed
@isabelrios isabelrios deleted the dbigu/MTE-1776 branch October 23, 2023 15:27
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