Skip to content

Commit 75e2a2e

Browse files
authored
Merge pull request #17063 from wordpress-mobile/uitests/un-ignore-screenshots
Un-ignore screenshot tests and exclude them properly instead
2 parents 435db3c + dbf7496 commit 75e2a2e

File tree

5 files changed

+3
-6
lines changed

5 files changed

+3
-6
lines changed

Gemfile.lock

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ PLATFORMS
274274

275275
DEPENDENCIES
276276
fastlane (~> 2)
277-
fastlane-plugin-wpmreleasetoolkit (~> 5.2)
277+
fastlane-plugin-wpmreleasetoolkit (~> 5.4)
278278
nokogiri
279279
rmagick (~> 4.1)
280280

WordPress/src/androidTest/java/org/wordpress/android/ui/screenshots/JPScreenshotTest.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import com.google.android.libraries.cloudtesting.screenshots.ScreenShotter;
1212

1313
import org.junit.ClassRule;
14-
import org.junit.Ignore;
1514
import org.junit.Test;
1615
import org.wordpress.android.BuildConfig;
1716
import org.wordpress.android.R;
@@ -96,7 +95,6 @@ public static String buildScreenshotName(Screenshots screen) {
9695
}
9796
}
9897

99-
@Ignore("Ignored until there's a way to exclude tests on FTL properly, via `--test-targets` option.")
10098
@Test
10199
public void jPScreenshotTest() {
102100
if (BuildConfig.IS_JETPACK_APP) {

WordPress/src/androidTest/java/org/wordpress/android/ui/screenshots/WPScreenshotTest.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import com.google.android.libraries.cloudtesting.screenshots.ScreenShotter;
1010

1111
import org.junit.ClassRule;
12-
import org.junit.Ignore;
1312
import org.junit.Test;
1413
import org.wordpress.android.BuildConfig;
1514
import org.wordpress.android.R;
@@ -50,7 +49,6 @@ public class WPScreenshotTest extends BaseTest {
5049

5150
private DemoModeEnabler mDemoModeEnabler = new DemoModeEnabler();
5251

53-
@Ignore("Ignored until there's a way to exclude tests on FTL properly, via `--test-targets` option.")
5452
@Test
5553
public void wPScreenshotTest() {
5654
if (!BuildConfig.IS_JETPACK_APP) {

fastlane/Pluginfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,4 @@ group :screenshots, optional: true do
77
end
88

99
# gem 'fastlane-plugin-wpmreleasetoolkit', git: 'git@github.com:wordpress-mobile/release-toolkit.git', branch: 'trunk'
10-
gem 'fastlane-plugin-wpmreleasetoolkit', '~> 5.2'
10+
gem 'fastlane-plugin-wpmreleasetoolkit', '~> 5.4'

fastlane/lanes/test.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
version: 28,
2929
test_apk_path: File.join(apk_dir, 'androidTest', 'wordpressVanilla', 'debug', 'org.wordpress.android-wordpress-vanilla-debug-androidTest.apk'),
3030
apk_path: File.join(apk_dir, 'wordpressVanilla', 'debug', 'org.wordpress.android-wordpress-vanilla-debug.apk'),
31+
test_targets: 'notPackage org.wordpress.android.ui.screenshots',
3132
results_output_dir: File.join(PROJECT_ROOT_FOLDER, 'build', 'instrumented-tests')
3233
)
3334
end

0 commit comments

Comments
 (0)