Skip to content

Commit

Permalink
Merge branch 'main' into loremattei/test-sync-e2e-on-robin
Browse files Browse the repository at this point in the history
  • Loading branch information
loremattei committed Nov 20, 2024
2 parents d081082 + 6642b4b commit a7cbe8b
Show file tree
Hide file tree
Showing 172 changed files with 5,078 additions and 2,534 deletions.
7 changes: 0 additions & 7 deletions .github/workflows/alpha.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,6 @@ jobs:

steps:

- name: Assert main branch
run: |
case "${{ github.ref }}" in
*main) ;;
*) echo "👎 Not main branch"; exit 1 ;;
esac
- name: Register SSH keys for access to certificates
uses: webfactory/[email protected]
with:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/end-to-end.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ jobs:
run: |
set -o pipefail && xcodebuild \
-scheme "DuckDuckGo" \
-destination "platform=iOS Simulator,name=iPhone 15,OS=17.2" \
-destination "platform=iOS Simulator,name=iPhone 16,OS=18.1" \
-derivedDataPath "DerivedData" \
-skipPackagePluginValidation \
-skipMacroValidation \
Expand All @@ -68,7 +68,7 @@ jobs:
end-to-end-tests:
name: End to end Tests
needs: build-end-to-end-tests
runs-on: macos-14-xlarge
runs-on: macos-14
timeout-minutes: 90
strategy:
matrix:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
run: |
set -o pipefail && xcodebuild test \
-scheme "AtbUITests" \
-destination "platform=iOS Simulator,name=iPhone 15,OS=17.2" \
-destination "platform=iOS Simulator,name=iPhone 16,OS=18.1" \
-derivedDataPath "DerivedData" \
-skipPackagePluginValidation \
-skipMacroValidation \
Expand Down Expand Up @@ -87,7 +87,7 @@ jobs:
run: |
set -o pipefail && xcodebuild test \
-scheme "FingerprintingUITests" \
-destination "platform=iOS Simulator,name=iPhone 15,OS=17.2" \
-destination "platform=iOS Simulator,name=iPhone 16,OS=18.1" \
-derivedDataPath "DerivedData" \
-skipPackagePluginValidation \
-skipMacroValidation \
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ jobs:

name: Unit Tests

runs-on: macos-14-xlarge
timeout-minutes: 15
runs-on: macos-15
timeout-minutes: 20

outputs:
commit_author: ${{ steps.fetch_commit_author.outputs.commit_author }}
Expand Down Expand Up @@ -79,7 +79,7 @@ jobs:
run: |
set -o pipefail && xcodebuild test \
-scheme "DuckDuckGo" \
-destination "platform=iOS Simulator,name=iPhone 15,OS=17.2" \
-destination "platform=iOS Simulator,name=iPhone 16,OS=18.1" \
-derivedDataPath "DerivedData" \
-skipPackagePluginValidation \
-skipMacroValidation \
Expand Down Expand Up @@ -132,7 +132,7 @@ jobs:
# Dependabot doesn't have access to all secrets, so we skip this job
if: github.actor != 'dependabot[bot]'

runs-on: macos-14-xlarge
runs-on: macos-15
timeout-minutes: 30

steps:
Expand Down Expand Up @@ -185,7 +185,7 @@ jobs:
set -o pipefail && xcodebuild \
-scheme "DuckDuckGo" \
-destination "platform=iOS Simulator,name=iPhone 15" \
-destination "platform=iOS Simulator,name=iPhone 16" \
-derivedDataPath "DerivedData" \
-configuration "Release" \
-skipPackagePluginValidation \
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/sync-end-to-end.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ jobs:
run: |
set -o pipefail && xcodebuild \
-scheme "DuckDuckGo" \
-destination "platform=iOS Simulator,name=iPhone 15" \
-destination "platform=iOS Simulator,name=iPhone 16" \
-derivedDataPath "DerivedData" \
-skipPackagePluginValidation \
-skipMacroValidation \
Expand All @@ -68,7 +68,7 @@ jobs:
sync-end-to-end-tests:
name: Sync End To End Tests
needs: build-for-sync-end-to-end-tests
runs-on: macos-14-xlarge
runs-on: macos-14
timeout-minutes: 90
strategy:
matrix:
Expand Down
2 changes: 1 addition & 1 deletion .maestro/release_tests/emailprotection.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ tags:
- scroll
- assertVisible: Email Protection
- tapOn: Email Protection
- assertVisible: Email privacy, simplified.
- assertVisible: Email privacy, protected.
- assertVisible:
id: searchEntry
- tapOn:
Expand Down
6 changes: 3 additions & 3 deletions .maestro/setup_ui_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ source $(dirname $0)/common.sh
## Constants

# The simulator command requires the hyphens
target_device="iPhone-15"
target_os="iOS-17-0"
target_device="iPhone-16"
target_os="iOS-18-1"

## Functions

Expand Down Expand Up @@ -50,7 +50,7 @@ build_app() {
echo "⏲️ Building the app"
set -o pipefail && xcodebuild -project "$project_root"/DuckDuckGo.xcodeproj \
-scheme "DuckDuckGo" \
-destination "platform=iOS Simulator,name=iPhone 15,OS=17.2" \
-destination "platform=iOS Simulator,name=iPhone 16,OS=18.1" \
-derivedDataPath "$derived_data_path" \
-skipPackagePluginValidation \
-skipMacroValidation \
Expand Down
4 changes: 3 additions & 1 deletion .maestro/shared/remove_local_logins.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ appId: com.duckduckgo.mobile.ios
---

- tapOn: Settings
- scrollUntilVisible:
element: Passwords
- tapOn: Passwords
- tapOn: Passcode field
- inputText: "0000"
Expand All @@ -10,4 +12,4 @@ appId: com.duckduckgo.mobile.ios
- tapOn: Delete Password
- tapOn: Delete Password
- tapOn: Settings
- tapOn: Done
- tapOn: Done
2 changes: 1 addition & 1 deletion .ruby-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3.0.4
3.3.4
2 changes: 1 addition & 1 deletion .xcode-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
15.4
16.1
2 changes: 1 addition & 1 deletion Configuration/Version.xcconfig
Original file line number Diff line number Diff line change
@@ -1 +1 @@
MARKETING_VERSION = 7.144.0
MARKETING_VERSION = 7.146.0
4 changes: 2 additions & 2 deletions Core/AppPrivacyConfigurationDataProvider.swift
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ import BrowserServicesKit
final public class AppPrivacyConfigurationDataProvider: EmbeddedDataProvider {

public struct Constants {
public static let embeddedDataETag = "\"516f95a16f7a556c58e14ee6f193cc30\""
public static let embeddedDataSHA = "87314e1ac02784472a722844a27b443b0387a164ac72afaac00d9a70731fc572"
public static let embeddedDataETag = "\"d4536be677d4324aa872d0e59d6fd4fa\""
public static let embeddedDataSHA = "a9cce4d5c3430bc3bbca33ad26aa980dcc1c0e792b71ce2696f4483401efbf72"
}

public var embeddedDataEtag: String {
Expand Down
16 changes: 14 additions & 2 deletions Core/FeatureFlag.swift
Original file line number Diff line number Diff line change
Expand Up @@ -45,13 +45,21 @@ public enum FeatureFlag: String {
case onboardingAddToDock
case autofillSurveys
case autcompleteTabs
case textZoom
case adAttributionReporting

/// https://app.asana.com/0/72649045549333/1208231259093710/f
case networkProtectionUserTips

/// https://app.asana.com/0/72649045549333/1208617860225199/f
case networkProtectionEnforceRoutes
}

extension FeatureFlag: FeatureFlagSourceProviding {
extension FeatureFlag: FeatureFlagDescribing {
public var supportsLocalOverriding: Bool {
false
}

public var source: FeatureFlagSource {
switch self {
case .debugMenu:
Expand Down Expand Up @@ -104,6 +112,10 @@ extension FeatureFlag: FeatureFlagSourceProviding {
return .remoteReleasable(.feature(.autocompleteTabs))
case .networkProtectionUserTips:
return .remoteReleasable(.subfeature(NetworkProtectionSubfeature.userTips))
case .textZoom:
return .remoteReleasable(.feature(.textZoom))
case .networkProtectionEnforceRoutes:
return .remoteDevelopment(.subfeature(NetworkProtectionSubfeature.enforceRoutes))
case .adAttributionReporting:
return .remoteReleasable(.feature(.adAttributionReporting))
}
Expand All @@ -112,6 +124,6 @@ extension FeatureFlag: FeatureFlagSourceProviding {

extension FeatureFlagger {
public func isFeatureOn(_ featureFlag: FeatureFlag) -> Bool {
return isFeatureOn(forProvider: featureFlag)
return isFeatureOn(for: featureFlag)
}
}
6 changes: 4 additions & 2 deletions Core/Pixel.swift
Original file line number Diff line number Diff line change
Expand Up @@ -86,8 +86,9 @@ public struct PixelParameters {
public static let count = "count"
public static let source = "source"

public static let textSizeInitial = "text_size_initial"
public static let textSizeUpdated = "text_size_updated"
// Text size is the legacy name
public static let textZoomInitial = "text_size_initial"
public static let textZoomUpdated = "text_size_updated"

public static let canAutoPreviewMIMEType = "can_auto_preview_mime_type"
public static let mimeType = "mime_type"
Expand Down Expand Up @@ -142,6 +143,7 @@ public struct PixelParameters {
public static let adAttributionKeywordID = "keyword_id"
public static let adAttributionAdID = "ad_id"
public static let adAttributionToken = "attribution_token"
public static let adAttributionIsReinstall = "is_reinstall"

// Autofill
public static let countBucket = "count_bucket"
Expand Down
Loading

0 comments on commit a7cbe8b

Please sign in to comment.