From 45bce370b6c465adc7e2dd77ad3bd8b6929146b6 Mon Sep 17 00:00:00 2001 From: Yasith Vidanaarachchi Date: Wed, 22 Nov 2023 11:50:22 -0500 Subject: Update CI to run on pull requests instead of push (#82) Some of our CI tests are not triggering when there's a new pull request from outside of the repository. This should be fixed by changing the event it's triggered on to `pull_request` instead of `push` --- .github/workflows/PullRequestWorkflow.yaml | 108 +++++++++++++++++++++++++++++ .github/workflows/PushWorkflow.yaml | 108 ----------------------------- 2 files changed, 108 insertions(+), 108 deletions(-) create mode 100644 .github/workflows/PullRequestWorkflow.yaml delete mode 100644 .github/workflows/PushWorkflow.yaml diff --git a/.github/workflows/PullRequestWorkflow.yaml b/.github/workflows/PullRequestWorkflow.yaml new file mode 100644 index 0000000..ea32c21 --- /dev/null +++ b/.github/workflows/PullRequestWorkflow.yaml @@ -0,0 +1,108 @@ +name: Presubmit + +on: [pull_request] + +concurrency: + group: build-${{ github.ref }} + cancel-in-progress: true + +env: + JDK_VERSION: 17 + DISTRIBUTION: 'zulu' + +jobs: + build: + name: Build + runs-on: ubuntu-latest + timeout-minutes: 120 + steps: + - name: Checkout + uses: actions/checkout@v3 + + - name: Validate Gradle Wrapper + uses: gradle/wrapper-validation-action@v1 + + - name: Set up JDK + uses: actions/setup-java@v3 + with: + distribution: ${{ env.DISTRIBUTION }} + java-version: ${{ env.JDK_VERSION }} + cache: gradle + + - name: Setup Gradle + uses: gradle/gradle-build-action@v2 + + - name: Build all build type and flavor permutations + run: ./gradlew assemble --parallel --build-cache + + - name: Upload build outputs (APKs) + uses: actions/upload-artifact@v3 + with: + name: build-outputs + path: app/build/outputs + + - name: Upload build reports + if: always() + continue-on-error: true + uses: actions/upload-artifact@v3 + with: + name: build-reports + path: "*/build/reports" + + test: + name: Unit Tests + runs-on: ubuntu-latest + timeout-minutes: 120 + steps: + - name: Checkout + uses: actions/checkout@v3 + + - name: Validate Gradle Wrapper + uses: gradle/wrapper-validation-action@v1 + + - name: Set up JDK + uses: actions/setup-java@v3 + with: + distribution: ${{ env.DISTRIBUTION }} + java-version: ${{ env.JDK_VERSION }} + cache: gradle + + - name: Setup Gradle + uses: gradle/gradle-build-action@v2 + continue-on-error: true + + - name: Run local tests + run: ./gradlew test --parallel --build-cache + + - name: Upload test reports on failure + if: failure() + uses: actions/upload-artifact@v3 + with: + name: test-reports + path: "*/build/reports/tests" + + spotless: + name: Spotless Check + runs-on: ubuntu-latest + timeout-minutes: 60 + steps: + - name: Checkout + uses: actions/checkout@v3 + with: + fetch-depth: 0 + + - name: Validate Gradle Wrapper + uses: gradle/wrapper-validation-action@v1 + + - name: Set up JDK + uses: actions/setup-java@v3.9.0 + with: + distribution: ${{ env.DISTRIBUTION }} + java-version: ${{ env.JDK_VERSION }} + cache: gradle + + - name: Setup Gradle + uses: gradle/gradle-build-action@v2 + + - name: Spotless Check + run: ./gradlew spotlessCheck --init-script gradle/init.gradle.kts --parallel --build-cache diff --git a/.github/workflows/PushWorkflow.yaml b/.github/workflows/PushWorkflow.yaml deleted file mode 100644 index 06c8921..0000000 --- a/.github/workflows/PushWorkflow.yaml +++ /dev/null @@ -1,108 +0,0 @@ -name: Presubmit - -on: [push] - -concurrency: - group: build-${{ github.ref }} - cancel-in-progress: true - -env: - JDK_VERSION: 17 - DISTRIBUTION: 'zulu' - -jobs: - build: - name: Build - runs-on: ubuntu-latest - timeout-minutes: 120 - steps: - - name: Checkout - uses: actions/checkout@v3 - - - name: Validate Gradle Wrapper - uses: gradle/wrapper-validation-action@v1 - - - name: Set up JDK - uses: actions/setup-java@v3 - with: - distribution: ${{ env.DISTRIBUTION }} - java-version: ${{ env.JDK_VERSION }} - cache: gradle - - - name: Setup Gradle - uses: gradle/gradle-build-action@v2 - - - name: Build all build type and flavor permutations - run: ./gradlew assemble --parallel --build-cache - - - name: Upload build outputs (APKs) - uses: actions/upload-artifact@v3 - with: - name: build-outputs - path: app/build/outputs - - - name: Upload build reports - if: always() - continue-on-error: true - uses: actions/upload-artifact@v3 - with: - name: build-reports - path: "*/build/reports" - - test: - name: Unit Tests - runs-on: ubuntu-latest - timeout-minutes: 120 - steps: - - name: Checkout - uses: actions/checkout@v3 - - - name: Validate Gradle Wrapper - uses: gradle/wrapper-validation-action@v1 - - - name: Set up JDK - uses: actions/setup-java@v3 - with: - distribution: ${{ env.DISTRIBUTION }} - java-version: ${{ env.JDK_VERSION }} - cache: gradle - - - name: Setup Gradle - uses: gradle/gradle-build-action@v2 - continue-on-error: true - - - name: Run local tests - run: ./gradlew test --parallel --build-cache - - - name: Upload test reports on failure - if: failure() - uses: actions/upload-artifact@v3 - with: - name: test-reports - path: "*/build/reports/tests" - - spotless: - name: Spotless Check - runs-on: ubuntu-latest - timeout-minutes: 60 - steps: - - name: Checkout - uses: actions/checkout@v3 - with: - fetch-depth: 0 - - - name: Validate Gradle Wrapper - uses: gradle/wrapper-validation-action@v1 - - - name: Set up JDK - uses: actions/setup-java@v3.9.0 - with: - distribution: ${{ env.DISTRIBUTION }} - java-version: ${{ env.JDK_VERSION }} - cache: gradle - - - name: Setup Gradle - uses: gradle/gradle-build-action@v2 - - - name: Spotless Check - run: ./gradlew spotlessCheck --init-script gradle/init.gradle.kts --parallel --build-cache -- cgit v1.2.3 From 5997b5c2b41bf3cc9c2aa3c859e2ada2a945e12b Mon Sep 17 00:00:00 2001 From: mrxx0 Date: Tue, 28 Nov 2023 03:13:24 +0100 Subject: Fix reset of front facing camera settings after leaving screen settings (#75) Fixes google#69 Signed-off-by: mrxx0 --- .../com/google/jetpackcamera/settings/LocalSettingsRepository.kt | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/data/settings/src/main/java/com/google/jetpackcamera/settings/LocalSettingsRepository.kt b/data/settings/src/main/java/com/google/jetpackcamera/settings/LocalSettingsRepository.kt index eab14ae..fb03ba9 100644 --- a/data/settings/src/main/java/com/google/jetpackcamera/settings/LocalSettingsRepository.kt +++ b/data/settings/src/main/java/com/google/jetpackcamera/settings/LocalSettingsRepository.kt @@ -106,8 +106,13 @@ class LocalSettingsRepository @Inject constructor( // if a front or back lens is not present, the option to change // the direction of the camera should be disabled jcaSettings.updateData { currentSettings -> + val newLensFacing = if (currentSettings.defaultFrontCamera) { + frontLensAvailable + } else { + false + } currentSettings.toBuilder() - .setDefaultFrontCamera(frontLensAvailable) + .setDefaultFrontCamera(newLensFacing) .setFrontCameraAvailable(frontLensAvailable) .setBackCameraAvailable(backLensAvailable) .build() -- cgit v1.2.3 From e974bcaf5b55a29ad8c1ee504cdbbbcc4dacf367 Mon Sep 17 00:00:00 2001 From: Kimberly Crevecoeur Date: Tue, 28 Nov 2023 10:39:04 -0800 Subject: Foundation of benchmark integration to JCA (#80) --- .idea/androidTestResultsUserPreferences.xml | 26 ++++++++ .idea/gradle.xml | 4 +- app/build.gradle.kts | 9 +++ app/src/main/AndroidManifest.xml | 1 + benchmark/.gitignore | 1 + benchmark/build.gradle.kts | 75 ++++++++++++++++++++++ benchmark/src/main/AndroidManifest.xml | 17 +++++ .../main/java/com/example/benchmark/Permissions.kt | 29 +++++++++ .../java/com/example/benchmark/StartupBenchmark.kt | 70 ++++++++++++++++++++ build.gradle.kts | 1 + data/settings/consumer-rules.pro | 1 + settings.gradle.kts | 1 + 12 files changed, 234 insertions(+), 1 deletion(-) create mode 100644 benchmark/.gitignore create mode 100644 benchmark/build.gradle.kts create mode 100644 benchmark/src/main/AndroidManifest.xml create mode 100644 benchmark/src/main/java/com/example/benchmark/Permissions.kt create mode 100644 benchmark/src/main/java/com/example/benchmark/StartupBenchmark.kt diff --git a/.idea/androidTestResultsUserPreferences.xml b/.idea/androidTestResultsUserPreferences.xml index 6abdd98..1f01370 100644 --- a/.idea/androidTestResultsUserPreferences.xml +++ b/.idea/androidTestResultsUserPreferences.xml @@ -3,6 +3,19 @@