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

[Player Model] Fix project errors #1934

Merged
merged 2 commits into from
Dec 7, 2023
Merged

Conversation

jkasten2
Copy link
Member

@jkasten2 jkasten2 commented Dec 7, 2023

Description

One Line Summary

Fix build errors the prevent building with Android Studio 2022.3.1 and running the tests at all.

Details

  • Fixed Android Studio 2022.3.1 build issue - Upgraded to Kotlin 1.5
  • Fixed unit tests - set minSdkVersion: 19 in tests (from 17).

Motivation

We want to allow building with modern Android Studio versions and we want to test to run again to prevent silently missing bugs.

Scope

Only fixing project building and test building issues in Player Model (v4).

Testing

Unit testing

Manual testing

I am able to build and run on Android Studio 2022.3.1 on Windows 11 23H2 with an Android 14 emulator after these fixes.

Affected code checklist

  • Notifications
    • Display
    • Open
    • Push Processing
    • Confirm Deliveries
  • Outcomes
  • Sessions
  • In-App Messaging
  • REST API requests
  • Public API changes

Checklist

Overview

  • I have filled out all REQUIRED sections above
  • PR does one thing
  • Any Public API changes are explained in the PR details and conform to existing APIs

Testing

  • I have included test coverage for these changes, or explained why they are not needed
  • All automated tests pass, or I explained why that is not possible
  • I have personally tested this on my device, or explained why that is not possible

Final pass

  • Code is as readable as possible.
  • I have reviewed this PR myself, ensuring it meets each checklist item

This change is Reviewable

In Android Studio 2022.3.1 it seems Kotlin 1.4 is no longer supported
the following error is shown when attempting to build:
java.lang.NoClassDefFoundError: Could not initialize class org.jetbrains.kotlin.com.intellij.pom.java.LanguageLevel
We bumped to a new FCM version which requires a min SDK version of 19,
it seems we only missed this in the tests.
@jkasten2 jkasten2 changed the title [Player model] Fix project errors [Player Model] Fix project errors Dec 7, 2023
Base automatically changed from player-model/fix-notification-permission-compile-error to main December 7, 2023 02:23
@jkasten2 jkasten2 merged commit 48860d2 into main Dec 7, 2023
1 of 2 checks passed
@jkasten2 jkasten2 deleted the player-model/fix-project-errors branch December 7, 2023 02:23
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.

2 participants