Fix and rewrite sync_project.rb (#3296)

* Fix and rewrite sync_project.rb

Ever since we added multiple platforms, removing a file has been broken.
This fixes that issue.

Also, rewrite target synchronization to make more sense. Essentially now
there's one diff between the filesystem and the groups and then an
explicit diff is done between project files and target contents.

* Run sync_project.rb on post-install

Author: wilhuff

Firestore/Example/Firestore.xcodeproj/project.pbxproj

@@ -3642,6 +3642,7 @@
 			isa = PBXSourcesBuildPhase;
 			buildActionMask = 2147483647;
 			files = (
+				124C933122C16ACB00CA8C2D /* CodableIntegrationTests.swift in Sources */,
 				73866AA12082B0A5009BB4FF /* FIRArrayTransformTests.mm in Sources */,
 				5492E079202154D600B64F25 /* FIRCursorTests.mm in Sources */,
 				5492E075202154D600B64F25 /* FIRDatabaseTests.mm in Sources */,

Firestore/Example/Podfile

@@ -1,3 +1,6 @@
+
+require 'pathname'
+
 # Uncomment the next two lines for pre-release testing on internal repo
 #source 'sso://cpdc-internal/firebase'
 #source 'https://cdn.cocoapods.org/'
@@ -6,6 +9,13 @@ source 'https://cdn.cocoapods.org/'
 
 use_frameworks!
 
+
+post_install do |installer|
+  sync = Pathname.new(__FILE__).dirname.join('../../scripts/sync_project.rb')
+  system(sync.to_s)
+end
+
+
 target 'Firestore_Example_iOS' do
   platform :ios, '8.0'
 

scripts/build.sh

@@ -316,7 +316,7 @@ case "$product-$method-$platform" in
 
     # Firestore_IntegrationTests_iOS require Swift 4, which needs Xcode 9
     # Other non-iOS platforms don't have swift integration tests yet.
-    if [["$platform" != 'iOS' || "$xcode_major" -ge 9 ]]; then
+    if [[ "$platform" != 'iOS' || "$xcode_major" -ge 9 ]]; then
       RunXcodebuild \
           -workspace 'Firestore/Example/Firestore.xcworkspace' \
           -scheme "Firestore_IntegrationTests_$platform" \