Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(cat-voices): Enabling integration tests #1533

Merged
merged 54 commits into from
Jan 28, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
54 commits
Select commit Hold shift + click to select a range
679a6a3
try enabling integration tests
oldGreg5 Jan 16, 2025
a6a9767
try with minimal tests
oldGreg5 Jan 16, 2025
c8d852a
try with minimal tests
oldGreg5 Jan 16, 2025
911bc5c
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 17, 2025
217a600
try with headless=new
oldGreg5 Jan 17, 2025
8d31e5e
try with headless
oldGreg5 Jan 17, 2025
7f96ba1
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 20, 2025
b0cace8
try with updated browsers
oldGreg5 Jan 20, 2025
df34d2e
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 20, 2025
25a883c
cleanup
oldGreg5 Jan 20, 2025
63d99df
revert
oldGreg5 Jan 20, 2025
acbd83e
use branch
oldGreg5 Jan 20, 2025
5e584e9
try 1
oldGreg5 Jan 21, 2025
1c58cd8
try 2
oldGreg5 Jan 21, 2025
d1bd473
try 3
oldGreg5 Jan 21, 2025
5270973
try 4
oldGreg5 Jan 22, 2025
2e01709
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 22, 2025
5fd2feb
try 5
oldGreg5 Jan 22, 2025
5f00091
debug driver logs
oldGreg5 Jan 22, 2025
1f0bcdd
try 2
oldGreg5 Jan 22, 2025
02beec5
try 3
oldGreg5 Jan 23, 2025
de6409f
try 4
oldGreg5 Jan 23, 2025
b19554b
use ci 3.2.33
oldGreg5 Jan 23, 2025
624aecf
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 23, 2025
6e3eb15
run tests on chrome only
oldGreg5 Jan 23, 2025
d7e3c5c
move integration tests to be run nightly
oldGreg5 Jan 23, 2025
47576f9
typo fix
oldGreg5 Jan 23, 2025
2edb20a
running minimal scope
oldGreg5 Jan 23, 2025
bbaf07b
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 23, 2025
044ef31
formatting
oldGreg5 Jan 23, 2025
a839936
formatting
oldGreg5 Jan 23, 2025
9c3b45a
try headless=old
oldGreg5 Jan 24, 2025
0205709
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 24, 2025
6928568
try running as non root
oldGreg5 Jan 24, 2025
42259fc
try 156
oldGreg5 Jan 24, 2025
d47995b
try 157
oldGreg5 Jan 24, 2025
d8ec4ef
try 242
oldGreg5 Jan 24, 2025
b2c634f
try 243
oldGreg5 Jan 24, 2025
8878b13
try 244
oldGreg5 Jan 24, 2025
3fb488f
try 244
oldGreg5 Jan 24, 2025
a0b69d8
Merge remote-tracking branch 'origin/enable-integration-tests' into e…
oldGreg5 Jan 24, 2025
4db402e
try 245
oldGreg5 Jan 24, 2025
6438f55
try 246
oldGreg5 Jan 24, 2025
7466980
try 247 root run
oldGreg5 Jan 25, 2025
4fe2c87
try 248 root run
oldGreg5 Jan 28, 2025
ba10dbb
try 249 root run
oldGreg5 Jan 28, 2025
69ec746
try 250 root run
oldGreg5 Jan 28, 2025
a92e77d
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 28, 2025
e839dd6
try 251
oldGreg5 Jan 28, 2025
5156321
try 252
oldGreg5 Jan 28, 2025
af37e89
cleanup
oldGreg5 Jan 28, 2025
fc40d3b
Merge branch 'main' into enable-integration-tests
oldGreg5 Jan 28, 2025
6503e9a
cleanup
oldGreg5 Jan 28, 2025
1ebcd57
Merge branch 'main' into enable-integration-tests
kukkok3 Jan 28, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .config/dictionaries/project.dic
Original file line number Diff line number Diff line change
Expand Up @@ -357,4 +357,4 @@ xpublic
xvfb
yoroi
stake1u94ullc9nj9gawc08990nx8hwgw80l9zpmr8re44kydqy9cdjq6rq
invalid1u9nlq5nmuzthw3vhgakfpxyq4r0zl2c0p8uqy24gpyjsa6c3df4h6
invalid1u9nlq5nmuzthw3vhgakfpxyq4r0zl2c0p8uqy24gpyjsa6c3df4h6
8 changes: 8 additions & 0 deletions .github/workflows/nightly-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,14 @@ jobs:
command: run
args: ./catalyst-gateway/tests/schemathesis_tests+nightly-test-fuzzer-api

