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.

Sign up for GitHub

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

Jump to bottom

Fix timezone uninitialized warning in integration test #1046

Merged
merged 2 commits into from
Mar 8, 2023
Merged

Fix timezone uninitialized warning in integration test #1046

merged 2 commits into from
Mar 8, 2023

Conversation

Eldar2021
Copy link
Member

@Eldar2021 Eldar2021 commented Mar 8, 2023 •
edited by clangenb
Loading

Fix timezone uninitialized warning in integration test (app test)

E/flutter ( 3321): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: LateInitializationError: Field '_local@447310200' has not been initialized.
E/flutter ( 3321): #0      _local (package:timezone/src/env.dart)
E/flutter ( 3321): #1      local (package:timezone/src/env.dart:28:23)
E/flutter ( 3321): #2      _EncointerHomePageState.initState.<anonymous closure> (package:encointer_wallet/page-encointer/home_page.dart:39:12)
E/flutter ( 3321): <asynchronous suspension>

I noticed this error give only when we test test_driver/app.dart. Because in test_driver/app.dart file we didn’t call await NotificationPlugin.setup();

@Eldar2021 Eldar2021 linked an issue Mar 8, 2023 that may be closed by this pull request
tz.local not not configured warning in integration tests #1037
Closed
@Eldar2021 Eldar2021 self-assigned this Mar 8, 2023
@clangenb clangenb changed the title Fix timezone initialized warning in integration test Fix timezone uninitialized warning in integration test Mar 8, 2023
clangenb
clangenb approved these changes Mar 8, 2023
View reviewed changes
Copy link
Member

@clangenb clangenb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, thanks for the simple fix! I can confirm that I don't see the logs in the integration tests!

