diff --git a/imagepipeline-base/src/test/java/com/facebook/imageformat/ImageFormatCheckerTest.kt b/imagepipeline-base/src/test/java/com/facebook/imageformat/ImageFormatCheckerTest.kt index efc637c82a..ea3509f540 100644 --- a/imagepipeline-base/src/test/java/com/facebook/imageformat/ImageFormatCheckerTest.kt +++ b/imagepipeline-base/src/test/java/com/facebook/imageformat/ImageFormatCheckerTest.kt @@ -70,6 +70,27 @@ class ImageFormatCheckerTest constructor() { singleImageTypeTest(getName("heifs/1.heif"), DefaultImageFormats.HEIF) } + @Test + fun testXmlVectorDrawable() { + singleImageTypeTest( + getName("xmls/compiled/vector_drawable.xml"), DefaultImageFormats.BINARY_XML) + } + + @Test + fun testXmlLayerListDrawable() { + singleImageTypeTest(getName("xmls/compiled/layer_list.xml"), DefaultImageFormats.BINARY_XML) + } + + @Test + fun testXmlLevelListDrawable() { + singleImageTypeTest(getName("xmls/compiled/level_list.xml"), DefaultImageFormats.BINARY_XML) + } + + @Test + fun testXmlStateListDrawable() { + singleImageTypeTest(getName("xmls/compiled/state_list.xml"), DefaultImageFormats.BINARY_XML) + } + private fun singleImageTypeTest(resourceNames: List, expectedImageType: ImageFormat) { for (name: String in resourceNames) { val resourceStream = getResourceStream(name) diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/AndroidManifest.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/AndroidManifest.xml new file mode 100644 index 0000000000..2cfa7b7e5f --- /dev/null +++ b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/AndroidManifest.xml @@ -0,0 +1,5 @@ + + + diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/README.md b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/README.md new file mode 100644 index 0000000000..ed62941db3 --- /dev/null +++ b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/README.md @@ -0,0 +1,14 @@ +# XMLs + +## Context + +Android compiles raw XML to binary XML during app build as layout inflation at runtime does not support parsing raw XML files. As a result, Fresco only supports loading binary XML files since it relies on the Android to perform layout inflation. Therefore, we need to mock the build step that performs this compilation with the Android packaging tool. This directory houses raw, uncompiled files and their compiled counterparts in respective directories so we can test against them. + +## How to add/update assets + +These instructions are for POSIX devices. If you are on Windows, you will have to manually run these steps or use Android Studio to generate an APK + +1. Install Android's command-line tools by [following these instructions](https://developer.android.com/tools). Make sure you have `$ANDROID_HOME` set in your PATH by running `echo $ANDROID_HOME` +1. Install the latest packages of Android's build tools and platform. For example, you can run `sdkmanager "build-tools;34.0.0"` or `sdkmanager "platforms;android-33"`. [More instructions for sdkmanager can be found here](https://developer.android.com/tools/sdkmanager) +1. Run `./convert.sh` to automatically compile and extract resources into the `compiled` folder. It will use the latest available version of your build-tools and platforms. +1. That's it! diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/layer_list.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/layer_list.xml new file mode 100644 index 0000000000..e50b6cdff8 Binary files /dev/null and b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/layer_list.xml differ diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/level_list.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/level_list.xml new file mode 100644 index 0000000000..8a7c9bd617 Binary files /dev/null and b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/level_list.xml differ diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/state_list.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/state_list.xml new file mode 100644 index 0000000000..58bb851b20 Binary files /dev/null and b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/state_list.xml differ diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/vector_drawable.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/vector_drawable.xml new file mode 100644 index 0000000000..9bf2eebfd6 Binary files /dev/null and b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/compiled/vector_drawable.xml differ diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/convert.sh b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/convert.sh new file mode 100755 index 0000000000..a888ed4062 --- /dev/null +++ b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/convert.sh @@ -0,0 +1,51 @@ +#!/bin/sh +# Copyright (c) Meta Platforms, Inc. and affiliates. +# +# This source code is licensed under the MIT license found in the +# LICENSE file in the root directory of this source tree. + +if [ -z "$ANDROID_HOME" ]; then + echo "Environment variable ANDROID_HOME is not set" + exit 1 +fi + +SCRIPT_DIR="$(readlink -f "$(dirname "$0")")" +cd "$SCRIPT_DIR" || exit + +# Use the latest available AAPT binary +BUILD_TOOLS_DIR="$ANDROID_HOME/build-tools" +LATEST_TOOLS_DIR=$(find "$BUILD_TOOLS_DIR" -maxdepth 1 -type d -print | sort -rn --key=4.1 | head -1) + +if [ ! -d "$LATEST_TOOLS_DIR" ]; then + echo "Could not find build tools in $BUILD_TOOLS_DIR" + exit 1 +fi + +# Use the latest available Android version +ANDROID_PLATFORMS_DIR="$ANDROID_HOME/platforms" +LATEST_PLATFORM_DIR=$(find "$ANDROID_PLATFORMS_DIR" -maxdepth 1 -type d -print | sort -rn --key=4.1 | head -1) + +if [ ! -d "$LATEST_PLATFORM_DIR" ]; then + echo "Could not find any platforms in $ANDROID_PLATFORMS_DIR" + exit 1 +fi + +# Define a temporary directory for the APK +TMP_DIR=$(mktemp -d) +APK_OUTPUT="$TMP_DIR/app.apk" + +# Build an APK with our raw resources +"$LATEST_TOOLS_DIR/aapt" package -f -m -M AndroidManifest.xml -S raw -0 "" -I "$LATEST_PLATFORM_DIR/android.jar" -F "$APK_OUTPUT" || exit 1 + +# Unzip all APK artifacts +cd "$TMP_DIR" || exit +unzip -q "app.apk" + +# Copy all compiled resources from the base drawable folder +cd "$SCRIPT_DIR" || exit +rm -rf compiled +mkdir compiled +cp -R "$TMP_DIR/res/drawable/" ./compiled + +# Remove the temporary folder +rm -rf "$TMP_DIR" diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/layer_list.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/layer_list.xml new file mode 100644 index 0000000000..387e8f1903 --- /dev/null +++ b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/layer_list.xml @@ -0,0 +1,15 @@ + + + + + diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/level_list.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/level_list.xml new file mode 100644 index 0000000000..8a658c9952 --- /dev/null +++ b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/level_list.xml @@ -0,0 +1,12 @@ + + + + + diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/state_list.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/state_list.xml new file mode 100644 index 0000000000..159e77f8fe --- /dev/null +++ b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/state_list.xml @@ -0,0 +1,5 @@ + + + + + diff --git a/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/vector_drawable.xml b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/vector_drawable.xml new file mode 100644 index 0000000000..b84f8093b6 --- /dev/null +++ b/imagepipeline-base/src/test/resources/com/facebook/imageformat/xmls/raw/drawable/vector_drawable.xml @@ -0,0 +1,13 @@ + + + +