Skip to content

Fix occasional CI failures on the iPhone simulator#1287

Merged
justinseanmartin merged 1 commit into
kif-framework:masterfrom
Fitzmaz:fix-ci-failures
Jul 14, 2023
Merged

Fix occasional CI failures on the iPhone simulator#1287
justinseanmartin merged 1 commit into
kif-framework:masterfrom
Fitzmaz:fix-ci-failures

Conversation

@Fitzmaz

@Fitzmaz Fitzmaz commented Jul 12, 2023

Copy link
Copy Markdown
Contributor

In iOS, the edit menu is designed to display two commands and a "Forward" or "Back" page button when there are more than two commands available.

If the pasteboard already contains a string, the edit menu will show "Paste" and "Select" commands. The "Select All" command can be accessed on the next page.

Executing a case that taps the "Cut" command will result in the failure of all subsequent cases that require tapping the "Select All" command.

One solution to this is to clear the pasteboard before the "Select All" cases.

@Fitzmaz

Fitzmaz commented Jul 13, 2023

Copy link
Copy Markdown
Contributor Author

Another failure in AccessibilityIdentifierTests.testClearingAndEnteringTextIntoViewWithAccessibilityLabel:

AccessibilityIdentifierTests
  testClearingAndEnteringTextIntoViewWithAccessibilityLabel, The step timed out after 10.00 seconds: Could not find element with BLOCKPREDICATE(0x6000030500f0)
  /Users/runner/work/KIF/KIF/KIF Tests/AccessibilityIdentifierTests.m:73
{
    [tester clearTextFromAndThenEnterText:@"Yo" intoViewWithAccessibilityIdentifier:@"idGreeting"];
}

  testClearingAndEnteringTextIntoViewWithAccessibilityLabel, The step timed out after 10.00 seconds: Found an accessibility element with the label "Test Suite", but not with the traits "1"
  /Users/runner/work/KIF/KIF/KIF Tests/AccessibilityIdentifierTests.m:137
{
    [tester tapViewWithAccessibilityLabel:@"Test Suite" traits:UIAccessibilityTraitButton];
}

Based on the error messages, it appears that the runner failed to enter the TapViewController and got stuck at the TestSuiteViewController. Not sure about the cause of this though.

@justinseanmartin

Copy link
Copy Markdown
Contributor

I think I've got a fix for the AccessibilityIdentifierTests flakiness in PR #1288. I only reproduced it locally on my machine once, but I'm going to run it against CI and see if it is more reliably green.

@justinseanmartin

Copy link
Copy Markdown
Contributor

If we keep having trouble getting this to go green, you might rebase latest master now that #1288 is merged.

@justinseanmartin justinseanmartin merged commit 7a06351 into kif-framework:master Jul 14, 2023
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