@clangenb clangenb merged commit 34c4ca4 into master Mar 8, 2023
@Eldar2021 Eldar2021 deleted the el/timezone-local-warning-integration-tests branch March 9, 2023 05:29
@SourbaevaJanaraJ SourbaevaJanaraJ added A2-technical PR introduces technical changes B0-low Does not elevate a release containing this beyond "low priority" C0-breaksnothing PR does not introduce any breaking changes labels Mar 23, 2023
clangenb added a commit that referenced this pull request Apr 1, 2023
@clangenb @pifragile
@Eldar2021 @SourbaevaJanaraJ @mosonyi @brenzi @azackmatoff @haerdib
Merge master to fdroid (#1099)
133feb9 
* potential fix for deeplinks (#973)

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Add integration test: reputable endorses newbie  (#972)

* success send money to Bin

* try test

* success test

* add test check registration status

* tested on IOS

* fix typo accont -> account

* added comment for store.appcast

* Enable lint: `avoid_dynamic_calls` (#956)

* change true avoid_dynamic_calls

* try fix avoid_dynamic_calls part 01

* try fix avoid_dynamic_calls part 02

* try fix avoid_dynamic_calls part 03

* try fix avoid_dynamic_calls part 04 (remove unused methods)

* try fix avoid_dynamic_calls part 04

* format code

* [encointer_api] extract helper function

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Placeholder page for transfer history (#974)

* create folder

* added store for transfer history view

* added TransferHistoryView to app router

* added icon button to transfer page

* started ui of the transactionPage

* ui draft of the transactionPage

* create trnsaction card

* canged emun values name

* added int field timestamp

* the button to go to transfer history behind the developer settings

* added traslations

* changed color icons

* format code

* added date time view

* changed incoming color to green

* fix translations

* remove unused import

---------

Co-authored-by: Janara Sourbayeva <janara2610@gmail.com>

* replace `store.appCast == null` with `store.isTestMode` (#976)

* added AppcastConfiguration to AppConfig

* fix tests

* remove initialroute and app comments all vars for app config

* Bump flutter v3.7.0 (#977)

* fix depracated api

* remove TapTooltip

* fix generate JsonSerializable

* tested on IOS and Android

* fix launch json

* fix unit test

* added includeFromJson: false to JsonKey

* move readme content from github folder int root readme (#982)

* Remove dynamic casts (#978)

* open ignore dynamic casts

* remove 4 unused methods

* try fix 01

* commented unused runction

* fix deeplink warnings

* try fix 02

* remove unused computed vars (SettingsStore)

* remove CodecApi

* remove unused code

* remove unused code 02

* remove unused codes 03

* remove work manager error

* fix dynamic casts 04

* try fix dynamic casts 05

* tested :)

* fix by comments

* Update issue template: add slack release info

* Readme: Make code snippets copyable and add remark about outdated `.flutter` submodule (#984)

* [Readme] make code snippets copyable

* [Readme] add comment about fixing the `.flutter` submodule

* [Readme] add shell lexer to code snippets for syntax highlighting

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Bump flutter v3.7.1 (#991)

* bump flutter version

* bump package and sdk

* tested

* enabled more lint (#996)

* cloned app

* enabled  no_default_cases: true

* Bump flutter v3.7.3 (#1001)

* bump flutter v

* added tested

* Introduce melos for workspace and subpackage managing (#983)

* create two package

* create melos.yaml file

* try activate melos 01

* try activate melos 02

* try activate melos 03

* try activate melos 04

* try activate melos 05

* try activate melos 06

* try activate melos 07

* try activate melos 08

* try activate melos 09

* try activate melos 10

* write script

* write script test

* write script test 01

* write script test 02

* write script test 03

* write script test 04

* write script test 05

* write script test 06

* delete comments from actions

* refactor README.md file

* write descriptions for melos commands

* Melos use flutterw (#985)

* add el-melos-packages unit and android build CI

* try use select package

* fix file exist to scope

* added export path for molos bootstrap

* try fix unit test

* try fix unit test 01

* remove Melos_sdk_path

* formatted code

* changed ./fluttew and ../../flutterw to flutter

* use for flutter actions

* try fix melos generate build runner

* Use Melos command for all GitHub Actions (#986)

* change Build JS to melos command

* remove some comment

* fix README.md file

* remove el-melos-packages from github actions

* fix by comment (#989)

* Readme: Make code snippets copyable and add remark about outdated `.flutter` submodule (#984)

* [Readme] make code snippets copyable

* [Readme] add comment about fixing the `.flutter` submodule

* [Readme] add shell lexer to code snippets for syntax highlighting

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Make melos_install.sh executable

* romove .gitignore files from packages

* remove coverage files from packages and fix readme md files

* remove export from melos install command

* romeve el-melos-packages from actions

---------

Co-authored-by: Zhanara <91150590+SourbaevaJanaraJ@users.noreply.github.com>
Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* fix real test(choosing cid change index)

* fix read me command fields

* Melos cache fixes and fixes for windows. (#992)

* Bump flutter v3.7.1 (#991)

* bump flutter version

* bump package and sdk

* tested

* add melos as dev-dependency

* fix some commands for windows shell and remove need for cache re-export.

* [GA] check if unit tests work with new melos config

* execute CI on on PR into el-melos-packages branch

* remove accidentally added readme section

* [melos] fix yarn build

* [melos] fix build-runner command

* [GA] remove config for build on `el-melos-package` branch because it works.

* fix qrScanHintAccount text

* fix readme files in packages

* added el-melos-packages to some actions

* fix melos commands

* added windows command to readme md

* remove el-melos-packages

---------

Co-authored-by: Eldiiar Almazbek <76556278+Eldar2021@users.noreply.github.com>
Co-authored-by: Eldar2021 <eldiiaralmazbekov@gmail.com>

* Melos fix by comment (#998)

* created branch

* fix android build action

* try text remove  in install melos

* replate ganarate-build-runner to run-build-runner

* fix melos commands

* fix unit test

* remove el-melos-packages from actions

* remove export from melos install script

---------

Co-authored-by: Zhanara <91150590+SourbaevaJanaraJ@users.noreply.github.com>
Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Remove some obsolete files (#1003)

* Change structure: root-directory is only for the workspace; app in dedicated directory (#1005)

* move app stuff from root to app

* add root workspace

* fix errors that android studio introduced when moving files

* re-add deleted settings_aar.gradle

* el/fix-melos-commands (#1007)

* try fix yarn build

* try test js

* get permission

* try fix melos build-apk

* try fix android integration test

* try fix ios integration test and fdroid test

* added install melos for ffroid actions

* added install melos for ffroid actions

* fix melos commands for app

* remove branch name (cl/fix-melos-for-android-studio) from hithub actions

* fixed vs configurations for new folder structure, run android and ios with vd code rummer

* remove melos from the app package

* fix melos multiline commands for windows

* fix command namings

* better documentation

* rename `melos_install` to `install_melos`

* Fix appcenter app-release (#1009)

* fix android appcenter post clone.

* fix pwd in ios appcenter-post-clone.sh

* run fdroid build with pure flutter again.

* better naming for build fdroid melos command

* fix permission for `install_melos.sh`

* github actions: fix melos command name

* [GA] remove unnecessary stuff from the github actions pipeline.

* Remove versioning and other unnecessary fields from workspace pubspec.yaml

---------

Co-authored-by: Eldiiar Almazbek <76556278+Eldar2021@users.noreply.github.com>

* el/add-greenbay-link-to-ceremony-box (#1004)

* try add greenbay link for info icon button

* changed onPressedInfo to onInfoPressed

* fix README.md file

* added real greenbay cid to case

* Remove obsolete fields from the accountStore (#1010)

* [accountStore] remove obsolete address icons map

* [accountStore] remove obsolete pubkeyIconsMap

* el/participants-avatars (#997)

* added animals avatars

* changed animals icons

* fixed assets

* fix unactive color avatar

* success :)

* depracated color property

* set by default false ParticipantAvatar

* fix by comment

* fixed by comment

* added MeetupAvatar

* fix by comment

* fix conflicts

* [JS] remove the codec module. (#1013)

* [AddressInputField] simplify code and and add account filtering based on name (#1017)

* [AddressInputField] simplify code and allow filtering based on address or name.

* Remove the support for account index whatsoever.

* [AddressInputField] use the widget as it is supposed to be used.

* [AddressInputField] remove unnecessary observers.

* [AddressInputField] remove unnecessary comment

* [AddressInputField] make assumptions that names are never empty.

* add todo for ss58-prefix handling

* better documentation

* Simplify network select page and remove `accountIndexMap` from account store (#1018)

* [NetworkSelectPage] fix icon size

* [NetworkSelectPage] remove option to create an account

* [NetworkSelectPage] remove option to create an account

* [NetworkSelectPage] remove account index handling, also remove the `accountIndexMap` from the account store.

* add root pubspec to repository (#1025)

* Fix broken invoice link: handle missing fields (#1028)

* fix problem

* some refactor

* Generate assets paths with flutter_gen (#1030)

* cloned app

* changed png names and integrated flutterGen

* deleted unused pngs

* test

* deleted unused pngs, and and used FlutterGen format in files

* fixed by comments from github review

* restores substrate.png for test

* renamed assets

* renamed default.png to polkadot_greyed_out.png

* deleted ceremonies_dark.png and ceremenies_indigo.png

* test

* last try

* git config core.ignorecase false

* git config core.ignorecase false

* test

* try fix

* Add assetlinks.json #1031 (#1036)

* fix broken paths in readme images #1014  (#1032)

* cloned app

* fixed broken images in README.md file

* changed width image from 300 to 32%

* use relative paths for the prints screens in readme and remove obsolete ceremony view printscreen.

* remove another printscreen so that we only use one line in the readme

* remove extra newline

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Remove unused files and Instructions View (#1033)

* deleted instruction module

* remove unused files

* remove dart code metrics

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Avoid unawaited futures (lint) (#995)

* enabled unawaited_futures lint

* some more changes

* commented addTxs method in assets api

* fix connect unwaited lint

* last changes

* changed in ceremony_box.dart

* changes in dart_api.dart

* minor changes in dart_api.dart

* remove commented code.

* [community_store_test] use await instead of unawaited.

* [dart_api] make `connect` and `reconnect` synchronous.

* [community_store_test] fix unit test.

---------

Co-authored-by: Eldar2021 <eldiiaralmazbekov@gmail.com>
Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* fixed encointer-network icons assetsby switch case func (#1040)

* renaming avatars (#1041)

* Changed export account icon from trash Icon to export Icon (#1043)

* changed export account icon to exportIcon

* named parameters constructor

* added const

* try fix timezone initial in integration test (#1046)

* simplify build setup instructions for ubuntu (#1049)

* Change contact name (#1048)

* try user can change contact name

* added textTheme variable

* added integration test

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Refactor account creation process (#1027)

* try refactor 01

* try refactor 02

* fox imports

* pull origin master

* fix try account create

* try fix create account proccess

* try fix add account

* created import account view

* remove import account store

* try fix import account proccess

* refactoring account create store

* create navigate functions

* fix navigate function

* deleted commented code

* fix tets

* changed store name

* change file name

* fix imports

* create navigate mixin

* change mixin name

* fix ProgressingIndicator width

* fix conflicts

* fix border size pin form

* create add account view

* added logs

* renaming store -> please rename to newAccountStore

* change mixin name

* changed function name

* create NewAccountResult

* remove setLoading function

* change loadiing true

* refactor _formatMetaData

* deleted commented AccountCreate();

* remove custom button

* change ProgressingIndicator to CenteredActivityIndicator

* fixed newAccountStore nameing

* renaming FormScrollable to ScrollableForm

* fix dailog to dialog

* renaming showInputPasswordDailog to showPasswordInputDialog

* use readonly for loading var

* added acc to NewAccountResult for saveAccount

* added assert for acoounKey is not null and is not empty conditions

* try remove _onDuplicateAccount lines

* upgrade melos v3.0.0 (#1055)

* deleted unused code (#1054)

* Fix validate account name (#1053)

* try fix validate account name

* try fix by comment

* use accountList

* try fix account creation texts (#1061)

* az/Move js_encointer outside lib (#1064)

* az/Move js_encointer outside lib

* "synced .flutter with master"

* "deleted android and ios folders outside app"

* formatted gen'd files

* Meetup point map blanks when zooming in too much (#1051)

* added _mapController

* fixed late error

* refactoring mapController, onPointerDown

* added localization to textButtons

* changed showDialog contentText

* added move method

* refactor EncointerMap constructor

* deleted cancel word

* changed alertDialog to cupertinoDialog, added translations to homeTranslation

* added ensureZoomWithinLimits

* relocated ensureZoomWithinLimits in side build meth

* test

* test

* refactoring ensureZoomWithinLimits

* changed initial zoom, and cupertinoDialog text

* changed alertDialog name

* fix translations

---------

Co-authored-by: Eldar2021 <eldiiaralmazbekov@gmail.com>
Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Fix qr code gets cropped on some screens (#1062)

* try fix cropped

* deleted CrossAxisAlignment.stretch

* fix naming gr_code -> qr_code

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Localized translations for transaction results (#1065)

* added translations

* added translations to params

* tested

* try test

* remove notifySubmitted

* remove null txInfo

* tested changes

* added generics

* udpated translations for the transaction confirmations

* deleted as String keyword

* added txError field

* deleted _senTx method

* fix ru translations

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Add CI label check (#1068)

* add check-labels.yml

* add strategy

* update naming

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Allow participants to unregister (#1066)

* added unregisterParticipantParams

* added UnregisteredLinkButton widget

* created submitUnRegisterParticipant functions

* added translations

* added integration test

* refactor ceremony box widgte

* refactor UnregisteredLinkButton

* remove comments

* fix unregistering from a gathering by correctly supplying the transaction params (#1067)

* fix unregistering from a gathering by correctly supplying the transaction params.

* add comment

* more ergonomic statement.

* set last proof of attendance to null when we enter the next ceremony.

* remove unnecessary nullcheck

* fix naming unRegister to unregister

* fix UnregisteredLinkButton

* fixed translations by comment

* added Unregisteri ntegration test for Bootstrapper, Newbie, Reputable, Endorsee

* added TranslationsTx fot translation transactions

* added translations for unregister participant

* integrated unregister translations

* rename cancelValue to onCancelValue

* fix onCancelValue

* fix translations by comment

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Fix flaky integration test: share and change account name (#1076)

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Fix packages test for github actions (#1082)

* try fix

* for test

* Hide mnemonic and rawSeed in logs (#1063)

* az/Hide sensitive info on logs

* Excluded "EncointerJsService" logs from WebView

* [log_service] only filter mnemonic and raw seed.

---------

Co-authored-by: clangenb <37865735+clangenb@users.noreply.github.com>

* Add dart pubKey -> ss58-address encoding (#1052)

* [format] add draft of ss58 codec for dart. Something is wrong with the blake2b hash.

* [format] try ss58 codec with second hasher, doesn't work yet.

* add more documentation

* remove one of the ss58 tries

* try mirroring the rust code.

* remove obsolete blake2b dep

* improve readability

* fix prefix error use the function argument

* replace blake2 library. Blake2 simple blake 2 hash is now correct. The address still fails.

* [Fmt] add method: blake2WithSs58Pre, which works.

* fix ss58 encode test

* cleanup

* extend documentation

* [Fmt] rename `SS58PRE` to `ss58Prefix`

* [Fmt] add unit tests for prefix two.

* [Fmt] assert that the prefix is < 64

* [Fmt] better test documentation

* [CI] add automatic draft release generation (#1085)

* add release drafter

* add collapse to high priority

* Reaping the voucher uses the new `encointerBalances.transferAll` extrinsic. (#1090)

* [JS] the voucher uses now transfer all.

* extract bash script for the boostrap demo community command

* [GHA] bootstrap demo community for the encointer client

* rename:  run_encointer_node_notee_docker.sh docker_run_encointer_node_notee.sh

* add newline to docker run script

* fix encointer client docker on linux.

* fix new js unit test

* v1.9.1

* [IOS] delete ios because it is not needed in fdroid and there are always conflicts when merging master

* fix merge errors

* add ios folder again because it is the most reliable integration test.

* deleted leftover filesystem metadata files

* Fix `melos bootstrap` error in github actions (#1100)

* try fix error melos bs in github actions

* pull origin master

---------

Co-authored-by: pifragile <pigu@bluewin.ch>
Co-authored-by: Eldiiar Almazbek <76556278+Eldar2021@users.noreply.github.com>
Co-authored-by: Janara Sourbayeva <janara2610@gmail.com>
Co-authored-by: Zhanara <91150590+SourbaevaJanaraJ@users.noreply.github.com>
Co-authored-by: mosonyi <zoltan@integritee.network>
Co-authored-by: Eldar2021 <eldiiaralmazbekov@gmail.com>
Co-authored-by: brenzi <brenzi@users.noreply.github.com>
Co-authored-by: AZ Ackmatoff <ackmatoff@gmail.com>
Co-authored-by: Bigna Härdi <73821294+haerdib@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clangenb clangenb clangenb approved these changes

@azackmatoff azackmatoff Awaiting requested review from azackmatoff

@SourbaevaJanaraJ SourbaevaJanaraJ Awaiting requested review from SourbaevaJanaraJ

Assignees

@Eldar2021 Eldar2021

Labels
A2-technical PR introduces technical changes B0-low Does not elevate a release containing this beyond "low priority" C0-breaksnothing PR does not introduce any breaking changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

tz.local not not configured warning in integration tests
3 participants
@Eldar2021 @clangenb @SourbaevaJanaraJ