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

chore(android): update gradle tools and dependencies #12821

Merged
merged 4 commits into from May 27, 2021

Conversation

jquick-axway
Copy link
Contributor

@jquick-axway jquick-axway commented May 19, 2021

JIRA:

Summary:

  • TIMOB-28455 Updated Android gradle tool to version 4.2.1.
  • TIMOB-28457 Updated Kotlin language version to 1.5.0 for module builds.
  • TIMOB-28456 Replaced deprecated usage of jcenter() repo with mavenCentral().
  • Updated all of the Titanium SDK's library dependencies to the newest versions.

Hyperloop Test:

  1. Build and run hyperloop-examples for Android.
  2. Verify each window successfully displays its contents. Particularly the 3rd party libraries section.

Kotlin Test:

  1. appc ti sdk select 10.1.0
  2. appc new
  3. Select "Titanium Module".
  4. Select "Kotlin" language for the Android question.
  5. CD to the module's "android" folder.
  6. appc run -p android
  7. Verify it builds and runs successfully.

@jquick-axway jquick-axway added android improvement BREAKING CHANGES ⚠️ no tests backport 10_2_X when applied, PRs with this label will get an auto-generated backport to 10_2_X branch on merge labels May 19, 2021
@jquick-axway jquick-axway added this to the 10.1.0 milestone May 19, 2021
@build
Copy link
Contributor

build commented May 19, 2021

Fails
🚫 Tests have failed, see below for more information.
Messages
📖 ✊ The commits in this PR match our conventions! Feel free to Rebase and Merge this PR when ready.
📖 ❌ 6 tests have failed There are 6 tests failing and 974 skipped out of 16655 total tests.
📖

💾 Here's the generated SDK zipfile.

Tests:

ClassnameNameTimeError
android.emulator.Titanium.UI.NavigationWindowbasic open/close navigation (5.0.2)10.041
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (app.js)
android.emulator.Titanium.UI.View"after each" hook for "getOrCreateView() should always return a View" (5.0.2)12.664
Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (app.js)
ios.ipad.Titanium.Network.HTTPClient#timeoutForResource (14.4.0)0.001
Error: onload shouldn't fire. Resource request timeout should reach before transferring entire resource.
file:///Users/build/Library/Developer/CoreSimulator/Devices/8004FEFB-9C69-45DD-8F64-47B0C61BCF24/data/Containers/Bundle/Application/DC2A0D64-8268-4D6D-8568-CDD8C342625E/mocha.app/ti.network.httpclient.test.js:810:40
ios.ipad.Titanium.Network.HTTPClientresponseHeaders (14.4.0)60.001
Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (app.js)
ios.macos.Titanium.Network.HTTPClient#timeoutForResource (10.15.7)0.006
Error: onload shouldn't fire. Resource request timeout should reach before transferring entire resource.
file:///Users/build/jenkins/workspace/ium-sdk_titanium_mobile_PR-12821/tmp/mocha/build/iphone/build/Products/Debug-maccatalyst/mocha.app/Contents/Resources/ti.network.httpclient.test.js:810:40
ios.macos.Titanium.Network.HTTPClientresponseHeaders (10.15.7)60.126
Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (app.js)

Generated by 🚫 dangerJS against bc66ec8

@lokeshchdhry
Copy link
Contributor

FR Passed.
Android kotlin module builds & runs fine.
Hyperloop examples app works as expected.

@sgtcoolguy sgtcoolguy merged commit 284bf77 into tidev:master May 27, 2021
@build build removed the backport 10_2_X when applied, PRs with this label will get an auto-generated backport to 10_2_X branch on merge label May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants