Skip to content

Commit 00d9fa4

Browse files
Apply suggestions from code review
Co-authored-by: Brentley Jones <github@brentleyjones.com> Signed-off-by: Luis Padron <heyluispadron@gmail.com>
1 parent f676c07 commit 00d9fa4

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

test/internal/pbxproj_partials/write_pbxproj_prefix_tests.bzl

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,9 @@ def _write_pbxproj_prefix_test_impl(ctx):
5353
import_index_build_indexstores = (
5454
ctx.attr.import_index_build_indexstores
5555
),
56-
legacy_index_import = ctx.attr.legacy_index_import,
5756
index_import = ctx.attr.index_import,
5857
install_path = "a/project.xcodeproj",
58+
legacy_index_import = ctx.attr.legacy_index_import,
5959
minimum_xcode_version = ctx.attr.minimum_xcode_version,
6060
platforms = ctx.attr.platforms,
6161
post_build_script = ctx.attr.post_build_script,
@@ -145,8 +145,8 @@ write_pbxproj_prefix_test = unittest.make(
145145
"default_xcode_configuration": attr.string(mandatory = True),
146146
"execution_root_file": attr.string(mandatory = True),
147147
"import_index_build_indexstores": attr.bool(mandatory = True),
148-
"legacy_index_import": attr.string(mandatory = True),
149148
"index_import": attr.string(mandatory = True),
149+
"legacy_index_import": attr.string(mandatory = True),
150150
"minimum_xcode_version": attr.string(mandatory = True),
151151
"platforms": attr.string_list(mandatory = True),
152152
"post_build_script": attr.string(),
@@ -182,8 +182,8 @@ def write_pbxproj_prefix_test_suite(name):
182182
default_xcode_configuration,
183183
execution_root_file,
184184
import_index_build_indexstores,
185-
legacy_index_import,
186185
index_import,
186+
legacy_index_import,
187187
minimum_xcode_version,
188188
platforms,
189189
post_build_script = None,
@@ -207,8 +207,8 @@ def write_pbxproj_prefix_test_suite(name):
207207
default_xcode_configuration = default_xcode_configuration,
208208
execution_root_file = execution_root_file,
209209
import_index_build_indexstores = import_index_build_indexstores,
210-
legacy_index_import = legacy_index_import,
211210
index_import = index_import,
211+
legacy_index_import = legacy_index_import,
212212
minimum_xcode_version = minimum_xcode_version,
213213
platforms = platforms,
214214
post_build_script = post_build_script,
@@ -237,8 +237,8 @@ def write_pbxproj_prefix_test_suite(name):
237237
default_xcode_configuration = "Debug",
238238
execution_root_file = "an/execution/root/file",
239239
import_index_build_indexstores = True,
240-
legacy_index_import = "some/path/to/legacy/index_import",
241240
index_import = "some/path/to/index_import",
241+
legacy_index_import = "some/path/to/legacy/index_import",
242242
minimum_xcode_version = "14.2.1",
243243
platforms = [
244244
"MACOS",
@@ -303,8 +303,8 @@ def write_pbxproj_prefix_test_suite(name):
303303
default_xcode_configuration = "Release",
304304
execution_root_file = "an/execution/root/file",
305305
import_index_build_indexstores = False,
306-
legacy_index_import = "some/path/to/legacy/index_import",
307306
index_import = "some/path/to/index_import",
307+
legacy_index_import = "some/path/to/legacy/index_import",
308308
platforms = [
309309
"MACOS",
310310
"IOS_DEVICE",

tools/generators/pbxproj_prefix/test/PBXProjectBuildSettingsTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ class PBXProjectBuildSettingsTests: XCTestCase {
1111

1212
let config = "rxcp_custom_config"
1313
let importIndexBuildIndexstores = false
14-
let legacyIndexImport = "external/legacy-index-import"
14+
let legacyIndexImport = "external/legacy-index-import"
1515
let indexImport = "external/index-import"
1616
let indexingProjectDir = "/some/indexing/project dir"
1717
let projectDir = "/some/project dir"
@@ -85,7 +85,7 @@ class PBXProjectBuildSettingsTests: XCTestCase {
8585
let buildSettings = Generator.pbxProjectBuildSettings(
8686
config: config,
8787
importIndexBuildIndexstores: importIndexBuildIndexstores,
88-
legacyIndexImport: legacyIndexImport,
88+
legacyIndexImport: legacyIndexImport,
8989
indexImport: indexImport,
9090
indexingProjectDir: indexingProjectDir,
9191
projectDir: projectDir,

0 commit comments

Comments
 (0)