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

Fix iOS log exports #23334

Merged
merged 8 commits into from
Jul 21, 2024
Merged

Fix iOS log exports #23334

merged 8 commits into from
Jul 21, 2024

Conversation

PureWeen
Copy link
Member

@PureWeen PureWeen commented Jun 28, 2024

Description of Change

At one point we added code to collect all the iOS console logs when running tests. AFAICT none of those added steps successfully gather the logs during test runs. Because the iOS devices are created and then destroyed within the scope of the test run, the device logs no longer exist after the run has finished. This PR should now grab only the logs we care about.

@PureWeen PureWeen added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Jun 28, 2024
@PureWeen
Copy link
Member Author

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member Author

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member Author

PureWeen commented Jul 2, 2024

/rebase

@PureWeen PureWeen force-pushed the fix_device_logging branch from 5b9be05 to 19c910a Compare July 2, 2024 16:51
@PureWeen PureWeen changed the title Export logs before device is deleted Set device tests to release mode and export logs before device is deleted Jul 2, 2024
@PureWeen PureWeen force-pushed the fix_device_logging branch from 19c910a to 0354908 Compare July 5, 2024 19:44
@PureWeen PureWeen changed the base branch from net9.0 to main July 5, 2024 19:44
@PureWeen PureWeen changed the title Set device tests to release mode and export logs before device is deleted Fix iOS log exports Jul 5, 2024
@PureWeen PureWeen marked this pull request as ready for review July 5, 2024 21:53
@PureWeen PureWeen requested a review from a team as a code owner July 5, 2024 21:53
@PureWeen PureWeen requested review from rmarinho, jsuarezruiz and mattleibow and removed request for rmarinho and jsuarezruiz July 5, 2024 21:53
rolfbjarne
rolfbjarne previously approved these changes Jul 8, 2024
@mattleibow
Copy link
Member

/rebase

@PureWeen
Copy link
Member Author

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen PureWeen force-pushed the fix_device_logging branch from 13a3ef1 to 2b0dbbe Compare July 20, 2024 20:09
PureWeen added 2 commits July 21, 2024 11:11

Verified

This commit was signed with the committer’s verified signature.
andrzej-stencel Andrzej Stencel
@PureWeen PureWeen merged commit 9251b38 into main Jul 21, 2024
95 of 97 checks passed
@PureWeen PureWeen deleted the fix_device_logging branch July 21, 2024 22:21
pictos pushed a commit to pictos/maui that referenced this pull request Jul 22, 2024
[Windows] Fix CollectionView DisconnectHandler SelectionMode Crash (dotnet#23726)

* Reorder events to prevent disconnect issues

* Add test

---------

Co-authored-by: Mike Corsaro <mikecorsaro@microsoft.com>

Use UIView.Window instead of the global window (dotnet#23693)

[Windows] Subscribe pointer events only when needed (dotnet#23515)

Fix iOS log exports (dotnet#23334)

* Fix ios Device Logging

* - fix up device test logging a bit more

* Update ui-tests-steps.yml

* Update ui-tests-steps.yml

* Update ios.cake

* Update ui-tests-steps.yml

* Update maui-templates.yml

* Update maui-templates.yml

remove code from oldFragment

add new shiny DialogFragment

refactoring code to find and dismiss DialogFragment

code cleanup

delete ModalContainer to use only ModalFragment

handle animation and add a map between page and dialogFragment

We've back button enabled!

After dismissing several demons summoned using obscure Android APIs, I was able to deal with the BackButtonPressed event

add modal animations as anim.xml files

using cleanup

remowork PopModalPlatformAsync to work with dialogFragment

remove tag

final adjustments on DialogFragment

change the ShowNow for Show to fix the issue

Wait for animation to complete

change local functions order

fix build

create window hooks for android (like iOS)

clean up ModalFragment fields

change Dictionary to ConditionalWeakTable

clean up event animation

refactor on Null notation

remove comments

- adjust back button

- different back button

code style

remove unused prop.

fix DontPushModalPagesWhenWindowIsDeactivated DeviceTest

completes the task

return back the way how modalManager handles android modals

normilize animation duration

Co-authored-by: Shane Neuville <shane94@hotmail.com>

remove focusability code

change how fragments are looked-up

code style
@PureWeen
Copy link
Member Author

/backport to net9.0

Copy link
Contributor

Started backporting to net9.0: https://github.com/dotnet/maui/actions/runs/10062515188

@samhouts samhouts added fixed-in-net9.0-nightly This may be available in a nightly release! fixed-in-8.0.80 and removed fixed-in-net9.0-nightly This may be available in a nightly release! labels Aug 2, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions fixed-in-8.0.80 fixed-in-net9.0-nightly This may be available in a nightly release!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants