Skip to content

Commit 1a92cdb

Browse files
cipolleschikelset
authored andcommitted
Add Tests in CircleCI to check dynamic frameworks with the old arch (#36003)
Summary: Pull Request resolved: #36003 This diff adds 4 tests in CircleCI to make sure we don't regress in the support of Dynamic Frameworks for the old architecture. ## Changelog [iOS][Fixed] - Add CircleCI tests for dynamic frameworks with the Old Architecture. Reviewed By: cortinico Differential Revision: D42829895 fbshipit-source-id: 5669be45d4f55161a11a6ece161b2a2aa384a644
1 parent 88196cd commit 1a92cdb

File tree

1 file changed

+64
-2
lines changed

1 file changed

+64
-2
lines changed

.circleci/config.yml

Lines changed: 64 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -876,7 +876,7 @@ jobs:
876876
default: "StaticLibraries"
877877
description: Which kind of option we want to use for `use_frameworks!`
878878
type: enum
879-
enum: ["StaticLibraries", "StaticFrameworks"] #TODO: Add "DynamicFrameworks"
879+
enum: ["StaticLibraries", "StaticFrameworks", "DynamicFrameworks"]
880880
environment:
881881
- PROJECT_NAME: "iOSTemplateProject"
882882
- HERMES_WS_DIR: *hermes_workspace_root
@@ -929,6 +929,8 @@ jobs:
929929
930930
if [[ << parameters.use_frameworks >> == "StaticFrameworks" ]]; then
931931
export USE_FRAMEWORKS=static
932+
elif [[ << parameters.use_frameworks >> == "DynamicFrameworks" ]]; then
933+
export USE_FRAMEWORKS=dynamic
932934
fi
933935
934936
bundle exec pod install
@@ -1627,7 +1629,7 @@ workflows:
16271629
flavor: ["Debug", "Release"]
16281630
jsengine: ["Hermes", "JSC"]
16291631
flipper: ["WithFlipper", "WithoutFlipper"]
1630-
use_frameworks: [ "StaticLibraries", "StaticFrameworks" ] #TODO: make it works with DynamicFrameworks
1632+
use_frameworks: ["StaticLibraries", "StaticFrameworks", "DynamicFrameworks"]
16311633
exclude:
16321634
- architecture: "NewArch"
16331635
flavor: "Release"
@@ -1639,11 +1641,21 @@ workflows:
16391641
jsengine: "Hermes"
16401642
flipper: "WithFlipper"
16411643
use_frameworks: "StaticFrameworks"
1644+
- architecture: "NewArch"
1645+
flavor: "Release"
1646+
jsengine: "Hermes"
1647+
flipper: "WithFlipper"
1648+
use_frameworks: "DynamicFrameworks"
16421649
- architecture: "NewArch"
16431650
flavor: "Release"
16441651
jsengine: "Hermes"
16451652
flipper: "WithoutFlipper"
16461653
use_frameworks: "StaticFrameworks"
1654+
- architecture: "NewArch"
1655+
flavor: "Release"
1656+
jsengine: "Hermes"
1657+
flipper: "WithoutFlipper"
1658+
use_frameworks: "DynamicFrameworks"
16471659
- architecture: "NewArch"
16481660
flavor: "Release"
16491661
jsengine: "JSC"
@@ -1654,11 +1666,21 @@ workflows:
16541666
jsengine: "JSC"
16551667
flipper: "WithFlipper"
16561668
use_frameworks: "StaticFrameworks"
1669+
- architecture: "NewArch"
1670+
flavor: "Release"
1671+
jsengine: "JSC"
1672+
flipper: "WithFlipper"
1673+
use_frameworks: "DynamicFrameworks"
16571674
- architecture: "NewArch"
16581675
flavor: "Release"
16591676
jsengine: "JSC"
16601677
flipper: "WithoutFlipper"
16611678
use_frameworks: "StaticFrameworks"
1679+
- architecture: "NewArch"
1680+
flavor: "Release"
1681+
jsengine: "JSC"
1682+
flipper: "WithoutFlipper"
1683+
use_frameworks: "DynamicFrameworks"
16621684
- architecture: "OldArch"
16631685
flavor: "Release"
16641686
jsengine: "Hermes"
@@ -1669,6 +1691,11 @@ workflows:
16691691
jsengine: "Hermes"
16701692
flipper: "WithFlipper"
16711693
use_frameworks: "StaticFrameworks"
1694+
- architecture: "OldArch"
1695+
flavor: "Release"
1696+
jsengine: "Hermes"
1697+
flipper: "WithFlipper"
1698+
use_frameworks: "DynamicFrameworks"
16721699
- architecture: "OldArch"
16731700
flavor: "Release"
16741701
jsengine: "JSC"
@@ -1679,6 +1706,11 @@ workflows:
16791706
jsengine: "JSC"
16801707
flipper: "WithFlipper"
16811708
use_frameworks: "StaticFrameworks"
1709+
- architecture: "OldArch"
1710+
flavor: "Release"
1711+
jsengine: "JSC"
1712+
flipper: "WithFlipper"
1713+
use_frameworks: "DynamicFrameworks"
16821714
- architecture: "NewArch"
16831715
flavor: "Debug"
16841716
jsengine: "Hermes"
@@ -1709,6 +1741,36 @@ workflows:
17091741
jsengine: "JSC"
17101742
flipper: "WithFlipper"
17111743
use_frameworks: "StaticFrameworks"
1744+
- architecture: "NewArch"
1745+
flavor: "Debug"
1746+
jsengine: "Hermes"
1747+
flipper: "WithFlipper"
1748+
use_frameworks: "DynamicFrameworks"
1749+
- architecture: "NewArch"
1750+
flavor: "Debug"
1751+
jsengine: "Hermes"
1752+
flipper: "WithoutFlipper"
1753+
use_frameworks: "DynamicFrameworks"
1754+
- architecture: "NewArch"
1755+
flavor: "Debug"
1756+
jsengine: "JSC"
1757+
flipper: "WithFlipper"
1758+
use_frameworks: "DynamicFrameworks"
1759+
- architecture: "NewArch"
1760+
flavor: "Debug"
1761+
jsengine: "JSC"
1762+
flipper: "WithoutFlipper"
1763+
use_frameworks: "DynamicFrameworks"
1764+
- architecture: "OldArch"
1765+
flavor: "Debug"
1766+
jsengine: "Hermes"
1767+
flipper: "WithFlipper"
1768+
use_frameworks: "DynamicFrameworks"
1769+
- architecture: "OldArch"
1770+
flavor: "Debug"
1771+
jsengine: "JSC"
1772+
flipper: "WithFlipper"
1773+
use_frameworks: "DynamicFrameworks"
17121774
- test_ios_rntester:
17131775
requires:
17141776
- build_hermes_macos

0 commit comments

Comments
 (0)