Skip to content

Commit

Permalink
[tests][dotnet] Clear packages before building test apps (#11978)
Browse files Browse the repository at this point in the history
to make sure the latest locally build nuget are used

Also document `make strip-dotnet` as a time-saver to analyze the binary
assemblies produced for dotnet
  • Loading branch information
spouliot authored Jun 18, 2021
1 parent 91e47f2 commit 785bbbe
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions tests/dotnet/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ run-unit-tests:
all-local:: $(TARGETS)

compare compare-size: $(TARGETS)
rm -rf packages
cd size-comparison && git clean -xfdq
time $(MAKE) build-oldnet
time $(MAKE) build-dotnet
Expand Down
4 changes: 4 additions & 0 deletions tests/dotnet/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,3 +20,7 @@ Add this option inside the `Release|iPhone` configuration of `size-comparison/My

If you want to compare (trimmed) size you can manually call `mono-cil-strip`
on each assembly inside the app bundle.

`make strip-dotnet` will remove the IL from the dotnet app version.
However this is done after the code signature so it will not be possible
to deploy and execute the app afterward. Use for binary analysis only!

8 comments on commit 785bbbe

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

❌ [CI Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (no change)

Packages generated

View packages

Test results

1 tests failed, 220 tests passed.

Failed tests

  • introspection/watchOS 32-bits - simulator/Debug (watchOS 5.0): Failed

Pipeline on Agent XAMBOT-1036.BigSur'
[tests][dotnet] Clear packages before building test apps (#11978)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

🔥 Tests failed catastrophically on VSTS: device tests tvOS 🔥

Not enough free space in the host.

Pipeline on Agent
[tests][dotnet] Clear packages before building test apps (#11978)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

⚠️ Tests were not ran (VSTS: device tests tvOS). ⚠️

Results were skipped for this run due to provisioning problems Azure Devops. Please contact the bot administrator.

Pipeline on Agent
[tests][dotnet] Clear packages before building test apps (#11978)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

⚠️ Tests were not ran (VSTS: device tests iOS). ⚠️

Results were skipped for this run due to provisioning problems Azure Devops. Please contact the bot administrator.

Pipeline on Agent
[tests][dotnet] Clear packages before building test apps (#11978)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

✅ Tests passed on macOS Mac Catalina (10.15) ✅

Tests passed

All tests on macOS X Mac Catalina (10.15) passed.

Pipeline on Agent
[tests][dotnet] Clear packages before building test apps (#11978)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

✅ Tests passed on macOS Mac Mojave (10.14) ✅

Tests passed

All tests on macOS X Mac Mojave (10.14) passed.

Pipeline on Agent
[tests][dotnet] Clear packages before building test apps (#11978)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

✅ Tests passed on macOS Mac High Sierra (10.13) ✅

Tests passed

All tests on macOS X Mac High Sierra (10.13) passed.

Pipeline on Agent
[tests][dotnet] Clear packages before building test apps (#11978)

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Choose a reason for hiding this comment

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

⚠️ Tests were not ran (VSTS: device tests iOS32b). ⚠️

Results were skipped for this run due to provisioning problems Azure Devops. Please contact the bot administrator.

Pipeline on Agent
[tests][dotnet] Clear packages before building test apps (#11978)

Please sign in to comment.