- name: Get integration tests report
uses: input-output-hk/catalyst-forge/actions/run@ci/v1.7.0
if: always()
continue-on-error: true
with:
command: run
args: ./catalyst_voices/apps/voices/integration_test+nightly-test-web-all

- name: Collect and upload test reports
uses: actions/upload-artifact@v4
if: always()
Expand Down
16 changes: 8 additions & 8 deletions catalyst_voices/Earthfile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
VERSION 0.8

IMPORT ../catalyst-gateway AS catalyst-gateway
IMPORT github.com/input-output-hk/catalyst-ci/earthly/flutter:v3.2.31 AS flutter-ci
IMPORT github.com/input-output-hk/catalyst-ci/earthly/flutter:v3.2.33 AS flutter-ci

# repo-catalyst-voices - Creates artifacts of all configuration files,
# packages and folders related to catalyst_voices frontend.
Expand Down Expand Up @@ -32,17 +32,17 @@ filter-openapi:

RUN rm openapi/cat-gateway-api.json

SAVE ARTIFACT openapi/filtered-openapi.json
SAVE ARTIFACT openapi/filtered-openapi.json

# Generates flutter code.
# Generates codes for Catalyst Gateway OpenAPI, Voices Localization and
# VoicesAssets and other packages that depend on code-generator.
# It accepts [save_locally] ARG that when true place the artifacts in the
# proper folders
# proper folders
# It accepts [filter_openapi] ARG that when true filter the openapi spec
# using filters from /packages/internal/catalyst_voices_repositories/openapi-filters.json
code-generator:
ARG save_locally=false
ARG save_locally=false
ARG filter_openapi=true
FROM +builder
LET gen_code_path = lib/generated/api
Expand All @@ -57,7 +57,7 @@ code-generator:
FOR generated_file IN $(find . \( -name "*.g.dart" -o -name "*.freezed.dart" -o -name "*.chopper.dart" -o -name "*.swagger.dart" -o -name "*.openapi.dart" -o -name "*.gen.dart" -o -name "catalyst_voices_localizations*.dart" -o -name "cat_gateway_api.*.swagger.*" \))
SAVE ARTIFACT $generated_file AS LOCAL $generated_file
END
ELSE
ELSE
SAVE ARTIFACT .
END
WORKDIR packages/internal/catalyst_voices_repositories
Expand All @@ -75,12 +75,12 @@ code-generator:

# restore current working directory to the root of the project
WORKDIR /frontend


# Runs static analysis on the code.
check-static-analysis:
FROM +code-generator
DO flutter-ci+ANALYZE
DO flutter-ci+ANALYZE

# Runs code formatting checks.
check-code-formatting:
Expand Down Expand Up @@ -132,4 +132,4 @@ docker:
ARG container="voices"
ARG tag="latest"

SAVE IMAGE ${container}:${tag}
SAVE IMAGE ${container}:${tag}
12 changes: 6 additions & 6 deletions catalyst_voices/apps/voices/integration_test/Earthfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ integration-test-web:
# https://github.com/flutter/flutter/issues/154727
--web-browser-flag=--disable-web-security \
--web-browser-flag=--disable-gpu \
--web-browser-flag=--headless=old \
--web-browser-flag=--headless \
--web-browser-flag=--disable-search-engine-choice-screen \
--profile || echo fail > fail
WAIT
Expand All @@ -42,14 +42,14 @@ integration-test-web:
IF [ -f fail ]
RUN --no-cache echo ""$browser" integration test failed" && \
echo "Printing "$driver" logs..." && \
cat $driver.log && \
exit 1
cat $driver.log
# exit 1
kukkok3 marked this conversation as resolved.
Show resolved Hide resolved
END

# TODO(dtscalac): disabled integration tests due to them being flaky,
# reenable when enable-threads.js workaround is removed
# and https://github.com/fzyzcjy/flutter_rust_bridge/issues/2407 closed
disabled-test-web-all:
# Enable but do not fail the build for now
nightly-test-web-all:
BUILD +integration-test-web \
--browser=chrome \
--browser=firefox
--browser=chrome
Original file line number Diff line number Diff line change
Expand Up @@ -767,19 +767,5 @@ void main() async {
},
);
},
skip: true,
);

patrolWidgetTest(
'visitor - restore - keychain choice screen looks OK',
(PatrolTester $) async {
await $.pumpWidgetAndSettle(App(routerConfig: router));
await $(AppBarPage.getStartedBtn).tap(settleTimeout: Time.long.duration);
await OnboardingPage.detailsPartGetStartedRecoverBtn($).tap();
await OnboardingPage.onboardingScreenLooksAsExpected(
$,
RegistrationState.keychainRestoreChoice,
);
},
);
}
Loading