feat: Adds tasks notification #394
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
| distributionBase=GRADLE_USER_HOME | ||
| distributionPath=wrapper/dists | ||
| distributionUrl=https\://services.gradle.org/distributions/gradle-8.11.1-bin.zip | ||
| zipStoreBase=GRADLE_USER_HOME | ||
| zipStorePath=wrapper/dists |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,109 @@ | ||
| package org.medicmobile.webapp.mobile; | ||
|
|
||
| import static androidx.test.espresso.Espresso.onView; | ||
| import static androidx.test.espresso.action.ViewActions.click; | ||
| import static androidx.test.espresso.action.ViewActions.closeSoftKeyboard; | ||
| import static androidx.test.espresso.action.ViewActions.replaceText; | ||
| import static androidx.test.espresso.matcher.ViewMatchers.withId; | ||
| import static androidx.test.espresso.matcher.ViewMatchers.withText; | ||
| import static androidx.test.espresso.web.sugar.Web.onWebView; | ||
| import static androidx.test.espresso.web.webdriver.DriverAtoms.clearElement; | ||
| import static androidx.test.espresso.web.webdriver.DriverAtoms.findElement; | ||
| import static androidx.test.espresso.web.webdriver.DriverAtoms.webClick; | ||
| import static junit.framework.TestCase.assertEquals; | ||
|
|
||
| import android.Manifest; | ||
| import android.annotation.SuppressLint; | ||
| import android.content.Context; | ||
|
|
||
| import androidx.test.core.app.ActivityScenario; | ||
| import androidx.test.core.app.ApplicationProvider; | ||
| import androidx.test.espresso.ViewInteraction; | ||
| import androidx.test.espresso.web.webdriver.DriverAtoms; | ||
| import androidx.test.espresso.web.webdriver.Locator; | ||
| import androidx.test.ext.junit.rules.ActivityScenarioRule; | ||
| import androidx.test.ext.junit.runners.AndroidJUnit4; | ||
| import androidx.test.rule.GrantPermissionRule; | ||
| import androidx.work.WorkInfo; | ||
| import androidx.work.WorkManager; | ||
|
|
||
| import org.junit.After; | ||
| import org.junit.Ignore; | ||
| import org.junit.Rule; | ||
| import org.junit.Test; | ||
| import org.junit.runner.RunWith; | ||
|
|
||
| import java.util.List; | ||
| import java.util.concurrent.ExecutionException; | ||
|
|
||
| @RunWith(AndroidJUnit4.class) | ||
| public class NotificationTest { | ||
| private Context context; | ||
| private static final String TEST_SERVER = ""; | ||
| private static final String TEST_USERNAME = ""; | ||
| private static final String TEST_PASSWORD = ""; | ||
|
|
||
| @Rule | ||
| public ActivityScenarioRule<SettingsDialogActivity> mActivityTestRule = | ||
| new ActivityScenarioRule<>(SettingsDialogActivity.class); | ||
| @Rule | ||
| public GrantPermissionRule permissionRule = | ||
| GrantPermissionRule.grant( | ||
| Manifest.permission.POST_NOTIFICATIONS | ||
| ); | ||
|
|
||
| @After | ||
| public void cleanServerRepo() { | ||
| context.getSharedPreferences("ServerRepo", Context.MODE_PRIVATE) | ||
| .edit().clear().apply(); | ||
| } | ||
|
|
||
| @Ignore("TODO: add TEST_SERVER") | ||
| @SuppressLint("CheckResult") | ||
| @Test | ||
| public void startsNotificationWorkers() throws InterruptedException, ExecutionException { | ||
| context = ApplicationProvider.getApplicationContext(); | ||
| onView(withText("Custom")).perform(click()); | ||
| ViewInteraction textAppUrl = onView(withId(R.id.txtAppUrl)); | ||
| textAppUrl.perform(replaceText(TEST_SERVER), closeSoftKeyboard()); | ||
| onView(withId(R.id.btnSaveSettings)).perform(click()); | ||
|
|
||
| Thread.sleep(4000); | ||
|
|
||
| assertEquals("expect no worker running at login page", 0, getRunningWorkers()); | ||
|
|
||
| onWebView() | ||
| .withNoTimeout() | ||
| .withElement(findElement(Locator.ID, "user")) | ||
| .perform(clearElement()) | ||
| .perform(DriverAtoms.webKeys(TEST_USERNAME)) //to be created first | ||
| .withElement(findElement(Locator.ID, "password")) | ||
| .perform(clearElement()) | ||
| .perform(DriverAtoms.webKeys(TEST_PASSWORD)) | ||
| .withElement(findElement(Locator.ID, "login")) | ||
| .perform(webClick()); | ||
|
|
||
| Thread.sleep(10 * 1000); | ||
|
|
||
| ActivityScenario<EmbeddedBrowserActivity> embeddedScenario = ActivityScenario.launch(EmbeddedBrowserActivity.class); | ||
| assertEquals("expect no work manager while app is in foreground", 0, getRunningWorkers()); | ||
|
|
||
| //close app | ||
| embeddedScenario.onActivity(EmbeddedBrowserActivity::onBackPressed); | ||
| Thread.sleep(1000); | ||
| assertEquals("expect work manager enqueued while app is in background", 1, getRunningWorkers()); | ||
|
|
||
| //reopen app | ||
| embeddedScenario = ActivityScenario.launch(EmbeddedBrowserActivity.class); | ||
| assertEquals("no work manager on app restart", 0, getRunningWorkers()); | ||
| } | ||
|
|
||
| private long getRunningWorkers() throws ExecutionException, InterruptedException { | ||
| List<WorkInfo> workInfos = WorkManager.getInstance(context) | ||
| .getWorkInfosByTag(NotificationWorker.NOTIFICATION_WORK_REQUEST_TAG) | ||
| .get(); | ||
| return workInfos.stream() | ||
| .filter(info -> info.getState() == WorkInfo.State.ENQUEUED || info.getState() == WorkInfo.State.RUNNING) | ||
| .count(); | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,6 +13,7 @@ | |
| <uses-feature android:name="android.hardware.location.network" android:required="false"/> | ||
|
|
||
| <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/> | ||
| <uses-permission android:name="android.permission.POST_NOTIFICATIONS"/> | ||
| <uses-permission android:name="android.permission.INTERNET"/> | ||
| <uses-permission android:name="android.permission.VIBRATE"/> | ||
| <uses-permission android:name="com.android.launcher.permission.INSTALL_SHORTCUT"/> | ||
|
|
@@ -50,7 +51,8 @@ | |
| <action android:name="android.intent.action.VIEW" /> | ||
| <category android:name="android.intent.category.DEFAULT" /> | ||
| <category android:name="android.intent.category.BROWSABLE" /> | ||
| <data android:scheme="@string/scheme" android:host="@string/app_host" android:pathPattern=".*" | ||
| tools:ignore="AppLinkUrlError" /> | ||
| </intent-filter> | ||
| </activity> | ||
| <activity android:name="ConnectionErrorActivity" | ||
|
|
@@ -92,6 +94,10 @@ | |
| android:resource="@xml/file_paths"> | ||
| </meta-data> | ||
| </provider> | ||
| <service android:name="androidx.work.impl.background.systemjob.SystemJobService" | ||
| android:permission="android.permission.BIND_JOB_SERVICE" | ||
| android:exported="true" | ||
| tools:targetApi="23" /> | ||
|
Comment on lines
+97
to
+100
Contributor
There was a problem hiding this comment. What is the purpose of defining this service here? I tried removing it and it seems like my notifications still get generated as expected... 🤔
Contributor
Author
There was a problem hiding this comment. its just for linting
Contributor
There was a problem hiding this comment. Ahh, well I think I was just re-building/running the app when I tried removing it, so maybe there is a deeper linting that will fail if I tried doing the full apk assemble....
Contributor
There was a problem hiding this comment. @jonathanbataire Did you ever figure out if this was required?
Contributor
Author
There was a problem hiding this comment. yeah im able to build APKs but there are linting errors in IDE because of 
|
||
| </application> | ||
|
|
||
| <queries> | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignoring this test until a test server with these changes medic/cht-core#9987 is available cc @lorerod @jkuester